opendevreview | Yian Zong proposed openstack/os-brick master: Add NVMe-FC support in NVMeOF connector https://review.opendev.org/c/openstack/os-brick/+/923345 | 02:27 |
---|---|---|
opendevreview | OpenStack Proposal Bot proposed openstack/cinder master: Imported Translations from Zanata https://review.opendev.org/c/openstack/cinder/+/923806 | 03:04 |
opendevreview | Raghavendra Tilay proposed openstack/cinder master: DNM - test patch https://review.opendev.org/c/openstack/cinder/+/923820 | 06:54 |
*** geguileo is now known as Guest1051 | 09:26 | |
yuval | Hey I have 4 Patches that I need review on, if anyone is available I would appreciate | 11:06 |
yuval | https://review.opendev.org/c/openstack/cinder/+/885587 - XS | 11:06 |
yuval | https://review.opendev.org/c/openstack/cinder/+/904531 - S | 11:06 |
yuval | https://review.opendev.org/c/openstack/cinder/+/903573 - M | 11:07 |
yuval | https://review.opendev.org/c/openstack/os-brick/+/903574 - M | 11:07 |
frickler | rosmaita: fungi: elodilles: no response yet from jbernard on the CVE releases, can we override waiting for PTL approval in this case? | 11:43 |
rosmaita | frickler: cinder weekly meeting is 1400utc today; but if you are in a hurry, i think it would be OK to release, we (cinder) would probably not want to wait for anything else beyond the CVE fix, and we can always release again in a week or two if there's anything major to get out | 11:50 |
frickler | I guess waiting two more h should be fine, then | 12:01 |
elodilles | ACK, please discuss this on the meeting and PTL/release-liaison+1 if it's OK to release o:) | 12:09 |
frickler | might also be a good idea to see whether another liaison besides the PTL can be found | 12:26 |
rosmaita | frickler: elodilles: was in a meeting ... will make sure both issues (release + liaison) are on the agenda | 13:03 |
elodilles | thanks in advance rosmaita o/ | 13:17 |
opendevreview | Gorka Eguileor proposed openstack/cinder-tempest-plugin master: Conditionallly skip manage snapshot tests https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/923852 | 13:53 |
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: weekly meeting time | 14:01 |
jbernard | elodilles, frickler: will post a review right after our meeting | 14:12 |
elodilles | jbernard: thanks! i've +2'd now the release patch o/ | 14:28 |
jbernard | elodilles: excellent, thanks! | 14:28 |
rosmaita | elodilles: frickler: i'll be acting as release liaison, do i need to post a patch somewhere, or do you just note that it's me? | 14:32 |
elodilles | rosmaita: it needs a patch in releases repo: https://opendev.org/openstack/releases/src/branch/master/data/release_liaisons.yaml#L8 | 14:35 |
elodilles | as it is automated o:) | 14:35 |
rosmaita | elodilles: ack | 14:38 |
elodilles | mnaser: if you could take a quick look at this patch that would be awesome https://review.opendev.org/c/openstack/cinder/+/923314 | 15:11 |
elodilles | it was clean cherry pick and got Verified+1 from Zuul | 15:12 |
hemna | anyone from Mirantis still around? | 17:16 |
opendevreview | Brian Rosmaita proposed openstack/cinder-tempest-plugin master: Conditionallly skip manage snapshot tests https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/923852 | 20:12 |
opendevreview | Merged openstack/cinder master: Mark Quobyte Driver Unsupported https://review.opendev.org/c/openstack/cinder/+/922888 | 21:49 |
opendevreview | Merged openstack/cinder master: Deprecate GlusterFS backup driver https://review.opendev.org/c/openstack/cinder/+/923163 | 21:54 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!