| rm_work | hey, for a while now we've been patching in a concept of "read only admin", so we can allow some users to see everything owned by any user, but not actually create/delete/etc... it looks like permissions have been overhauled recently and our patches no longer apply cleanly. I figured before I do a bunch of work to fix that, I should ask if there's already a matching concept that I can just ... use 😛 | 00:01 |
|---|---|---|
| rm_work | and if not, if anyone else thinks something like that would be useful to include by default | 00:02 |
| rosmaita | rm_work: this doc explains what we're doing ... it may not fit exactly with your custom scheme, but see what you think: https://docs.openstack.org/cinder/xena/configuration/block-storage/policy-personas.html | 03:40 |
| rosmaita | rm_work: also, the xena release notes have a list of the changes made in xena: https://docs.openstack.org/releasenotes/cinder/xena.html | 03:42 |
| opendevreview | Harsh Ailani proposed openstack/cinder master: [SVF] Add cleanrate in volume-type extra specs https://review.opendev.org/c/openstack/cinder/+/811877 | 05:46 |
| *** elodilles_pto is now known as elodilles | 07:12 | |
| opendevreview | Ivan Kolodyazhny proposed openstack/os-brick master: Return only one path in NVMe connector https://review.opendev.org/c/openstack/os-brick/+/811886 | 08:10 |
| opendevreview | Vladislav Belogrudov proposed openstack/os-brick master: Fix paths for NVMe devices (non-nguid) https://review.opendev.org/c/openstack/os-brick/+/811944 | 09:44 |
| opendevreview | Vladislav Belogrudov proposed openstack/os-brick master: Fix paths for NVMe devices (non-nguid) https://review.opendev.org/c/openstack/os-brick/+/811944 | 09:57 |
| raghavendrat | hi geguileo: are you online ? | 10:17 |
| *** dviroel|out is now known as dviroel | 11:21 | |
| *** tobberydberg_ is now known as tobberydberg | 11:51 | |
| opendevreview | Seena Fallah proposed openstack/cinder master: snapshot: allow revert to any point for rbd driver https://review.opendev.org/c/openstack/cinder/+/806807 | 12:06 |
| *** bhagyashris is now known as bhagyashris|rover | 12:43 | |
| raghavendrat | Hi reviewers ... below patch has got one +2. | 13:02 |
| raghavendrat | It would be great if someone can have a look. Thanks. | 13:02 |
| raghavendrat | https://review.opendev.org/c/openstack/cinder/+/804698 [HPE 3PAR: Reuse existing session] | 13:02 |
| rosmaita | geguileo: eharney: could use your feedback on Add "known issues" to xena release note: https://review.opendev.org/c/openstack/cinder/+/811760 | 14:08 |
| geguileo | sophieh20201: sorry, I misseed your ping from yesterday | 14:13 |
| geguileo | sophieh20201: I'll have a look at the patch again | 14:13 |
| *** bhagyashris_ is now known as bhagyashris|rover | 14:19 | |
| geguileo | rosmaita: done | 14:28 |
| rosmaita | geguileo: ty | 14:28 |
| sophieh20201 | geguileo: Good morning! | 15:09 |
| geguileo | sophieh20201: morning! I replied in your patch to the locking vs retrying | 15:12 |
| sophieh20201 | geguileo: got your comments and understood your concerns. Based on the logs, it seems that the database error is introduced by the database access of all the multiple threads, not the same thread for a target plus portal - I only have one lun in that testing. Do you think I can just lock the _connect_to_iscsi_portal for all the access, instread of target and portal | 15:14 |
| opendevreview | Vladislav Belogrudov proposed openstack/os-brick master: Fix paths for NVMe devices (non-nguid) https://review.opendev.org/c/openstack/os-brick/+/811944 | 15:17 |
| opendevreview | Brian Rosmaita proposed openstack/os-brick stable/xena: Fix paths for NVMe devices (non-nguid) https://review.opendev.org/c/openstack/os-brick/+/812010 | 16:15 |
| fabiooliveira | needing review on this patch https://review.opendev.org/c/openstack/cinder/+/806616 -- can I ping the ones who already reviewed old patch sets? | 16:20 |
| fabiooliveira | tosky, walshh_, simondodsley, jungleboyj, raghavendrat, enriquetaso, geguileo ^^ at some point you all reviewed the patch above, could you guys take a look :) thank you | 18:05 |
| opendevreview | Merged openstack/cinder stable/xena: Add "known issues" to xena release note https://review.opendev.org/c/openstack/cinder/+/811760 | 20:07 |
| opendevreview | Merged openstack/os-brick master: Fix paths for NVMe devices (non-nguid) https://review.opendev.org/c/openstack/os-brick/+/811944 | 20:22 |
| rosmaita | cinder-stable-maint cores: need approval of https://review.opendev.org/c/openstack/os-brick/+/812010 | 20:26 |
| rosmaita | e0ne eharney geguileo jungleboyj smcginnis whoami-rajat ^^ | 20:26 |
| pots | rosmaita: can https://review.opendev.org/c/openstack/cinder/+/811156 and https://review.opendev.org/c/openstack/cinder/+/811154 can go into stable/xena? one is a doc fix and the other's a trivial one-liner | 20:52 |
| rosmaita | pots: i think we'll have to wait for the first point release | 20:53 |
| pots | rosmaita: ok :( | 20:54 |
| *** dviroel is now known as dviroel|out | 21:56 | |
| opendevreview | Seena Fallah proposed openstack/cinder-tempest-plugin master: volume: fix wrong snapshot_id for revert_to_snapshot_after_extended_negative https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/812032 | 21:56 |
| opendevreview | Merged openstack/os-brick stable/xena: Fix paths for NVMe devices (non-nguid) https://review.opendev.org/c/openstack/os-brick/+/812010 | 22:34 |
| rosmaita | \o/ | 22:35 |
| opendevreview | Merged openstack/cinder master: NetApp Solidfire: Fix retype and name exception on migration https://review.opendev.org/c/openstack/cinder/+/806616 | 22:48 |
| opendevreview | Merged openstack/cinder master: HPE 3PAR: Reuse existing session https://review.opendev.org/c/openstack/cinder/+/804698 | 23:09 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!