| opendevreview | saikrishnagk proposed openstack/cinder master: [SVF] Fixed Detach for multi-attach volumes https://review.opendev.org/c/openstack/cinder/+/809518 | 08:19 |
|---|---|---|
| *** dviroel|out is now known as dviroel | 11:41 | |
| whoami-rajat | Courtesy reminder: Cinder meeting in #openstack-meeting-alt at 1400 UTC | 13:58 |
| whoami-rajat | jungleboyj rosmaita smcginnis tosky whoami-rajat m5z e0ne geguileo eharney walshh_ jbernard sfernand enriquetaso hemna fabiooliveira | 13:58 |
| opendevreview | Rafael Weingartner proposed openstack/cinder master: Filter reserved image properties https://review.opendev.org/c/openstack/cinder/+/812685 | 13:58 |
| whoami-rajat | rosmaita had doctor's appointment today so i will be taking today's cinder meeting | 13:58 |
| enriquetaso | thanks | 14:01 |
| opendevreview | Sophie Huang proposed openstack/os-brick master: multipath/iscsi: iSCSI connections are not reinitiated after reboot https://review.opendev.org/c/openstack/os-brick/+/810331 | 14:02 |
| tosky | whoami-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 supported | 14:32 |
| tosky | so it's not a case where we have a test but we didn't enable it | 14:32 |
| eharney | i'm not sure the generic nfs driver is even in the matrix? | 14:32 |
| eharney | oh, it is, but yeah, we should fix this | 14:33 |
| opendevreview | Sophie Huang proposed openstack/os-brick master: multipath/iscsi: iSCSI connections are not reinitiated after reboot https://review.opendev.org/c/openstack/os-brick/+/810331 | 14:34 |
| opendevreview | Fernando Ferraz proposed openstack/cinder master: [DNM] testing NetApp CI https://review.opendev.org/c/openstack/cinder/+/794021 | 14:36 |
| whoami-rajat | tosky, 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 patch | 14:37 |
| whoami-rajat | I've mentioned the details in the commit msg | 14:37 |
| eharney | right | 14:38 |
| eharney | there's a generic flow so it should work everywhere | 14:38 |
| whoami-rajat | https://review.opendev.org/c/openstack/cinder/+/812429 | 14:38 |
| whoami-rajat | yep | 14:38 |
| tosky | ah, a non-"optimized" workflow? | 14:40 |
| *** dviroel is now known as dviroel|afk | 14:41 | |
| whoami-rajat | tosky, yes | 14:42 |
| enriquetaso | in this case, should we create a new patch to enable the volume_revert tests? | 14:53 |
| tosky | that would be a patch in devstack to enable it for some additional backends | 14:54 |
| eharney | in devstack? | 14:55 |
| tosky | https://review.opendev.org/c/openstack/devstack/+/727142 | 14:57 |
| tosky | either enable by default for some known backends, or set the variable | 14:57 |
| eharney | wouldn't it be done from devstack-plugin-nfs? | 14:57 |
| tosky | yes, it could automatically set that variable | 14:58 |
| tosky | something like https://review.opendev.org/c/openstack/devstack-plugin-ceph/+/726964 | 14:58 |
| enriquetaso | OK, cinder bug meeting | 15:00 |
| enriquetaso | #startmeeting cinder_bs | 15:00 |
| opendevmeet | Meeting 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 |
| opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 15:00 |
| opendevmeet | The meeting name has been set to 'cinder_bs' | 15:00 |
| enriquetaso | Welcome back | 15:00 |
| enriquetaso | Bug report: | 15:00 |
| enriquetaso | #link http://lists.openstack.org/pipermail/openstack-discuss/2021-October/025211.html | 15:00 |
| enriquetaso | Cinder has 5 new bugs reported. | 15:00 |
| enriquetaso | The 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 |
| enriquetaso | The second one it's a Pure Storage bug that already has a patch proposed. | 15:01 |
| enriquetaso | I 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 |
| enriquetaso | None of the next bugs have patches proposed so far: | 15:02 |
| enriquetaso | #topic bug_1 C-bak configure more than one worker issue | 15:02 |
| enriquetaso | #link https://bugs.launchpad.net/cinder/+bug/1945571 | 15:02 |
| enriquetaso | Summary: 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 |
| enriquetaso | Looks like this it's beyond of the backup driver the reported is using. However, I left some comments on the bug report. | 15:02 |
| enriquetaso | OK.. moving on | 15:04 |
| enriquetaso | bug_2 "[stable/wallaby] filter reserved image properties" | 15:05 |
| enriquetaso | #link https://bugs.launchpad.net/cinder/+bug/1945500 | 15:05 |
| enriquetaso | Summary: 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 |
| enriquetaso | whoami-rajat maybe you would like to read this bug report ^ I think you worked on this in the past, right? | 15:06 |
| enriquetaso | For 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 |
| enriquetaso | I should to update the bug report title to something less generic. | 15:06 |
| whoami-rajat | enriquetaso, sure, will take a look | 15:06 |
| eharney | looks like we should review this patch | 15:06 |
| enriquetaso | This issue is known by Nova and was fixed in Nova with: | 15:07 |
| enriquetaso | https://github.com/openstack/nova/commit/dda179d3f901e4f23091f3095f1af58bc26e222e | 15:07 |
| enriquetaso | #link https://review.opendev.org/c/openstack/cinder/+/812685 | 15:08 |
| enriquetaso | patch proposed ^ | 15:08 |
| enriquetaso | OK, last one | 15:10 |
| enriquetaso | #topic bug_3 wrong ddt version in stable/victoria | 15:10 |
| enriquetaso | #link https://bugs.launchpad.net/cinder/+bug/1946167 | 15:10 |
| enriquetaso | Summary: 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#L12 | 15:11 |
| enriquetaso | [2] https://zuul.opendev.org/t/openstack/build/a630bf25689141d8b329bc6119e9b218/log/job-output.txt#731 | 15:11 |
| enriquetaso | #topic open discussion | 15:12 |
| enriquetaso | Do we have any bugs for the open discussion ? | 15:12 |
| opendevreview | Simon Dodsley proposed openstack/cinder master: [WIP] Add Pure Storage NVMe-RoCE driver https://review.opendev.org/c/openstack/cinder/+/799871 | 15:13 |
| enriquetaso | Thanks!! | 15:15 |
| enriquetaso | #endmeeting | 15:15 |
| opendevmeet | Meeting ended Wed Oct 6 15:15:02 2021 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 15:15 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/cinder_bs/2021/cinder_bs.2021-10-06-15.00.html | 15:15 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/cinder_bs/2021/cinder_bs.2021-10-06-15.00.txt | 15:15 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/cinder_bs/2021/cinder_bs.2021-10-06-15.00.log.html | 15:15 |
| opendevreview | Sofia Enriquez proposed openstack/devstack-plugin-nfs master: Enable volume revert to snapshot NFS tests https://review.opendev.org/c/openstack/devstack-plugin-nfs/+/812716 | 15:20 |
| enriquetaso | tosky, eharney and whoami-rajat ^ something like this? | 15:20 |
| opendevreview | Simon Dodsley proposed openstack/cinder master: Pure Storage FlashArray - Version increment for Cinder drivers https://review.opendev.org/c/openstack/cinder/+/812719 | 15:24 |
| tosky | enriquetaso: there is a potential issue though | 15:25 |
| tosky | we couldn't merge anything in devstack-plugin-nfs due to an error in ussuri | 15:26 |
| tosky | I think lyarwood may remember something about that | 15:26 |
| tosky | enriquetaso: that's enabling it for all branches, so it's going to fail on older branches | 15:26 |
| tosky | enriquetaso: devstack-plugin-nfs is branchless, so you may need an if just like the devstack-plugin-ceph change | 15:27 |
| enriquetaso | sure tosky, like the rbd patch you've shared | 15:29 |
| tosky | enriquetaso: at the time devstack-plugin-rbd was branchless | 15:32 |
| opendevreview | Brian Rosmaita proposed openstack/cinder master: Update release cycle tasks https://review.opendev.org/c/openstack/cinder/+/810785 | 15:56 |
| *** geguileo is now known as Guest2003 | 16:02 | |
| simondodsley | jungleboyj, 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 CI | 16:17 |
| simondodsley | Anyone else happy to give a +W for this? | 16:17 |
| jungleboyj | simondodsley: Done. | 16:19 |
| simondodsley | jungleboyj, thanks | 16:19 |
| *** dviroel|afk is now known as dviroel | 17:41 | |
| opendevreview | Rafael Weingartner proposed openstack/cinder master: Filter reserved image properties https://review.opendev.org/c/openstack/cinder/+/812685 | 18:01 |
| opendevreview | Sofia Enriquez proposed openstack/devstack-plugin-nfs master: Enable volume revert to snapshot NFS tests https://review.opendev.org/c/openstack/devstack-plugin-nfs/+/812716 | 19:31 |
| nahimsouza | hi everyone, please we need more reviews to merge this patch https://review.opendev.org/c/openstack/cinder/+/798208/ | 20:20 |
| opendevreview | Felipe Rodrigues proposed openstack/cinder master: [NetApp] Fix storage assisted migration https://review.opendev.org/c/openstack/cinder/+/812751 | 21:50 |
| opendevreview | Felipe Rodrigues proposed openstack/cinder master: NetApp ONTAP: Fix check QoS min support for SVM account https://review.opendev.org/c/openstack/cinder/+/798189 | 21:54 |
| opendevreview | Felipe Rodrigues proposed openstack/cinder master: NetApp ONTAP: Fix storage assisted migration https://review.opendev.org/c/openstack/cinder/+/812751 | 21:56 |
| opendevreview | Felipe Rodrigues proposed openstack/cinder master: NetApp ONTAP: Fix storage assisted migration https://review.opendev.org/c/openstack/cinder/+/812751 | 21:58 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!