*** elodilles_ooo is now known as elodilles | 06:44 | |
opendevreview | Matus Brandys proposed openstack/cinder master: HPE 3par: getWsApiVersion login/logout https://review.opendev.org/c/openstack/cinder/+/926583 | 07:21 |
---|---|---|
opendevreview | Atsushi Kawai proposed openstack/cinder master: Hitachi: Support QoS https://review.opendev.org/c/openstack/cinder/+/912042 | 08:14 |
opendevreview | Rajat Dhasmana proposed openstack/cinder master: Add testing for optimized volume upload https://review.opendev.org/c/openstack/cinder/+/909847 | 08:31 |
*** geguileo is now known as Guest1057 | 08:52 | |
opendevreview | Markus Hentsch proposed openstack/os-brick master: Add get_passphrase_from_secret utility function https://review.opendev.org/c/openstack/os-brick/+/926293 | 09:16 |
opendevreview | Rajat Dhasmana proposed openstack/cinder master: Add cinder-manage command to update service_uuid https://review.opendev.org/c/openstack/cinder/+/823089 | 11:24 |
opendevreview | Gorka Eguileor proposed openstack/cinder stable/2023.2: Hitachi: Fix to use correct pool on secondary storage https://review.opendev.org/c/openstack/cinder/+/926728 | 11:36 |
opendevreview | Gorka Eguileor proposed openstack/cinder stable/2023.2: HPE XP and NEC V: Host group name is not correct https://review.opendev.org/c/openstack/cinder/+/926729 | 11:36 |
opendevreview | Gorka Eguileor proposed openstack/cinder stable/2023.2: Hitachi: Fix exception when deleted volume is busy https://review.opendev.org/c/openstack/cinder/+/926730 | 11:36 |
opendevreview | Gorka Eguileor proposed openstack/cinder stable/2023.2: Tests: Save 30s on hbsd FC tests https://review.opendev.org/c/openstack/cinder/+/926731 | 11:36 |
opendevreview | Gorka Eguileor proposed openstack/cinder stable/2023.2: Hitachi: Stop frequently REST API request in test https://review.opendev.org/c/openstack/cinder/+/926732 | 11:36 |
opendevreview | Gorka Eguileor proposed openstack/cinder stable/2023.2: Hitachi: Fix to set correct object ID for LDEV nickname https://review.opendev.org/c/openstack/cinder/+/926733 | 11:36 |
opendevreview | Gorka Eguileor proposed openstack/cinder stable/2023.2: Hitachi: Prevent to delete a LDEV assigned to multi objects https://review.opendev.org/c/openstack/cinder/+/926734 | 11:36 |
opendevreview | Gorka Eguileor proposed openstack/cinder stable/2023.1: Hitachi: Fix to use correct pool on secondary storage https://review.opendev.org/c/openstack/cinder/+/926735 | 11:38 |
opendevreview | Gorka Eguileor proposed openstack/cinder stable/2023.1: HPE XP and NEC V: Host group name is not correct https://review.opendev.org/c/openstack/cinder/+/926736 | 11:38 |
opendevreview | Gorka Eguileor proposed openstack/cinder stable/2023.1: Hitachi: Fix exception when deleted volume is busy https://review.opendev.org/c/openstack/cinder/+/926737 | 11:38 |
opendevreview | Gorka Eguileor proposed openstack/cinder stable/2023.1: Tests: Save 30s on hbsd FC tests https://review.opendev.org/c/openstack/cinder/+/926738 | 11:38 |
opendevreview | Gorka Eguileor proposed openstack/cinder stable/2023.1: Tests: Make NEC tests faster https://review.opendev.org/c/openstack/cinder/+/926739 | 11:38 |
opendevreview | Gorka Eguileor proposed openstack/cinder stable/2023.1: Hitachi: Stop frequently REST API request in test https://review.opendev.org/c/openstack/cinder/+/926740 | 11:39 |
opendevreview | Gorka Eguileor proposed openstack/cinder stable/2023.1: Hitachi: Fix to set correct object ID for LDEV nickname https://review.opendev.org/c/openstack/cinder/+/926741 | 11:39 |
opendevreview | Gorka Eguileor proposed openstack/cinder stable/2023.1: Hitachi: Prevent to delete a LDEV assigned to multi objects https://review.opendev.org/c/openstack/cinder/+/926742 | 11:39 |
*** weechat1 is now known as hemna_ | 12:52 | |
opendevreview | Tobias Urdin proposed openstack/cinder master: Add API document for availability zone list https://review.opendev.org/c/openstack/cinder/+/585675 | 13:48 |
opendevreview | Tobias Urdin proposed openstack/cinder master: Add support for backup service to os-availability-zone https://review.opendev.org/c/openstack/cinder/+/926263 | 13:48 |
*** jbernard_ is now known as jbernard | 14:02 | |
jbernard | jungleboyj rosmaita smcginnis tosky whoami-rajat m5z e0ne geguileo eharney jbernard hemna fabiooliveira yuval tobias-urdin adiare happystacker dosaboy hillpd msaravan sp-bmilanov Luzi zaitcev: meeting time! see ya'll in #openstack-meeting-alt | 14:03 |
jbernard | in case you missed the meeting, the focus this week is on reviews, a list is being maintained here: https://etherpad.opendev.org/p/cinder-dalmatian-reviews any help is much appreciated | 14:48 |
eharney | whoami-rajat: basically the style checker decided that the same rules that apply to code should apply to code inside of f-strings -- that seems fairly debatable to me, and makes me wonder how much of this check is even done on purpose here | 15:03 |
yuval | jbernard: what is the deadline for the cinder patches to be merge? | 15:03 |
whoami-rajat | eharney, yeah i also had questions about E226 rule in the f-string context for readability, I'm thinking now does print('a+b=%s' % c) also raise similar error ... | 15:06 |
eharney | yuval: https://bugs.launchpad.net/os-brick/+bug/1884059 and the related commit message describes the reason for DAC_READ_SEARCH -- i wonder if modifying the file permissions of the python libs would work around it | 15:11 |
eharney | also worth noting that the cinder privsep code does always grant that capability | 15:12 |
opendevreview | Bryan Neumann proposed openstack/cinder master: Dell PowerMax: RDF consistency exempt https://review.opendev.org/c/openstack/cinder/+/915935 | 15:22 |
yuval | eharney: also nova give this permission without questions asked | 15:27 |
yuval | https://opendev.org/openstack/nova/src/branch/master/nova/privsep/__init__.py | 15:27 |
yuval | so - os-brick is used by these 2 services - which already gives this permission. isn't that safe enough to add it to os-brick? | 15:29 |
yuval | how do I find gorka email? | 15:29 |
opendevreview | Nilesh Thathagar proposed openstack/os-brick master: Dell PowerFlex: Added retry after disconnect volume https://review.opendev.org/c/openstack/os-brick/+/916062 | 16:00 |
opendevreview | Tobias Urdin proposed openstack/cinder master: Add support for backup service to os-availability-zone https://review.opendev.org/c/openstack/cinder/+/926263 | 17:17 |
opendevreview | Merged openstack/os-brick master: Support hacking 6.1.0 https://review.opendev.org/c/openstack/os-brick/+/916737 | 17:22 |
opendevreview | Nilesh Thathagar proposed openstack/os-brick master: Dell PowerFlex: Added retry after disconnect volume https://review.opendev.org/c/openstack/os-brick/+/916062 | 17:54 |
opendevreview | Nilesh Thathagar proposed openstack/os-brick master: Dell PowerFlex: Added retry after disconnect volume https://review.opendev.org/c/openstack/os-brick/+/916062 | 18:02 |
opendevreview | Nilesh Thathagar proposed openstack/os-brick master: Dell PowerFlex: Added retry after disconnect volume https://review.opendev.org/c/openstack/os-brick/+/916062 | 18:04 |
opendevreview | Nilesh Thathagar proposed openstack/os-brick master: Dell PowerFlex: Added retry after disconnect volume https://review.opendev.org/c/openstack/os-brick/+/916062 | 18:16 |
opendevreview | Nilesh Thathagar proposed openstack/os-brick master: Dell PowerFlex: Added retry after disconnect volume https://review.opendev.org/c/openstack/os-brick/+/916062 | 18:44 |
opendevreview | Harsh Ailani proposed openstack/cinder master: [IBM SVf driver] Added support for volumegroup-snapshot for temporary volumegroup https://review.opendev.org/c/openstack/cinder/+/926844 | 18:54 |
opendevreview | Bryan Neumann proposed openstack/cinder master: Dell PowerMax: RDF consistency exempt https://review.opendev.org/c/openstack/cinder/+/915935 | 18:56 |
opendevreview | Merged openstack/os-brick stable/2023.1: Dell PowerFlex: Unnecessary login happen https://review.opendev.org/c/openstack/os-brick/+/926406 | 20:25 |
simondodsley | eharney, jbernard - could you have a look at https://review.opendev.org/c/openstack/cinder/+/926475. Already has one +2. Our CI will never pass any other patches until this one is merged. | 21:12 |
jbernard | simondodsley: done | 21:16 |
simondodsley | thank you jbernard | 21:17 |
opendevreview | Merged openstack/cinder master: [Pure Storage] Fix CG, FC WWN and NVME-TCP NSID bugs https://review.opendev.org/c/openstack/cinder/+/926475 | 22:51 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!