amar_ | Hi rosmaita: Could you pls have a look at this, https://review.opendev.org/c/openstack/cinder/+/790910 really need it get merged ASAP, CI is also green. | 00:57 |
---|---|---|
caiqilong | Hi rosmaita: Could you pls have a look at this: https://review.opendev.org/c/openstack/cinder/+/803000, CI is green. | 01:24 |
opendevreview | Brian Rosmaita proposed openstack/cinder master: Support Images API v2.11 https://review.opendev.org/c/openstack/cinder/+/755654 | 01:27 |
opendevreview | Brian Rosmaita proposed openstack/cinder master: Test glance with cinder backend optimization https://review.opendev.org/c/openstack/cinder/+/757936 | 01:31 |
opendevreview | Brian Rosmaita proposed openstack/os-brick master: Fix instance hard reboot error https://review.opendev.org/c/openstack/os-brick/+/780511 | 01:50 |
opendevreview | Zohar Mamedov proposed openstack/os-brick master: NVMeOF Connection Agent https://review.opendev.org/c/openstack/os-brick/+/802691 | 03:12 |
Jamison | ls | 06:33 |
*** redrobot1 is now known as redrobot | 07:59 | |
opendevreview | Balazs Gibizer proposed openstack/cinder master: Make sure that nfs backup path has proper access rights https://review.opendev.org/c/openstack/cinder/+/773876 | 08:39 |
venkatakrishnat | Hi eharney, geguileo, rosmaita,whoami-rajat__: Could you please review https://review.opendev.org/c/openstack/cinder/+/786524 which is to manage Global mirror change volumes on separate storage pools on IBM SVC cinder driver. | 10:17 |
opendevreview | Takashi Kajinami proposed openstack/cinder master: Use the json format output of qemu-img info https://review.opendev.org/c/openstack/cinder/+/799663 | 10:24 |
opendevreview | Stefan Hoffmann proposed openstack/cinder master: Try nas_secure access with cinder user at retype https://review.opendev.org/c/openstack/cinder/+/802882 | 10:31 |
*** dviroel|ruck|out is now known as dviroel|ruck | 11:08 | |
opendevreview | Rajat Dhasmana proposed openstack/cinder master: WIP: Remove leftover nfs backup dirs https://review.opendev.org/c/openstack/cinder/+/804603 | 11:52 |
opendevreview | Eric Harney proposed openstack/cinder stable/wallaby: LVM: Retry lvextend commands on code 139 https://review.opendev.org/c/openstack/cinder/+/805144 | 12:24 |
rosmaita | need reviews for second +2 on these os-brick patches so we can release today: | 13:42 |
rosmaita | https://review.opendev.org/c/openstack/os-brick/+/800014 | 13:42 |
rosmaita | https://review.opendev.org/c/openstack/os-brick/+/801917 | 13:42 |
rosmaita | https://review.opendev.org/c/openstack/os-brick/+/803210 | 13:42 |
rosmaita | also need opinion on whether this one is ready: | 13:42 |
rosmaita | https://review.opendev.org/c/openstack/os-brick/+/802691 | 13:42 |
rosmaita | e0ne eharney geguileo hemna jungleboyj smcginnis whoami-rajat enriquetaso ^^ | 13:42 |
opendevreview | Takashi Kajinami proposed openstack/cinder master: Use the json format output of qemu-img info https://review.opendev.org/c/openstack/cinder/+/799663 | 13:56 |
opendevreview | Takashi Kajinami proposed openstack/cinder master: Use the json format output of qemu-img info https://review.opendev.org/c/openstack/cinder/+/799663 | 14:05 |
caiqilong | rosmaita: Hi, Could you pls have a look at this: https://review.opendev.org/c/openstack/cinder/+/803000, CI is green. | 14:06 |
rosmaita | caiqilong: the new driver merge deadline has passed, so that can't be merged until after the stable/xena branch is cut | 14:07 |
caiqilong | rosmaita: this proposed change is based on a previous driver, and we added some new features. | 14:10 |
rosmaita | so did the FC driver already exist? | 14:11 |
opendevreview | Helen Walsh proposed openstack/cinder master: PowerMax Driver - Fix for GVG snapshot delete https://review.opendev.org/c/openstack/cinder/+/805228 | 14:12 |
caiqilong | rosmaita: the FC driver is new added, the iscsi driver has already exist, I mistakenly think FC driver was a new feature, but I added some features for previous iscsi driver, could I added these iscsi features in xena, then propose the FC driver features after xena? | 14:17 |
rosmaita | caiqilong: yes, that would be a good plan | 14:18 |
rosmaita | sorry about the confusion | 14:18 |
caiqilong | rosmaita: thanks, I will reproposed the files on a new patch. | 14:21 |
rosmaita | sounds good | 14:21 |
venkatakrishnat | Hi eharney, geguileo, jungleboyj, whoami-rajat__: In your free time, Could you please review https://review.opendev.org/c/openstack/cinder/+/786524 which is to manage Global mirror change volumes on separate storage pools on IBM SVC cinder driver. | 14:36 |
opendevreview | Merged openstack/os-brick master: Replace deprecated tenacity.Retrying.call https://review.opendev.org/c/openstack/os-brick/+/801917 | 15:17 |
opendevreview | Merged openstack/os-brick master: Add support for multiple volumes within subsystem to NVMe-OF connector https://review.opendev.org/c/openstack/os-brick/+/800014 | 15:17 |
opendevreview | Merged openstack/os-brick master: rbd windows: log the device number https://review.opendev.org/c/openstack/os-brick/+/803210 | 15:18 |
venkatakrishnat | Hi eharney, whoami-rajat__, Could you review this simple patch https://review.opendev.org/c/openstack/cinder/+/788782 to update missing RCCG properties to metadata on IBM SVC driver. CI passed. | 15:46 |
opendevreview | Zohar Mamedov proposed openstack/os-brick master: NVMeOF Connection Agent https://review.opendev.org/c/openstack/os-brick/+/802691 | 16:48 |
opendevreview | Girish Chilukuri proposed openstack/cinder master: [SVF]:Retype in-use hyperswap volume https://review.opendev.org/c/openstack/cinder/+/782390 | 17:28 |
opendevreview | mitya-eremeev-2 proposed openstack/cinder master: Fix simultaneous volume creation error in multiattach mode. https://review.opendev.org/c/openstack/cinder/+/805072 | 17:37 |
manoj_katari_ | Hi core team, Addressed review comments and CI is green for this change https://review.opendev.org/c/openstack/cinder/+/784092, please review, thanks! | 17:49 |
opendevreview | Brian Rosmaita proposed openstack/cinder master: WIP/DNM: Add fips check jobs https://review.opendev.org/c/openstack/cinder/+/790535 | 17:56 |
opendevreview | Merged openstack/cinder master: Add releasenote for schema validation fix https://review.opendev.org/c/openstack/cinder/+/804611 | 18:13 |
gksk | To the core reviewers , Please review the below patch related to update replication properties for empty values fix | 18:33 |
gksk | https://review.opendev.org/c/openstack/cinder/+/801617 | 18:33 |
amar_ | Hi rosmaita: Could you pls have a look at this, https://review.opendev.org/c/openstack/cinder/+/790910 really need it to get merged ASAP, CI is also green. | 18:39 |
*** dviroel|ruck is now known as dviroel|ruck|out | 21:08 | |
opendevreview | Alan Bishop proposed openstack/cinder master: Implement project personas for attachments https://review.opendev.org/c/openstack/cinder/+/759943 | 23:02 |
opendevreview | Alan Bishop proposed openstack/cinder master: Add infrastructure for testing new RBAC policies https://review.opendev.org/c/openstack/cinder/+/805316 | 23:02 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!