| opendevreview | prashkre proposed openstack/cinder master: Fix locking issue with powermax cinder driver. https://review.opendev.org/c/openstack/cinder/+/884957 | 00:09 |
|---|---|---|
| opendevreview | Keigo Noha proposed openstack/cinder master: [WIP] Consolidate schema validation between volumes and snapshots. https://review.opendev.org/c/openstack/cinder/+/884958 | 00:17 |
| opendevreview | prashkre proposed openstack/cinder master: Fix locking issue with powermax cinder driver. https://review.opendev.org/c/openstack/cinder/+/884957 | 00:25 |
| *** amoralej|off is now known as amoralej | 06:12 | |
| *** zaitcev_ is now known as zaitcev | 06:49 | |
| enriquetaso | Hi whoami-rajat, one question regarding https://bugs.launchpad.net/cinder/+bug/2017575, in yesterday bug meeting hillpd asked this question: | 08:20 |
| enriquetaso | <hillpd> Passing the cinder.conf "host" value to os-brick makes sense going forward. Unfortunately, that would not be safe to backport due to how the host value gets used in existing deployments. | 08:20 |
| enriquetaso | <hillpd> To address this in prior releases, we are considering adding a "[cinder-oceanstor]" config option, "use_host_fqdn" to address this in the huawei driver. Is that a reasonable direction? | 08:20 |
| opendevreview | Merged openstack/cinder master: Revert "Add tempest integrated storage job to run on ubuntu focal" https://review.opendev.org/c/openstack/cinder/+/884671 | 08:29 |
| raghavendrat | hi, regarding gate fixes ... now https://review.opendev.org/c/openstack/cinder/+/884671 is merged | 08:45 |
| raghavendrat | could someone please provide +2 and +W on https://review.opendev.org/c/openstack/cinder/+/884896 | 08:45 |
| whoami-rajat | hillpd, hey, I'm not sure how that would work out, the code that does the connection via os-brick is not part of the driver, how will huawei driver use that config option? you mean not taking connector['host'] value but use the config option instead? | 09:44 |
| raghavendrat | hi whoami-rajat: are you online ? | 10:01 |
| whoami-rajat | raghavendrat, hey | 10:03 |
| raghavendrat | whoami-rajat: to resolve gate failures, eharney submitted patch 884896 .. could you please provide +W | 10:05 |
| raghavendrat | https://review.opendev.org/c/openstack/cinder/+/884896 | 10:05 |
| whoami-rajat | raghavendrat, ack looking | 10:20 |
| whoami-rajat | it's in merge conflict | 10:21 |
| raghavendrat | ok | 10:23 |
| whoami-rajat | we can abandon that patch, this will fix the gate failure https://review.opendev.org/c/openstack/cinder/+/884194 | 10:24 |
| raghavendrat | ok | 10:30 |
| raghavendrat | thanks Rajat for looking into this. | 10:35 |
| whoami-rajat | no problem, thanks for the reminder | 10:37 |
| opendevreview | Keigo Noha proposed openstack/cinder master: [WIP] Consolidate schema validation between volumes and snapshots. https://review.opendev.org/c/openstack/cinder/+/884958 | 11:17 |
| *** amoralej is now known as amoralej|lunch | 12:09 | |
| *** amoralej|lunch is now known as amoralej | 12:59 | |
| opendevreview | Sofia Enriquez proposed openstack/cinder master: Automate generation of attachments api-ref samples https://review.opendev.org/c/openstack/cinder/+/884420 | 14:20 |
| opendevreview | Sofia Enriquez proposed openstack/cinder master: Automate generation of attachments api-ref samples https://review.opendev.org/c/openstack/cinder/+/884420 | 14:22 |
| opendevreview | Sofia Enriquez proposed openstack/cinder master: Automate generation of attachments api-ref samples https://review.opendev.org/c/openstack/cinder/+/884420 | 14:25 |
| whoami-rajat | jungleboyj, hey, here is the recording for yesterday's meeting https://drive.google.com/file/d/1-2HXBeIUZLMfmBrmw4aa_QQFoVS4VIMV/view | 14:30 |
| opendevreview | Eric Harney proposed openstack/cinder stable/2023.1: Make paramiko import optional https://review.opendev.org/c/openstack/cinder/+/880604 | 14:42 |
| hillpd | whoami-rajat, howdy, yeah the idea is to intercept and modify the connector['host'] value that gets used by the huawei driver. | 15:02 |
| enriquetaso | hillpd, please write your answer on https://bugs.launchpad.net/cinder/+bug/2017575 (I left a comment today: comment #6) | 16:12 |
| hillpd | enriquetaso, thanks, I will take a look | 16:14 |
| *** amoralej is now known as amoralej|off | 16:23 | |
| *** JayF is now known as Guest1962 | 21:37 | |
| *** JasonF is now known as JayF | 21:37 | |
| opendevreview | Merged openstack/cinder master: zuul: Extend timeout for cinder-plugin-ceph-tempest https://review.opendev.org/c/openstack/cinder/+/884194 | 23:04 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!