Tuesday, 2021-11-02

opendevreviewMerged openstack/cinder master: Update rbd warning message  https://review.opendev.org/c/openstack/cinder/+/81221601:34
opendevreviewMounika Sreeram proposed openstack/cinder master: [SVF] Fix Retype issue of mirror volume  https://review.opendev.org/c/openstack/cinder/+/81632411:54
opendevreviewNaoki Saito proposed openstack/cinder master: Add cinder volume drivers for NEC V series Storage  https://review.opendev.org/c/openstack/cinder/+/81561412:20
opendevreviewMerged openstack/cinder stable/xena: PowerMax Docs - Xena release notes  https://review.opendev.org/c/openstack/cinder/+/81398612:42
geguileorosmaita: While suffering with the quotas I've noticed a couple of things related to our policies13:13
geguileoa lot of warnings13:13
geguileothe fact that a user can no longer see their own quotas13:14
rosmaitathat one is surprising13:14
rosmaitado we have the policy incorrect, or is there something else going on?13:15
geguileommmmmm, maybe I'm not running the latest code and that was something that only happened for a bit...13:15
* geguileo goes to check13:15
geguileorosmaita: OK, most of the warnings seem to be gone with latest code, but we still get things like: /usr/local/lib/python3.6/site-packages/oslo_policy/policy.py:1552: DeprecationWarning: volume_extension:quotas:show deprecated without deprecated_reason or deprecated_since. This will be an error in a future release13:18
geguileorosmaita: sorry, the warnings are there, but apparently we do some lazy loading and will only show if I hit the API13:19
geguileorosmaita: Lots of Nov 02 13:18:48 localhost.localdomain devstack@c-api.service[261871]: /usr/local/lib/python3.6/site-packages/oslo_policy/policy.py:785: UserWarning: Policy "volume:create_from_image":"" was deprecated in X in favor of "volume:create_from_image":"rule:xena_system_admin_or_project_member". Reason: Default policies now support the three Keystone default roles,13:19
geguileonamely 'admin', 'member', and 'reader' to implement three Cinder "personas".  See "Policy Personas and Permissions" in the "Cinder Service Configuration" documentation (Xena release) for details.. Either ensure your deployment is ready for the new default or copy/paste the deprecated policy into your policy file and maintain it manually.                                      13:19
geguileo                                                                                                                  13:19
geguileoNov 02 13:18:48 localhost.localdomain devstack@c-api.service[261871]:   warnings.warn(deprecated_msg)                                                                                                                                                                                                                                                                                    13:19
geguileo                                                13:19
rosmaitado you get those on every request, or just startup?13:20
geguileorosmaita: I believe on the first request that hits the policy (we must have some kind of lazy loading)13:23
geguileoI also found a quota bug (shows type dependent quota for private types), but since that's not policy related I won't bother you with it  ;-)13:24
rosmaitaworks for me!13:25
rosmaitageguileo: i will follow up about that deprecation_since etc notice13:25
geguileorosmaita: that one happens on load if you write a policy.yaml file13:25
geguileothe other warning happens on first API hit13:26
rosmaitaok, thanks13:26
opendevreviewWalt proposed openstack/cinder master: Rework backup process to make it async  https://review.opendev.org/c/openstack/cinder/+/78447714:25
opendevreviewHelen Walsh proposed openstack/cinder master: PowerMax Driver - Improve error handling around deletes  https://review.opendev.org/c/openstack/cinder/+/79628614:54
simondodsleyrosmaita: Could I get some core eye on https://review.opendev.org/c/openstack/cinder/+/815884 and https://review.opendev.org/c/openstack/cinder/+/81584814:58
rosmaitasimondodsley: i'm in the middle of reviewing a patch, will promise to get to yours next15:06
simondodsleymine are small and will be a light relief for you15:09
*** ianw_pto is now known as ianw19:00
opendevreviewSimon Dodsley proposed openstack/cinder master: [Pure Storage] Add check for NVMe-FC capable array  https://review.opendev.org/c/openstack/cinder/+/81584822:26
opendevreviewrajadeepan proposed openstack/cinder master: Fixed copy-on-write mode in GPFS NFS driver  https://review.opendev.org/c/openstack/cinder/+/81398422:55

Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!