| gksk | To the core reviewers please review the below patch related to update replication properties fix for empty values | 04:07 |
|---|---|---|
| gksk | https://review.opendev.org/c/openstack/cinder/+/801617 | 04:07 |
| opendevreview | Brian Rosmaita proposed openstack/os-brick master: DNM: prototyping the config for nvmeof-agent https://review.opendev.org/c/openstack/os-brick/+/804813 | 04:10 |
| opendevreview | Girish Chilukuri proposed openstack/cinder master: [SVF]:Fix multiple lshost calls during attach. https://review.opendev.org/c/openstack/cinder/+/772623 | 07:28 |
| *** dviroel|out is now known as dviroel|ruck | 11:26 | |
| opendevreview | Tushar Trambak Gite proposed openstack/cinder master: Reset state robustification for backup os-reset_status https://review.opendev.org/c/openstack/cinder/+/778193 | 11:26 |
| opendevreview | Venkata krishna Thumu proposed openstack/cinder master: [SVF] Fix extend issue for mirroring volumes https://review.opendev.org/c/openstack/cinder/+/784140 | 11:51 |
| venkatakrishnat | Hi eharney, geguileo, hemna, jungleboyj: In your available time, please review https://review.opendev.org/c/openstack/cinder/+/788253 which is to fix volume-relationship fetching issue on IBM SVC cinder driver. CI also passed. | 11:55 |
| *** thelounge555 is now known as thelounge55 | 13:35 | |
| opendevreview | Rajat Dhasmana proposed openstack/cinder master: Add releasenote for schema validation fix https://review.opendev.org/c/openstack/cinder/+/804611 | 14:13 |
| opendevreview | Vladislav Belogrudov proposed openstack/os-brick master: Add support for multiple volumes within subsystem to NVMe-OF connector https://review.opendev.org/c/openstack/os-brick/+/800014 | 14:20 |
| opendevreview | Vladislav Belogrudov proposed openstack/os-brick master: Add support for multiple volumes within subsystem to NVMe-OF connector https://review.opendev.org/c/openstack/os-brick/+/800014 | 14:29 |
| opendevreview | Helen Walsh proposed openstack/cinder master: PowerMax Driver - Allow for retype to _DEFAULT__ VT https://review.opendev.org/c/openstack/cinder/+/803773 | 14:43 |
| opendevreview | Eric Harney proposed openstack/cinder master: mypy: coordination.py https://review.opendev.org/c/openstack/cinder/+/788364 | 14:50 |
| opendevreview | Eric Harney proposed openstack/cinder master: mypy: filter scheduler https://review.opendev.org/c/openstack/cinder/+/788365 | 14:51 |
| venkatakrishnat | Hi geguileo, hemna, jungleboyj, rosmaita, whoami-rajat: Waiting for workflow +2 for https://review.opendev.org/c/openstack/cinder/+/788253 which is to fix volume-relationship fetch issue on IBM SVC cinder driver. | 14:51 |
| opendevreview | Rajat Dhasmana proposed openstack/cinder master: Add releasenote for schema validation fix https://review.opendev.org/c/openstack/cinder/+/804611 | 14:54 |
| amar___ | Hi eharney : Could you pls have a look at this patch : https://review.opendev.org/c/openstack/cinder/+/790910 | 15:09 |
| opendevreview | katari manoj kumar proposed openstack/cinder master: [SVF]:Fix create volume on drp https://review.opendev.org/c/openstack/cinder/+/787115 | 15:37 |
| opendevreview | Simon Dodsley proposed openstack/cinder master: [WIP] Add Pure Storage NVMe-RoCE driver https://review.opendev.org/c/openstack/cinder/+/799871 | 16:12 |
| venkatakrishnat | Hi geguileo, hemna, jungleboyj, rosmaita: Waiting for workflow +2 for https://review.opendev.org/c/openstack/cinder/+/788253 Please review this patch which is to fix volume-relationship fetch issue on IBM SVC cinder driver. | 17:18 |
| simondodsley | Hi cores. Need last +2 & +W for https://review.opendev.org/c/openstack/cinder/+/804345. Needed for next PowerVC release, so as soon as possible would be good | 17:53 |
| opendevreview | Merged openstack/cinder master: [SVF] Fix issue to get volume relationship details https://review.opendev.org/c/openstack/cinder/+/788253 | 19:09 |
| opendevreview | Eric Harney proposed openstack/cinder master: mypy: filter scheduler https://review.opendev.org/c/openstack/cinder/+/788365 | 19:26 |
| opendevreview | Merged openstack/cinder master: [SVF]:Fix create volume on drp https://review.opendev.org/c/openstack/cinder/+/787115 | 19:40 |
| opendevreview | Merged openstack/cinder stable/wallaby: [rbd] Fix create encrypted volume from snapshot https://review.opendev.org/c/openstack/cinder/+/803181 | 20:48 |
| lbragstad | abishop i know glance's architecture is a little different, but this is how they are working through the 403/404 uniformity https://github.com/openstack/glance/blob/master/glance/api/v2/policy.py#L92-L105 | 20:51 |
| abishop | lbragstad: ack, makes sense (and good to see an example). The challenge for cinder is determining how many variations of the conditional on L103 need to be considered | 20:57 |
| lbragstad | abishop yeah - that snippet is specific to images | 20:57 |
| lbragstad | so - i think glance is assuming it's safe to check those conditions there | 20:58 |
| lbragstad | but they re-use that strategy for other like-checks https://github.com/openstack/glance/blob/master/glance/api/v2/policy.py#L177-L189 | 20:58 |
| lbragstad | if cinder took a similar approach, you could tailor the 404 if you wanted to | 20:59 |
| abishop | yeah, that's probably the way to go | 20:59 |
| *** dviroel|ruck is now known as dviroel|ruck|out | 21:15 | |
| opendevreview | Ade Lee proposed openstack/cinder master: WIP/DNM: Add fips check jobs https://review.opendev.org/c/openstack/cinder/+/790535 | 21:31 |
| opendevreview | Merged openstack/cinder stable/wallaby: [Pure Storage] Resolve missing provider_id issue (PowerVC) https://review.opendev.org/c/openstack/cinder/+/804345 | 21:53 |
| *** dwhite is now known as dwhite4 | 22:46 | |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!