Wednesday, 2021-10-06

opendevreviewsaikrishnagk proposed openstack/cinder master: [SVF] Fixed Detach for multi-attach volumes  https://review.opendev.org/c/openstack/cinder/+/80951808:19
*** dviroel|out is now known as dviroel11:41
whoami-rajatCourtesy reminder: Cinder meeting in #openstack-meeting-alt at 1400 UTC13:58
whoami-rajatjungleboyj rosmaita smcginnis tosky whoami-rajat m5z e0ne geguileo eharney walshh_ jbernard sfernand enriquetaso hemna fabiooliveira13:58
opendevreviewRafael Weingartner proposed openstack/cinder master: Filter reserved image properties  https://review.opendev.org/c/openstack/cinder/+/81268513:58
whoami-rajatrosmaita had doctor's appointment today so i will be taking today's cinder meeting13:58
enriquetasothanks 14:01
opendevreviewSophie Huang proposed openstack/os-brick master: multipath/iscsi: iSCSI connections are not reinitiated after reboot  https://review.opendev.org/c/openstack/os-brick/+/81033114:02
toskywhoami-rajat, eharney: (I was late) for the record, volume_revert tests are not enabled in the gates because the cinder feature matrix says it's not supported14:32
toskyso it's not a case where we have a test but we didn't enable it14:32
eharneyi'm not sure the generic nfs driver is even in the matrix?14:32
eharneyoh, it is, but yeah, we should fix this14:33
opendevreviewSophie Huang proposed openstack/os-brick master: multipath/iscsi: iSCSI connections are not reinitiated after reboot  https://review.opendev.org/c/openstack/os-brick/+/81033114:34
opendevreviewFernando Ferraz proposed openstack/cinder master: [DNM] testing NetApp CI  https://review.opendev.org/c/openstack/cinder/+/79402114:36
whoami-rajattosky, eharney there might be confusion with my explanation, nfs driver doesn't support that feature but the generic flow in which we create a temporary volume from snapshot (when snap is in 'restoring' state) the operation fails, which is what I'm fixing with my patch14:37
whoami-rajatI've mentioned the details in the commit msg14:37
eharneyright14:38
eharneythere's a generic flow so it should work everywhere14:38
whoami-rajathttps://review.opendev.org/c/openstack/cinder/+/81242914:38
whoami-rajatyep14:38
toskyah, a non-"optimized" workflow?14:40
*** dviroel is now known as dviroel|afk14:41
whoami-rajattosky, yes14:42
enriquetasoin this case, should we create a new patch to enable the volume_revert tests?14:53
toskythat would be a patch in devstack to enable it for some additional backends14:54
eharneyin devstack?14:55
toskyhttps://review.opendev.org/c/openstack/devstack/+/72714214:57
toskyeither enable by default for some known backends, or set the variable14:57
eharneywouldn't it be done from devstack-plugin-nfs?14:57
toskyyes, it could automatically set that variable14:58
toskysomething like https://review.opendev.org/c/openstack/devstack-plugin-ceph/+/72696414:58
enriquetasoOK, cinder bug meeting15:00
enriquetaso#startmeeting cinder_bs15:00
opendevmeetMeeting started Wed Oct  6 15:00:19 2021 UTC and is due to finish in 60 minutes.  The chair is enriquetaso. Information about MeetBot at http://wiki.debian.org/MeetBot.15:00
opendevmeetUseful Commands: #action #agreed #help #info #idea #link #topic #startvote.15:00
opendevmeetThe meeting name has been set to 'cinder_bs'15:00
enriquetasoWelcome back15:00
enriquetasoBug report:15:00
enriquetaso#link http://lists.openstack.org/pipermail/openstack-discuss/2021-October/025211.html15:00
enriquetasoCinder has 5 new bugs reported.15:00
enriquetasoThe high importance bug it's 'revert to snapshot doesn't work for nfs driver' that we've discussed on the main meeting so I won't mentioned it here again. 15:00
enriquetasoThe second one it's a Pure Storage bug that already has a patch proposed.15:01
enriquetasoI think we still need more information on the rest of the bugs so I've marked it as incomplete but I could be important so I'm going to mentioned them in case someone is interested in reproduce and work on them.15:01
enriquetasoNone of the next bugs have patches proposed so far:15:02
enriquetaso#topic bug_1  C-bak configure more than one worker issue15:02
enriquetaso#link https://bugs.launchpad.net/cinder/+bug/194557115:02
enriquetasoSummary: C-bak service with multiple workers´╝łe.g. backup_workers=5. When multiple backups are created concurrently, one of the workers exited abnormally and the child process was respawned. Because the process_number was not 1, _cleanup_incomplete_backup_operations function was not executed. As a result, the backup status on this worker could not be changed from creating to error. In the same way, restore has the same problem.15:02
enriquetasoLooks like this it's beyond of the backup driver the reported is using. However, I left some comments on the bug report. 15:02
enriquetasoOK.. moving on15:04
enriquetasobug_2 "[stable/wallaby] filter reserved image properties"15:05
enriquetaso#link https://bugs.launchpad.net/cinder/+bug/194550015:05
enriquetasoSummary: This may also be yoga bug - Cinder (including Wallaby release) is not able to upload a volume that was build from an image back to glance, if multistore (glance) is enabled.15:05
enriquetasowhoami-rajat maybe you would like to read this bug report ^ I think you worked on this in the past, right?  15:06
enriquetasoFor the record, I don't think it's much help but I've tried the to upload using rbd for both glance and cinder and it worked. So this may only happened with multistore.15:06
enriquetasoI should to update the bug report title to something less generic.15:06
whoami-rajatenriquetaso, sure, will take a look15:06
eharneylooks like we should review this patch15:06
enriquetasoThis issue is known by Nova and was fixed in Nova with:15:07
enriquetasohttps://github.com/openstack/nova/commit/dda179d3f901e4f23091f3095f1af58bc26e222e15:07
enriquetaso#link https://review.opendev.org/c/openstack/cinder/+/81268515:08
enriquetasopatch proposed ^15:08
enriquetasoOK, last one15:10
enriquetaso#topic bug_3 wrong ddt version in stable/victoria15:10
enriquetaso#link https://bugs.launchpad.net/cinder/+bug/194616715:10
enriquetasoSummary: ddt.testNameFormat is only available on ddt>1.2.1 and victoria uses 1.2.1.[1]This is interesting because the current CI for stable/victoria passed because I think it's using the master file test-requirements.txt ( ddt==1.4.1 ) instead of the stable/victoria one [2].15:11
enriquetaso[1] https://opendev.org/openstack/cinder/src/branch/stable/victoria/test-requirements.txt#L1215:11
enriquetaso[2] https://zuul.opendev.org/t/openstack/build/a630bf25689141d8b329bc6119e9b218/log/job-output.txt#73115:11
enriquetaso#topic open discussion 15:12
enriquetasoDo we have any bugs for the open discussion ?15:12
opendevreviewSimon Dodsley proposed openstack/cinder master: [WIP] Add Pure Storage NVMe-RoCE driver  https://review.opendev.org/c/openstack/cinder/+/79987115:13
enriquetasoThanks!!15:15
enriquetaso#endmeeting15:15
opendevmeetMeeting ended Wed Oct  6 15:15:02 2021 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)15:15
opendevmeetMinutes:        https://meetings.opendev.org/meetings/cinder_bs/2021/cinder_bs.2021-10-06-15.00.html15:15
opendevmeetMinutes (text): https://meetings.opendev.org/meetings/cinder_bs/2021/cinder_bs.2021-10-06-15.00.txt15:15
opendevmeetLog:            https://meetings.opendev.org/meetings/cinder_bs/2021/cinder_bs.2021-10-06-15.00.log.html15:15
opendevreviewSofia Enriquez proposed openstack/devstack-plugin-nfs master: Enable volume revert to snapshot NFS tests  https://review.opendev.org/c/openstack/devstack-plugin-nfs/+/81271615:20
enriquetasotosky, eharney and whoami-rajat ^ something like this?15:20
opendevreviewSimon Dodsley proposed openstack/cinder master: Pure Storage FlashArray - Version increment for Cinder drivers  https://review.opendev.org/c/openstack/cinder/+/81271915:24
toskyenriquetaso: there is a potential issue though15:25
toskywe couldn't merge anything in devstack-plugin-nfs due to an error in ussuri15:26
toskyI think lyarwood may remember something about that15:26
toskyenriquetaso: that's enabling it for all branches, so it's going to fail on older branches15:26
toskyenriquetaso: devstack-plugin-nfs is branchless, so you may need an if just like the devstack-plugin-ceph change15:27
enriquetasosure tosky, like the rbd patch you've shared 15:29
toskyenriquetaso: at the time devstack-plugin-rbd was branchless15:32
opendevreviewBrian Rosmaita proposed openstack/cinder master: Update release cycle tasks  https://review.opendev.org/c/openstack/cinder/+/81078515:56
*** geguileo is now known as Guest200316:02
simondodsleyjungleboyj, geguileo: can you re-add your +2 for the iSCSI patch https://review.opendev.org/c/openstack/os-brick/+/810331/. The pep8 error has been resolved and it now passes the CI16:17
simondodsleyAnyone else happy to give a +W for this?16:17
jungleboyjsimondodsley:  Done.16:19
simondodsleyjungleboyj, thanks16:19
*** dviroel|afk is now known as dviroel17:41
opendevreviewRafael Weingartner proposed openstack/cinder master: Filter reserved image properties  https://review.opendev.org/c/openstack/cinder/+/81268518:01
opendevreviewSofia Enriquez proposed openstack/devstack-plugin-nfs master: Enable volume revert to snapshot NFS tests  https://review.opendev.org/c/openstack/devstack-plugin-nfs/+/81271619:31
nahimsouzahi everyone,  please we need more reviews to merge this patch https://review.opendev.org/c/openstack/cinder/+/798208/  20:20
opendevreviewFelipe Rodrigues proposed openstack/cinder master: [NetApp] Fix storage assisted migration  https://review.opendev.org/c/openstack/cinder/+/81275121:50
opendevreviewFelipe Rodrigues proposed openstack/cinder master: NetApp ONTAP: Fix check QoS min support for SVM account  https://review.opendev.org/c/openstack/cinder/+/79818921:54
opendevreviewFelipe Rodrigues proposed openstack/cinder master: NetApp ONTAP: Fix storage assisted migration  https://review.opendev.org/c/openstack/cinder/+/81275121:56
opendevreviewFelipe Rodrigues proposed openstack/cinder master: NetApp ONTAP: Fix storage assisted migration  https://review.opendev.org/c/openstack/cinder/+/81275121:58

Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!