| opendevreview | Rajat Dhasmana proposed openstack/os-brick master: Fix iSCSI disconnect_volume when flush fails https://review.opendev.org/c/openstack/os-brick/+/878045 | 06:42 |
|---|---|---|
| opendevreview | Sofia Enriquez proposed openstack/cinder master: Zuul: Job For LVM + Ceph With c-bak Enabled https://review.opendev.org/c/openstack/cinder/+/881032 | 10:16 |
| opendevreview | Sofia Enriquez proposed openstack/cinder master: Ceph: Fix restore backups from diff backend https://review.opendev.org/c/openstack/cinder/+/880965 | 13:27 |
| opendevreview | Sofia Enriquez proposed openstack/cinder master: Ceph: Fix restore backups from diff backend https://review.opendev.org/c/openstack/cinder/+/880965 | 13:30 |
| enriquetaso | XS festival of reviews? | 14:02 |
| enriquetaso | https://meetpad.opendev.org/cinder-festival-of-reviews ? | 14:02 |
| whoami-rajat | Festival of XS reviews at : https://meet.google.com/mfu-wnuo-qrc | 14:03 |
| whoami-rajat | jungleboyj rosmaita smcginnis tosky whoami-rajat m5z e0ne geguileo eharney walshh_ jbernard sfernand enriquetaso hemna fabiooliveira happystacker sfv880 | 14:03 |
| Tony_Saad | Hey guys, question about https://bugs.launchpad.net/cinder/+bug/2003179 Dell Powerflex password appear in plain text while attaching/detaching volume. It was brought up a couple weeks ago at the cinder meeting and someone suggested that this patch should fix it https://review.opendev.org/c/openstack/os-brick/+/871835 | 14:28 |
| Tony_Saad | Do you think logging needs to set elsewhere as well? | 14:28 |
| jungleboyj | Hey guys. Sorry, was presenting and have some conflicts. Ping me if there are any reviews I can help get through. | 14:42 |
| whoami-rajat | rosmaita, chrome crashed, rejoining | 14:43 |
| rosmaita | Festival calendar invite is here: http://eavesdrop.openstack.org/#Cinder_Festival_of_XS_Reviews | 14:48 |
| opendevreview | Eric Harney proposed openstack/cinder master: WIP: RBD: Log librbd version at startup https://review.opendev.org/c/openstack/cinder/+/881233 | 15:01 |
| ganso | rosmaita: hi! was wondering if you could kindly point me to the code that does the glance v1/v2 dictionary handling so I can dig further on the fix for the os_glance metadata issue | 15:05 |
| rosmaita | ganso: hello! i will have to go look | 15:09 |
| rosmaita | ganso: there are _translate_to_glance and _translate_from_glance in cinder/image/glance.py | 15:12 |
| rosmaita | but on further thought, they happen too early/late to be part of the problem | 15:13 |
| rosmaita | ganso: i wonder if the problem is in this function: https://opendev.org/openstack/cinder/src/branch/master/cinder/volume/api.py#L1432 | 15:19 |
| ganso | rosmaita: it could be! | 15:28 |
| ganso | rosmaita: that function was modified in the original code, but then when the code was simplified the changes were reverted | 15:29 |
| ganso | s/code/patch | 15:29 |
| rosmaita | i didn't notice that | 15:30 |
| opendevreview | Merged openstack/python-cinderclient master: Remove USE_PYTHON3 setting from .zuul.yaml https://review.opendev.org/c/openstack/python-cinderclient/+/879980 | 16:23 |
| opendevreview | Merged openstack/os-brick master: Set packages in setup.cfg https://review.opendev.org/c/openstack/os-brick/+/876992 | 16:39 |
| opendevreview | Merged openstack/cinder master: Require novaclient >= 18.2.0 https://review.opendev.org/c/openstack/cinder/+/879058 | 17:30 |
| opendevreview | Merged openstack/cinderlib master: Don't use deprecated config opt name in unit test https://review.opendev.org/c/openstack/cinderlib/+/879035 | 17:30 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!