| opendevreview | Brian Rosmaita proposed openstack/cinder master: Don't destroy existing backup by mistake on import https://review.opendev.org/c/openstack/cinder/+/839451 | 00:00 |
|---|---|---|
| opendevreview | Brian Rosmaita proposed openstack/cinder master: Don't destroy existing backup by mistake on import https://review.opendev.org/c/openstack/cinder/+/839451 | 03:08 |
| opendevreview | Brian Rosmaita proposed openstack/cinder master: Don't destroy existing backup by mistake on import https://review.opendev.org/c/openstack/cinder/+/839451 | 03:19 |
| opendevreview | Arun KV proposed openstack/cinder master: Reintroduce DataCore driver https://review.opendev.org/c/openstack/cinder/+/836996 | 04:59 |
| opendevreview | 王昊 proposed openstack/cinder master: Move centos stream testing to centos-9-stream https://review.opendev.org/c/openstack/cinder/+/839469 | 06:14 |
| opendevreview | Lior Friedman proposed openstack/os-brick master: Add NVMeoF Multipathing support Phase 1 (Native) implementation of NVMeoF Multipathing. https://review.opendev.org/c/openstack/os-brick/+/830800 | 06:27 |
| opendevreview | Lior Friedman proposed openstack/os-brick master: Add NVMeoF Multipathing support Phase 1 (Native) implementation of NVMeoF Multipathing. https://review.opendev.org/c/openstack/os-brick/+/830800 | 06:50 |
| opendevreview | 王昊 proposed openstack/cinder master: Move centos stream testing to centos-9-stream https://review.opendev.org/c/openstack/cinder/+/839469 | 06:53 |
| opendevreview | Konstantin Shalygin proposed openstack/cinder master: RBD driver: added support for ceph-mon's msgr2 https://review.opendev.org/c/openstack/cinder/+/839426 | 07:29 |
| opendevreview | Konstantin Shalygin proposed openstack/cinder master: RBD driver: added support for ceph-mon's msgr2 https://review.opendev.org/c/openstack/cinder/+/839426 | 07:52 |
| opendevreview | Konstantin Shalygin proposed openstack/cinder master: RBD driver: added support for ceph-mon's msgr2 https://review.opendev.org/c/openstack/cinder/+/839426 | 08:30 |
| opendevreview | Arun KV proposed openstack/cinder master: Reintroduce DataCore driver https://review.opendev.org/c/openstack/cinder/+/836996 | 08:31 |
| opendevreview | Merged openstack/cinder master: db: Migrate "service" APIs to enginefacade https://review.opendev.org/c/openstack/cinder/+/830089 | 09:53 |
| opendevreview | Merged openstack/cinder master: db: Migrate "cluster" APIs to enginefacade https://review.opendev.org/c/openstack/cinder/+/830091 | 09:56 |
| opendevreview | Merged openstack/cinder master: db: Migrate "quota" APIs to enginefacade https://review.opendev.org/c/openstack/cinder/+/830092 | 09:57 |
| opendevreview | ZhangFan proposed openstack/cinder master: Add REIMAGE_VOLUME https://review.opendev.org/c/openstack/cinder/+/837502 | 09:57 |
| opendevreview | Konstantin Shalygin proposed openstack/cinder master: RBD driver: added support for ceph-mon's msgr2 https://review.opendev.org/c/openstack/cinder/+/839426 | 11:02 |
| *** dviroel|rover|out is now known as dviroel|rover | 11:16 | |
| opendevreview | Merged openstack/cinder master: db: Migrate "quota class" APIs to enginefacade https://review.opendev.org/c/openstack/cinder/+/830093 | 12:07 |
| opendevreview | Lucas Morais de oliveira proposed openstack/cinder master: WIP api-ref: Add docs for clusters https://review.opendev.org/c/openstack/cinder/+/795785 | 12:55 |
| opendevreview | Mounika Sreeram proposed openstack/cinder master: [SVf] Delete/Extend issue in reverse replication https://review.opendev.org/c/openstack/cinder/+/831776 | 13:15 |
| rosmaita | Cinder meeting in #openstack-meeting-alt at 1400 UTC | 14:01 |
| rosmaita | jungleboyj rosmaita smcginnis tosky whoami-rajat m5z e0ne geguileo eharney walshh_ jbernard sfernand enriquetaso hemna fabiooliveira yuval ^^ | 14:02 |
| geguileo | rosmaita: thanks! | 14:02 |
| fabiooliveira | thanks | 14:02 |
| opendevreview | Eric Harney proposed openstack/cinder master: API: fix volume reset-state status checking https://review.opendev.org/c/openstack/cinder/+/839416 | 14:04 |
| opendevreview | Luigi Toscano proposed openstack/cinder-tempest-plugin master: Replace the CentOS Stream 8 lvm job with a CS9 one https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/828743 | 14:12 |
| opendevreview | Luigi Toscano proposed openstack/cinder-tempest-plugin master: DNM test ceph on CentOS Stream 9 https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/729766 | 14:16 |
| opendevreview | Luigi Toscano proposed openstack/cinder-tempest-plugin master: DNM WIP Try to import cinder tests from barbican-tempest-plugin https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/743492 | 14:28 |
| opendevreview | Luigi Toscano proposed openstack/cinder-tempest-plugin master: zuul: explicitly enable c-bak in the lvm-lio job https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/839563 | 14:52 |
| enriquetaso | #startmeeting cinder_bs | 15:07 |
| opendevmeet | Meeting started Wed Apr 27 15:07:46 2022 UTC and is due to finish in 60 minutes. The chair is enriquetaso. Information about MeetBot at http://wiki.debian.org/MeetBot. | 15:07 |
| opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 15:07 |
| opendevmeet | The meeting name has been set to 'cinder_bs' | 15:07 |
| enriquetaso | 12 new bugs reported this period! I'll try to cover all of them today. | 15:07 |
| enriquetaso | Please check full report: | 15:07 |
| enriquetaso | #link http://lists.openstack.org/pipermail/openstack-discuss/2022-April/028332.html | 15:07 |
| enriquetaso | #topic backport of the fix for bug #1947370 make lock_path a requird config option when prvisouls it was optional | 15:08 |
| enriquetaso | #link https://bugs.launchpad.net/os-brick/+bug/1969794 | 15:08 |
| enriquetaso | Please keep an eye on this bug. | 15:08 |
| enriquetaso | as part of a os-brick bugfix made the external lock_path config option required with no default provided | 15:08 |
| enriquetaso | #link https://review.opendev.org/c/openstack/os-brick/+/814139 | 15:08 |
| enriquetaso | However, Please read Sean comment #1 becasue he proposed a solution to this. | 15:09 |
| enriquetaso | #topic Simultaneous volume creation with the same image in multi-attach mode returns error | 15:09 |
| enriquetaso | #link https://bugs.launchpad.net/cinder/+bug/1969373 | 15:09 |
| enriquetaso | #topic NetApp NFS Storage Migration between backends is Failing | 15:11 |
| enriquetaso | #link https://bugs.launchpad.net/cinder/+bug/1969531 | 15:11 |
| enriquetaso | #topic Wrong property to look up remote address | 15:13 |
| enriquetaso | #link https://bugs.launchpad.net/cinder/+bug/1967683 | 15:13 |
| enriquetaso | The webob.Request class does not have the remote_address attribute but | 15:13 |
| enriquetaso | the remote_addr attribute. We need to update the attribute accordingly. | 15:13 |
| enriquetaso | Fix proposed to maste https://review.opendev.org/c/openstack/cinder/+/836247 | 15:13 |
| enriquetaso | #topic Complex config formula evaluation causes RecursionError | 15:13 |
| enriquetaso | #link https://bugs.launchpad.net/cinder/+bug/1966904 | 15:13 |
| geguileo | whoami-rajat: I added a comment on the original bug report https://bugs.launchpad.net/cinder/+bug/1940466/comments/9 | 15:14 |
| whoami-rajat | rosmaita, should we stop sharing etherpad? | 15:14 |
| geguileo | some time ago, explaining why it was still relevant for Cinder | 15:14 |
| geguileo | (to add the doc) | 15:14 |
| whoami-rajat | geguileo, oh ok, totally forgot about the doc part, thanks for mentioning it | 15:14 |
| geguileo | np | 15:15 |
| enriquetaso | Fix proposed to master | 15:15 |
| enriquetaso | #link https://review.opendev.org/c/openstack/cinder/+/835635 | 15:15 |
| enriquetaso | #topic RBD: Unable to delete a volume which has snapshot/volume children | 15:16 |
| enriquetaso | #link https://bugs.launchpad.net/cinder/+bug/1969643 | 15:16 |
| enriquetaso | Tempest test for this | 15:17 |
| enriquetaso | #link https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/823719 | 15:17 |
| whoami-rajat | jfyi, I've also proposed a patch for recursion limit thing but currently failing on CI if anyone wants to take a look -- https://review.opendev.org/c/openstack/cinder/+/835553 | 15:18 |
| enriquetaso | #link https://review.opendev.org/c/openstack/cinder/+/838757 | 15:20 |
| eharney | https://review.opendev.org/c/openstack/cinder/+/835384 | 15:20 |
| enriquetaso | #topic [Documentation] Migration in cinder, lvm-rbd example is not correct | 15:20 |
| enriquetaso | #link https://bugs.launchpad.net/cinder/+bug/1969913 | 15:21 |
| enriquetaso | The bug report says that lvm to rbd migration for live VMs works fine so the documentation is not accurate. | 15:21 |
| enriquetaso | #link https://opendev.org/openstack/cinder/src/commit/3e068b5ce0990a576b55fea952319884c5c0ed6d/doc/source/contributor/migration.rst | 15:21 |
| enriquetaso | Maybe we can have a volunteer to try this out and see if we should update the documentation. | 15:21 |
| enriquetaso | #action: move to incomplete and ask what's the goal of this? | 15:23 |
| enriquetaso | #topic [RBD] Multiple full backups cannot be created from one snapshot | 15:23 |
| enriquetaso | #link https://bugs.launchpad.net/cinder/+bug/1970237 | 15:23 |
| enriquetaso | #topic [rebranding] Dell EMC will be renamed to Dell | 15:26 |
| enriquetaso | #link https://bugs.launchpad.net/cinder/+bug/1970208 | 15:26 |
| whoami-rajat | who would've thought? | 15:26 |
| enriquetaso | Fix proposed to master | 15:26 |
| enriquetaso | #link https://review.opendev.org/c/openstack/cinder/+/839223 | 15:26 |
| enriquetaso | #topic [Enhancement] No way to disable image conversion. | 15:28 |
| whoami-rajat | I assume there will be a patch changing the directory name -- https://github.com/openstack/cinder/tree/master/cinder/volume/drivers/dell_emc | 15:28 |
| *** dviroel|rover is now known as dviroel|rover|lunch | 15:29 | |
| rosmaita | https://bugs.launchpad.net/cinder/+bug/1970114 | 15:30 |
| whoami-rajat | https://bugs.launchpad.net/cinder/+bug/1970114 | 15:30 |
| enriquetaso | #topic [Enhancement] Image conversion with RBD is not efficient | 15:31 |
| enriquetaso | #link https://bugs.launchpad.net/cinder/+bug/1970114 | 15:31 |
| enriquetaso | thanks! | 15:31 |
| enriquetaso | #topic [Pure Storage] Replicated arrays communication failure not handled correctly | 15:31 |
| enriquetaso | #link https://bugs.launchpad.net/cinder/+bug/1969784the loss of one of the arrays - be it primary or secondary, network issue (e.g. no reachability) or storage issue (e.g. pod down) - causes the Cinder driver to stop working in some or even all aspectsThe ugly workaround is to reconfigure cinder-volume and restart it each time there was a problem with storage array, which isinflexible and prevents automatic failure recovery.No fix | 15:32 |
| enriquetaso | proposed to master yet. | 15:32 |
| enriquetaso | #endmeeting | 15:34 |
| opendevmeet | Meeting ended Wed Apr 27 15:34:59 2022 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 15:34 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/cinder_bs/2022/cinder_bs.2022-04-27-15.07.html | 15:34 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/cinder_bs/2022/cinder_bs.2022-04-27-15.07.txt | 15:34 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/cinder_bs/2022/cinder_bs.2022-04-27-15.07.log.html | 15:34 |
| opendevreview | Merged openstack/cinder master: [doc] update driver review checklist https://review.opendev.org/c/openstack/cinder/+/838779 | 15:39 |
| *** dviroel|rover|lunch is now known as dviroel|rover | 16:18 | |
| jungleboyj | whoami-rajat: I have gotten the videos from the PTG updloaded and put links into the etherpad. | 16:28 |
| jungleboyj | Let me know when you have the recording from today and I will upload that as well. | 16:28 |
| opendevreview | Eric Harney proposed openstack/cinder master: Remove contrib/block-box from zuul config https://review.opendev.org/c/openstack/cinder/+/839597 | 16:31 |
| opendevreview | Eric Harney proposed openstack/cinder master: Run pylint tox env on all files by default https://review.opendev.org/c/openstack/cinder/+/839600 | 16:46 |
| twilit_dragon | I made a simple change (839256) that fixes a small error in a documentation example (Bug #1970135). Upon pushing my change to gerrit, the following CI Systems failed: LightbitsCI, DellEMC PowerStoreCI, DellEMC VNX CI, Nexenta CI, Cisco Cinder CI, and DellEMC Unity CI. Is it normal for these CI systems to fail even though I didn't modify any python code in Cinder? | 17:06 |
| whoami-rajat | jungleboyj, hey, thanks for uploading those, here's the link to today's meeting recording https://bluejeans.com/s/uRfIpBjT3FV | 17:21 |
| opendevreview | Luigi Toscano proposed openstack/cinder-tempest-plugin master: DNM test ceph on CentOS Stream 9 https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/729766 | 17:36 |
| opendevreview | Gorka Eguileor proposed openstack/os-brick master: Fix encryption symlink issues https://review.opendev.org/c/openstack/os-brick/+/836391 | 17:55 |
| opendevreview | Merged openstack/cinder stable/yoga: [SVf]:Fix retype failure for replication volume-type https://review.opendev.org/c/openstack/cinder/+/838880 | 18:18 |
| opendevreview | Luigi Toscano proposed openstack/cinder-tempest-plugin master: DNM test ceph on CentOS Stream 9 https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/729766 | 19:06 |
| opendevreview | Brian Rosmaita proposed openstack/cinder master: Address pylint error https://review.opendev.org/c/openstack/cinder/+/839628 | 19:15 |
| opendevreview | Walt proposed openstack/cinder master: WIP Add pools support to LVM driver https://review.opendev.org/c/openstack/cinder/+/839634 | 19:49 |
| hemna | can't seem to fast forward through the blue jeans recorded meeting | 19:54 |
| rosmaita | hemna: there's no fast-forward button, but i think you can just drag the slider | 19:56 |
| hemna | I tried, it kept going back to the current mark | 19:59 |
| rosmaita | that's weird | 20:01 |
| rosmaita | maybe you'll have better luck on youtube when jungleboyj uploads a converted copy | 20:02 |
| hemna | ok | 20:02 |
| jungleboyj | Do we have the link for today's meeting. I will upload it. | 20:02 |
| hemna | yah it's in the history from 13:21 EST | 20:03 |
| hemna | "<whoami-rajat> jungleboyj, hey, thanks for uploading those, here's the link to today's meeting recording https://bluejeans.com/s/uRfIpBjT3FV" | 20:03 |
| jungleboyj | Ah, I was in another meeting when I got that ping. | 20:04 |
| hemna | np | 20:04 |
| hemna | so....I just put up the lvm pools wip patch | 20:04 |
| hemna | and it's failing tons of tests of course (re: WIP) | 20:04 |
| hemna | but the failures are from bogus volume['host'] entries | 20:04 |
| hemna | like 'test_host' being a volume['host'] entry | 20:05 |
| jungleboyj | rosmaita: Are you able to add download capability to the video? Looks like rajat forgot to check that option. | 20:05 |
| rosmaita | looking | 20:05 |
| hemna | shouldn't the tests provide valid formatted fields for the inputs | 20:05 |
| rosmaita | hemna: probably depends on the age of the tests | 20:06 |
| rosmaita | but, yeah, they should | 20:07 |
| hemna | current one I'm looking at is https://github.com/openstack/cinder/blob/master/cinder/tests/unit/volume/test_volume_reimage.py#L51 | 20:07 |
| hemna | self.volume.create_volume(self.context, volume). creates the volume object | 20:08 |
| rosmaita | jungleboyj: can't do it, i don't own the video | 20:08 |
| jungleboyj | Ok. Will need to ping Rajat tomorrow. | 20:08 |
| rosmaita | hemna: that may explain your problem, too, i think the defaults are more permissive for people in the redhat org | 20:08 |
| hemna | https://github.com/openstack/cinder/blob/master/cinder/tests/unit/volume/__init__.py#L66 | 20:09 |
| hemna | I think that's where the bogus host entry comes from? | 20:10 |
| hemna | no idea where that is declared | 20:10 |
| hemna | ah nope, I was wrong. here it is https://github.com/openstack/cinder/blob/master/cinder/tests/unit/utils.py#L58 | 20:12 |
| rosmaita | hemna: i guess your issue is that you want a string like 'host@backend#pool' in there? | 20:16 |
| hemna | yah | 20:18 |
| hemna | the lvm patch needs to find a pool in the volume host | 20:19 |
| *** dviroel|rover is now known as dviroel|rover|biab | 21:12 | |
| *** dviroel|rover|biab is now known as dviroel|rover | 22:31 | |
| *** dviroel|rover is now known as dviroel|rover|out | 22:54 | |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!