14:00:42 #startmeeting cinder 14:00:42 Meeting started Wed Sep 27 14:00:42 2023 UTC and is due to finish in 60 minutes. The chair is whoami-rajat. Information about MeetBot at http://wiki.debian.org/MeetBot. 14:00:42 Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. 14:00:42 The meeting name has been set to 'cinder' 14:00:47 o/ 14:00:48 #topic roll call 14:00:51 o/ 14:01:05 o/ 14:01:26 o/ 14:01:28 o/ 14:01:32 o/ 14:01:55 o/ 14:02:17 #link https://etherpad.opendev.org/p/cinder-bobcat-meetings 14:02:30 o/ 14:02:34 o/ 14:04:05 hello everyone 14:04:09 let's get started 14:04:14 #topic announcements 14:04:24 first, RC2 14:04:29 #link https://lists.openstack.org/pipermail/openstack-discuss/2023-September/035207.html 14:04:52 it's not exactly related to RC2 but we are on the final days of 2023.2 Bobcat cycle 14:05:05 so RC2 is the only project specific thing remaining from this email 14:05:10 there is a patch up for RC2 release 14:05:13 #link https://review.opendev.org/c/openstack/releases/+/896625 14:05:30 we will discuss more about the RC2 candidate patches (that we need to merge by tomorrow) in the topics section 14:05:38 next, Call for Outreachy mentors & projects 14:05:43 #link https://lists.openstack.org/pipermail/openstack-discuss/2023-September/035177.html 14:06:15 so outreachy time is here again and the deadline for sending proposals is 29th september 14:06:28 if anyone is interested to mentor and has a suitable project from cinder side 14:06:37 please feel free to propose it 14:07:19 next, TC and PTL election results 14:07:26 as of now everyone might be aware about the results 14:07:31 but thought of concluding it once more 14:07:39 here is the list of TC and PTL election results 14:07:44 #link https://lists.openstack.org/pipermail/openstack-discuss/2023-September/035171.html 14:07:49 #lik https://lists.openstack.org/pipermail/openstack-discuss/2023-September/035172.html 14:08:46 i think JayF is also the new chair of TC 14:09:02 yep 14:09:17 thanks 14:09:59 ok moving on 14:10:03 next, OpenInfra Live 14:10:08 #link https://lists.openstack.org/pipermail/openstack-discuss/2023-September/035166.html 14:10:35 o/ 14:10:38 so in next week's open infra live, we are going to discuss project updates 14:10:48 I signed up for cinder to provide it 14:10:55 if anyone is interested, they can watch it on youtube 14:10:57 \o/ 14:11:01 date and time: 05th October, 2023 - 1400 UTC 14:12:25 it is mostly going to be related to the cycle highlights + a little more detail on the CVE part 14:12:57 so yeah, if you are interested, do join or watch the recording later 14:13:05 Nice! 14:13:31 :) 14:13:59 finally we have the reminder to add topics for PTG 14:14:17 this will be a weekly thing till PTG 14:14:33 #link https://etherpad.opendev.org/p/caracal-ptg-cinder-planning 14:14:39 here is the planning etherpad 14:16:30 so yeah, please add your topics on time, it will help me arrange them in the best way possible 14:16:53 lastly, here are some upcoming events 14:16:54 Final 2023.2 Bobcat release: October 4, 2023 14:16:54 2024.1 Caracal Virtual PTG: October 23-27, 2023 14:17:04 which is nothing but the bobcat release and caracal PTG 14:17:27 that's it for the announcements 14:17:49 anyone has anything interesting that the team should be aware about 14:19:31 let's move to the topics then 14:19:38 #topic RC2 patches 14:19:46 #link https://etherpad.opendev.org/p/cinder-bobcat-rc-patches 14:20:07 so some of the patches and their backports merged 14:20:09 which is good 14:20:25 let me quickly highlight the patches that are merged in master but no backports have been proposed 14:20:40 just a reminder that these patches won't be part of Bobcat cycle 14:20:50 1. Dell: Dell PowerStore [DOC] : Correct PowerStore NFS driver document to support extending volume (roquej) 14:20:50 master: https://review.opendev.org/c/openstack/cinder/+/893330 14:20:59 2. Dell: Dell PowerFlex : password appears in plain text when creating a volume from an image (roquej) 14:20:59 :master: https://review.opendev.org/c/openstack/os-brick/+/885979 14:21:05 3. Dell: Dell PowerFlex - Rescue instance failing with PowerFlex 4.x (roquej) 14:21:05 master: https://review.opendev.org/c/openstack/os-brick/+/885772 14:21:35 coming to the patch that has a backport 14:21:48 [Nimble] Enable thin provisioning as default (raghavendrat) 14:21:54 #link https://review.opendev.org/c/openstack/cinder/+/896205 14:22:13 I've reviewed it, need another core to merge this one ^ 14:22:26 i reviewed the master patch, will look at this one 14:22:31 i mean review if not appropriate and merge if appropriate 14:22:36 thanks rosmaita ! 14:22:39 thanks Rajat, Brian 14:23:13 no problem 14:23:27 finally let's discuss some of the patches that need review 14:23:33 1. Dell:Generic NFS - Implement disk format validation in volume metadata (roquej) 14:23:34 #link https://review.opendev.org/c/openstack/cinder/+/887081 14:23:41 so this is passing CI now 14:23:52 i think the issue was, the save call wasn't made as an admin 14:24:49 that's correct, it works fine now 14:24:58 cool 14:25:51 I don't have an NFS deployment so won't be able to test it, would be good to know that this works as expected 14:26:04 also this is a change in our generic nfs driver from which other drivers inherit from 14:26:11 so any wrong change can break other vendor drivers as well 14:26:19 I had tested it with NFS 14:26:24 Yes true 14:26:54 has anyone from netapp looked it over? 14:27:02 good question 14:27:37 looks like the netapp ci is ok with the patch 14:27:55 not sure it would actually encounter this change, though 14:28:30 somehow I've failed with NFS scenarios that aren't covered in tempest, so I rely more on some manual scenario tests 14:29:06 would be good if someone from netapp takes a look 14:29:16 that patch might do the wrong thing for NFS drivers that inherit from the generic one 14:29:53 not sure, need to look closer 14:31:03 Sure, we'll take a look at it rajat 14:31:23 that doesn't give a good feeling since tomorrow is the RC2 release 14:31:25 Thanks 14:31:26 thanks msaravan ! 14:31:59 this one seems covered 14:32:02 let's move to the next one 14:32:06 2. Dell: Dell PowerFlex: Unnecessary login happen (roquej) 14:32:12 #link https://review.opendev.org/c/openstack/os-brick/+/876284 14:32:28 so i reiterated our discussion during last cinder meeting to update the commit message 14:33:31 it's updated now so we can take a look at it again 14:34:11 facing some issue with Zuul on os-brick-src-tempest-lvm-lio-barbican, not sure it's related 14:35:52 failure looks unrelated 14:36:05 yeah that was my feeling too 14:36:13 anyway, we can review it again 14:36:18 thanks for the update 14:36:43 finally we have 14:36:45 3. [HPE 3par] Fix issue seen during retype/migrate 14:36:52 #link https://review.opendev.org/c/openstack/cinder/+/887559 14:38:34 i signed up to review this 14:39:02 ok 14:39:09 but to be honest, I'm not sure i understand all the changes 14:39:19 the patch is doing a lot of things 14:39:52 but yeah i will remind myself to look at it later today 14:40:12 thanks 14:40:19 but will still need another reviewer 14:41:00 that's all the patches we have for RC2 14:41:39 in the meantime i see we have patches up for the Dell backports 14:41:45 so please take a look at them as well 14:41:47 I have cherry-picked all the patches (the 3 above) for backporting them to 2023.2 14:42:01 thanks happystacker 14:42:09 I wasn't aware that we had to, I'm still learning sorry for that 14:43:10 no problem 14:43:18 that was all for this topic 14:43:28 let's move to the next one (which looks important) 14:43:34 #topic gates 14:43:39 rosmaita, i guess that's you? 14:43:45 hello 14:43:55 yes, i think we have some gate breakage since yesterday 14:44:11 cinder-tempest-plugin-lvm-lio-barbican and related jobs 14:44:24 i think i have a fix posted, but won't be sure for a while 14:44:30 https://review.opendev.org/c/openstack/cinderlib/+/896658 14:44:46 another cycle where cinder breaks cinderlib :D 14:45:00 and cinderlib returns the favor by breaking our gates! 14:45:24 :( 14:45:52 so IIUC, this will unblock the gates but doesn't solve the actual problem, i.e. change cinderlib code to be compatible with sqlalchemy 2.0 14:46:07 anyway, i will watch the patch and hopefully it works and then i will ping people in the cinder channel for reviews/approval 14:46:29 whoami-rajat: correct, that will have to be done in cinderlib over the next few months 14:47:11 ok great, thanks for the fix, let me know once zuul votes on it 14:47:29 will do 14:47:45 may have to also watch this patch, that actually runs that test: 14:47:55 https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/896594 14:48:54 i guess that's all from me 14:49:31 that patch kills to birds with one stone 14:49:54 great, thanks for the update! 14:50:42 that's all the topics we had for today 14:50:46 let's move to open discussion 14:50:49 #topic open discussion 14:54:02 jbernard, is having issues with his laptop so he might start the bug squad meeting a little late 14:54:15 just letting people know if they wonder why the bug squad meeting didn't start on time 14:56:28 ack 14:59:52 we're out of time 14:59:56 thanks everyone for joining 15:00:02 do remember to add topics for the PTG! 15:00:06 #endmeeting