| *** hemna6 is now known as hemna | 07:37 | |
| *** _pewp__ is now known as _pewp_ | 07:37 | |
| bingyanh | Hi Core-reviewers, this need one more +2 on https://review.opendev.org/c/openstack/cinder/+/819790 can someone help review ~ | 10:00 |
|---|---|---|
| opendevreview | yuval proposed openstack/os-brick master: nvmeof.py - in case of missing get-hostnqn command https://review.opendev.org/c/openstack/os-brick/+/823654 | 10:39 |
| *** dviroel|rover|afk is now known as dviroel|rover | 11:16 | |
| opendevreview | Helen Walsh proposed openstack/cinder master: PowerMax Driver - Improve error handling around deletes https://review.opendev.org/c/openstack/cinder/+/796286 | 14:06 |
| opendevreview | Helen Walsh proposed openstack/cinder master: PowerMax Driver - Manage volume into correct storage group https://review.opendev.org/c/openstack/cinder/+/797333 | 14:18 |
| abishop | stable-cores: hi, one final backport request for this one https://review.opendev.org/c/openstack/os-brick/+/823366 | 14:22 |
| jungleboyj | bingyanh: Merging. There are some typos that need to be addressed but I can push up a patch to fix those. | 14:27 |
| jungleboyj | abishop: Merging. | 14:27 |
| abishop | jungleboyj: thx! | 14:28 |
| jungleboyj | Welcome. :-) | 14:28 |
| bingyanh | jungleboyj: thank you | 14:36 |
| opendevreview | Sofia Enriquez proposed openstack/devstack-plugin-ceph master: WIP Allow the usage of Clone v2 API https://review.opendev.org/c/openstack/devstack-plugin-ceph/+/782624 | 14:57 |
| opendevreview | Helen Walsh proposed openstack/cinder master: PowerMax Driver - RDF checks https://review.opendev.org/c/openstack/cinder/+/768105 | 15:22 |
| *** dviroel|rover is now known as dviroel | 15:26 | |
| zigo | Hi. I just filled: https://bugs.launchpad.net/cinder/+bug/1956601 | 15:36 |
| zigo | Is there any consequence of the max= value appart from what's in the generated cinder.conf? For example, does oslo_config checks for that value? | 15:36 |
| opendevreview | Eric Harney proposed openstack/cinder master: Fix spacing in CapacityFilter debug log message https://review.opendev.org/c/openstack/cinder/+/823707 | 15:39 |
| opendevreview | Jay Bryant proposed openstack/cinder master: Fix Typos from Review 819790 https://review.opendev.org/c/openstack/cinder/+/823711 | 15:50 |
| jungleboyj | bingyanh: ^^ There is the follow-up patch. | 15:51 |
| opendevreview | Merged openstack/os-brick stable/train: linuxscsi: Only raise and log multipathd errors when required https://review.opendev.org/c/openstack/os-brick/+/823366 | 16:28 |
| venkatakrishnat | Hi core-reviewers, Could someone please review this patch https://review.opendev.org/c/openstack/cinder/+/810917. Thanks | 16:31 |
| venkatakrishnat | Hi, Looking for another +2 for https://review.opendev.org/c/openstack/cinder/+/809830 . Could someone please have a look. Thanks in adv. | 16:44 |
| jungleboyj | venkatakrishnat: Done. :-) | 16:52 |
| venkatakrishnat | jungleboyj: Thank you. | 17:02 |
| opendevreview | Sofia Enriquez proposed openstack/cinder-tempest-plugin master: WIP delete vol with dependency https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/823719 | 17:03 |
| opendevreview | Sofia Enriquez proposed openstack/devstack-plugin-ceph master: Set Ceph min client version to Mimic https://review.opendev.org/c/openstack/devstack-plugin-ceph/+/782624 | 17:20 |
| walshh_ | Hi all, would really appreciate a review of https://review.opendev.org/c/openstack/cinder/+/801354 and https://review.opendev.org/c/openstack/cinder/+/797970. Both have clean CI. Thank you. | 17:22 |
| _pewp_ | hemna ヽ(´･ω･`)､ | 18:10 |
| opendevreview | Eric Harney proposed openstack/cinder master: WIP: RBD: Attempt remove() before trash_move() https://review.opendev.org/c/openstack/cinder/+/823726 | 18:26 |
| *** dviroel is now known as dviroel|afk | 18:29 | |
| opendevreview | Sofia Enriquez proposed openstack/cinder-tempest-plugin master: Basic Delete Volumes With Dependencies Tests https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/823719 | 19:37 |
| opendevreview | Sofia Enriquez proposed openstack/cinder-tempest-plugin master: Add test to create multiple volume from resource https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/795753 | 19:59 |
| simondodsley | anyone around who understands the core revert-to-snapshot code? I think there maybe a fundamental issue with this code but want to run it past someone else | 20:28 |
| opendevreview | Merged openstack/cinder master: PowerMax Driver - Allow for volume uuid in manage volume https://review.opendev.org/c/openstack/cinder/+/772388 | 21:37 |
| opendevreview | Merged openstack/cinder master: [SVF] Fix rccg and relationship creation issues https://review.opendev.org/c/openstack/cinder/+/809830 | 22:12 |
| opendevreview | Merged openstack/cinder master: Fix spacing in CapacityFilter debug log message https://review.opendev.org/c/openstack/cinder/+/823707 | 22:12 |
| opendevreview | Tom Barron proposed openstack/cinder master: Don't drop group type extra specs at DB layer https://review.opendev.org/c/openstack/cinder/+/796114 | 23:46 |
| opendevreview | Tom Barron proposed openstack/cinder master: Rename db implementation API extra specs method https://review.opendev.org/c/openstack/cinder/+/796113 | 23:46 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!