| opendevreview | Yosi Ben Shimon proposed openstack/cinder-tempest-plugin master: Test srbac on backups https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/878587 | 05:53 |
|---|---|---|
| opendevreview | Yosi Ben Shimon proposed openstack/cinder-tempest-plugin master: Test srbac on user_messages https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/876730 | 06:07 |
| opendevreview | Yosi Ben Shimon proposed openstack/cinder-tempest-plugin master: Test srbac on user_messages https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/876730 | 06:10 |
| opendevreview | Yosi Ben Shimon proposed openstack/cinder-tempest-plugin master: test srbac on volume-types https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/880566 | 06:13 |
| opendevreview | Yosi Ben Shimon proposed openstack/cinder-tempest-plugin master: Test srbac on backups https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/878587 | 09:18 |
| opendevreview | Yosi Ben Shimon proposed openstack/cinder-tempest-plugin master: Test srbac on backups https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/882149 | 11:30 |
| opendevreview | Yosi Ben Shimon proposed openstack/cinder-tempest-plugin master: Test srbac on backups https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/878587 | 11:47 |
| opendevreview | Yosi Ben Shimon proposed openstack/cinder-tempest-plugin master: Test srbac on backups https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/878587 | 13:08 |
| whoami-rajat | Cinder meeting in #openstack-meeting-alt at 1400 UTC | 13:59 |
| whoami-rajat | jungleboyj rosmaita smcginnis tosky whoami-rajat m5z e0ne geguileo eharney walshh_ jbernard sfernand enriquetaso hemna fabiooliveira yuval tobias-urdin adiare happystacker dosaboy | 13:59 |
| rosmaita | whoami-rajat: ty | 14:00 |
| whoami-rajat | np | 14:00 |
| *** dviroel_ is now known as dviroel | 14:28 | |
| enriquetaso | #startmeeting cinder_bs | 15:00 |
| opendevmeet | Meeting started Wed May 3 15:00:50 2023 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 | Hello, Cinder Bug Meeting starting | 15:01 |
| enriquetaso | Full report of bugs: | 15:01 |
| enriquetaso | #link https://lists.openstack.org/pipermail/openstack-discuss/2023-May/033545.html | 15:01 |
| enriquetaso | Only 3 bugs for today's meeting | 15:01 |
| enriquetaso | let's start | 15:01 |
| rosmaita | o/ | 15:01 |
| enriquetaso | hi Brian | 15:01 |
| enriquetaso | #topic Infinidat driver should use the pool's compression setting when | 15:01 |
| enriquetaso | creating volumes. | 15:01 |
| enriquetaso | #link https://bugs.launchpad.net/cinder/+bug/2017815 | 15:02 |
| enriquetaso | Infinidat driver should use pool's compression setting as default when creating volumes | 15:02 |
| enriquetaso | Fix proposed to master: | 15:02 |
| enriquetaso | #link https://review.opendev.org/c/openstack/cinder/+/881188 | 15:02 |
| enriquetaso | okay, moving on | 15:08 |
| enriquetaso | #topic Cinder retype fails with "no host supplied" for SolidFire driver | 15:08 |
| enriquetaso | #link https://bugs.launchpad.net/cinder/+bug/2018001 | 15:08 |
| enriquetaso | When I perform a cinder retype from a Solidfire volume type to a ONTAP volume type, I see | 15:09 |
| enriquetaso | a "No host supplied" error. For Yoga, Zed | 15:09 |
| enriquetaso | no fix proposed to master yet | 15:09 |
| enriquetaso | Okay, last bug: | 15:13 |
| enriquetaso | #topic When cinder-backup and nova-compute at the same node,backup restore will delete multipath. | 15:13 |
| enriquetaso | #linkhttps://bugs.launchpad.net/cinder/+bug/2017952 | 15:13 |
| enriquetaso | #link https://bugs.launchpad.net/cinder/+bug/2017952 | 15:13 |
| enriquetaso | I'm waiting for reporter update ^ | 15:14 |
| enriquetaso | #topic open discussion | 15:14 |
| enriquetaso | Feel free to proposed any bugs now | 15:14 |
| opendevreview | Dan Smith proposed openstack/devstack-plugin-ceph master: Revert "Temporary pin the ceph jobs nodeset to Focal" https://review.opendev.org/c/openstack/devstack-plugin-ceph/+/865315 | 15:17 |
| enriquetaso | okay | 15:18 |
| enriquetaso | looks like that all we have to today's meeting | 15:20 |
| enriquetaso | thanks | 15:20 |
| enriquetaso | #endmeeting | 15:20 |
| opendevmeet | Meeting ended Wed May 3 15:20:40 2023 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 15:20 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/cinder_bs/2023/cinder_bs.2023-05-03-15.00.html | 15:20 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/cinder_bs/2023/cinder_bs.2023-05-03-15.00.txt | 15:20 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/cinder_bs/2023/cinder_bs.2023-05-03-15.00.log.html | 15:20 |
| opendevreview | Merged openstack/python-cinderclient stable/xena: [stable-only] Pin tox <4 https://review.opendev.org/c/openstack/python-cinderclient/+/869265 | 15:51 |
| opendevreview | Merged openstack/cinder master: Pylint: Cleanup ds8k driver workaround https://review.opendev.org/c/openstack/cinder/+/879964 | 15:55 |
| opendevreview | Simon Dodsley proposed openstack/cinder master: [Pure Storage] Add check for new error message https://review.opendev.org/c/openstack/cinder/+/881782 | 16:00 |
| opendevreview | Merged openstack/python-cinderclient stable/yoga: [stable-only] Pin tox <4 https://review.opendev.org/c/openstack/python-cinderclient/+/869264 | 16:20 |
| opendevreview | Yosi Ben Shimon proposed openstack/cinder-tempest-plugin master: Test srbac on backups https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/878587 | 16:25 |
| opendevreview | Merged openstack/cinder stable/wallaby: [stable-only] Pin tox <4 https://review.opendev.org/c/openstack/cinder/+/868446 | 16:42 |
| opendevreview | Merged openstack/cinder master: Bump mypy to 1.2.0 https://review.opendev.org/c/openstack/cinder/+/879961 | 16:43 |
| opendevreview | Merged openstack/cinder master: Run py311 unit tests (non-voting) https://review.opendev.org/c/openstack/cinder/+/869258 | 16:51 |
| opendevreview | Merged openstack/cinder master: [Pure Storage] Add TCP transport type to NVMe driver https://review.opendev.org/c/openstack/cinder/+/877918 | 16:51 |
| ganso | rosmaita: hey I have unexpected good news lol | 16:51 |
| rosmaita | i could use some unexpected good news! | 16:52 |
| ganso | rosmaita: previously I had tested the glance_core_properties config option workaround on the failed fix, and confirmed it worked. We discussed this a few days ago. This time I decided to test the glance_core_properties config on the original, non-patched code, and IT ALSO WORKS LOL | 16:53 |
| rosmaita | well, it's kind of a misuse of that config option, so it's worth having the patch (plus the fix), i think | 16:54 |
| rosmaita | but it's nice for your customer that they can have a workaround without making a code change | 16:54 |
| ganso | rosmaita: so basically, the fix was apparently never needed in the first place. I am re-reading the code with this newfound evidence, and apparently this workaround always avoided the issue. The os_glance properties don't get added to {"properties":{}" sub-dict, instead they stay at the top level, but glance also does not incorporate them when uploading the volume back | 16:55 |
| ganso | rosmaita: yes the fix is still nice to have to things fit properly where they should be, but I suppose if this was known back then maybe it wouldn't been implemented | 16:56 |
| rosmaita | well, the fix takes a prefix whereas the config opt only works when you specify each property by full name ... is that right? | 16:57 |
| ganso | rosmaita: correct. The config opt only works in a predictable scenario | 17:01 |
| rosmaita | ganso: ok, so the filter is still worth having, since glance has reserved the entire os_glance namespace | 17:03 |
| ganso | rosmaita: yes | 17:03 |
| ganso | rosmaita: I will work on the release note update | 17:03 |
| rosmaita | ok, cool | 17:03 |
| opendevreview | Rodrigo Barbieri proposed openstack/cinder master: Fix glance metadata properties filtering https://review.opendev.org/c/openstack/cinder/+/882087 | 17:48 |
| opendevreview | Dan Smith proposed openstack/cinder-tempest-plugin master: Wait for servers to be SSHABLE https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/881764 | 18:17 |
| dansmith | rosmaita: it seems like at some point cinder had a volume_clear or wipe_method configuration to zero volumes before handing them to the guest | 18:56 |
| dansmith | I'm trying to figure out if that's enabled in our CI runs and if not, if we could, but I'm not finding a current reference | 18:56 |
| opendevreview | Dan Smith proposed openstack/devstack-plugin-ceph master: Revert "Temporary pin the ceph jobs nodeset to Focal" https://review.opendev.org/c/openstack/devstack-plugin-ceph/+/865315 | 18:57 |
| rosmaita | dansmith: i'm not aware of such a method, it's always been our position that the backend is responsible for providing a clean volume upon creation | 18:57 |
| dansmith | orly | 18:57 |
| dansmith | rosmaita: so, one of the volume failures I've seen happens when we attach a volume to an instance and the guest kernel crashes | 18:58 |
| rosmaita | of course, "always" only means since rocky for me | 18:58 |
| dansmith | in at least one case, it's in the middle of looking at the volume, finding what it thinks is a bad GPT and then it runs off the end of a string or something | 18:58 |
| dansmith | so I'm guessing the volume is full of garbage, which should be fine of course, | 18:59 |
| dansmith | but the fix for that is much more complicated than just enabling some sort of zeroing | 18:59 |
| eharney | dansmith: zeroing was just for thick-provisioned lvm (no longer the default) because we had to do it there to get clean volumes. normally the backend ensures that new volumes are clear | 18:59 |
| dansmith | eharney: ack and so reads from a thin lvm look like zeroed pages or something? | 19:00 |
| eharney | dansmith: yes | 19:00 |
| dansmith | and that's what we should be using in the jobs yeah? what about for ceph? | 19:00 |
| dansmith | cinder backed with ceph I mean | 19:00 |
| eharney | it's not needed for ceph, afaik, thick LVM was the only backend that ever actually had that behavior | 19:01 |
| dansmith | what I mean is, what do reads from empty/fresh ceph volumes look like? also zero? | 19:01 |
| eharney | yes | 19:01 |
| dansmith | okay, that kernel crash I pointed you at earlier this week: https://kashyapc.fedorapeople.org/CirrOS_segfault/guest-boot-console-and-segfault.txt | 19:01 |
| dansmith | the virt/kernel people think that's in the middle of examining the GPT of the new volume | 19:02 |
| dansmith | and their solution (or workaround) is to make sure we don't have volumes with broken GPTs, which of course makes no sense since they should be "blank" | 19:02 |
| eharney | humm right | 19:03 |
| dansmith | so I'm just trying to reconcile that with what we see | 19:03 |
| dansmith | and if we weren't actually zeroing, then doing so might just avoid us hitting this, kernel bug aside | 19:03 |
| opendevreview | Dan Smith proposed openstack/devstack-plugin-ceph master: Revert "Temporary pin the ceph jobs nodeset to Focal" https://review.opendev.org/c/openstack/devstack-plugin-ceph/+/865315 | 19:07 |
| opendevreview | Ashley Rodriguez proposed openstack/devstack-plugin-ceph master: [WIP][DNM] Remote Ceph with cephadm https://review.opendev.org/c/openstack/devstack-plugin-ceph/+/876747 | 19:11 |
| opendevreview | Ashley Rodriguez proposed openstack/devstack-plugin-ceph master: [WIP][DNM] Remote Ceph with cephadm https://review.opendev.org/c/openstack/devstack-plugin-ceph/+/876747 | 19:16 |
| opendevreview | Ashley Rodriguez proposed openstack/devstack-plugin-ceph master: [WIP][DNM] Remote Ceph with cephadm https://review.opendev.org/c/openstack/devstack-plugin-ceph/+/876747 | 19:36 |
| opendevreview | Ashley Rodriguez proposed openstack/devstack-plugin-ceph master: [WIP][DNM] Remote Ceph with cephadm https://review.opendev.org/c/openstack/devstack-plugin-ceph/+/876747 | 19:48 |
| opendevreview | Ashley Rodriguez proposed openstack/devstack-plugin-ceph master: [WIP][DNM] Remote Ceph with cephadm https://review.opendev.org/c/openstack/devstack-plugin-ceph/+/876747 | 19:59 |
| opendevreview | Simon Dodsley proposed openstack/cinder master: [Pure Storage] Add check for new error message https://review.opendev.org/c/openstack/cinder/+/881782 | 20:50 |
| opendevreview | Ashley Rodriguez proposed openstack/devstack-plugin-ceph master: [WIP][DNM] Remote Ceph with cephadm https://review.opendev.org/c/openstack/devstack-plugin-ceph/+/876747 | 21:03 |
| opendevreview | Rodrigo Barbieri proposed openstack/cinder master: Fix glance metadata properties filtering https://review.opendev.org/c/openstack/cinder/+/882087 | 21:31 |
| *** zaitcev_ is now known as zaitcev | 22:50 | |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!