| opendevreview | Merged openstack/cinder master: Update rbd warning message https://review.opendev.org/c/openstack/cinder/+/812216 | 01:34 |
|---|---|---|
| opendevreview | Mounika Sreeram proposed openstack/cinder master: [SVF] Fix Retype issue of mirror volume https://review.opendev.org/c/openstack/cinder/+/816324 | 11:54 |
| opendevreview | Naoki Saito proposed openstack/cinder master: Add cinder volume drivers for NEC V series Storage https://review.opendev.org/c/openstack/cinder/+/815614 | 12:20 |
| opendevreview | Merged openstack/cinder stable/xena: PowerMax Docs - Xena release notes https://review.opendev.org/c/openstack/cinder/+/813986 | 12:42 |
| geguileo | rosmaita: While suffering with the quotas I've noticed a couple of things related to our policies | 13:13 |
| geguileo | a lot of warnings | 13:13 |
| geguileo | the fact that a user can no longer see their own quotas | 13:14 |
| rosmaita | that one is surprising | 13:14 |
| rosmaita | do we have the policy incorrect, or is there something else going on? | 13:15 |
| geguileo | mmmmmm, maybe I'm not running the latest code and that was something that only happened for a bit... | 13:15 |
| * geguileo goes to check | 13:15 | |
| geguileo | rosmaita: 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 release | 13:18 |
| geguileo | rosmaita: sorry, the warnings are there, but apparently we do some lazy loading and will only show if I hit the API | 13:19 |
| geguileo | rosmaita: 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 |
| geguileo | namely '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 | |
| geguileo | Nov 02 13:18:48 localhost.localdomain devstack@c-api.service[261871]: warnings.warn(deprecated_msg) | 13:19 |
| geguileo | 13:19 | |
| rosmaita | do you get those on every request, or just startup? | 13:20 |
| geguileo | rosmaita: I believe on the first request that hits the policy (we must have some kind of lazy loading) | 13:23 |
| geguileo | I 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 |
| rosmaita | works for me! | 13:25 |
| rosmaita | geguileo: i will follow up about that deprecation_since etc notice | 13:25 |
| geguileo | rosmaita: that one happens on load if you write a policy.yaml file | 13:25 |
| geguileo | the other warning happens on first API hit | 13:26 |
| rosmaita | ok, thanks | 13:26 |
| opendevreview | Walt proposed openstack/cinder master: Rework backup process to make it async https://review.opendev.org/c/openstack/cinder/+/784477 | 14:25 |
| opendevreview | Helen Walsh proposed openstack/cinder master: PowerMax Driver - Improve error handling around deletes https://review.opendev.org/c/openstack/cinder/+/796286 | 14:54 |
| simondodsley | rosmaita: Could I get some core eye on https://review.opendev.org/c/openstack/cinder/+/815884 and https://review.opendev.org/c/openstack/cinder/+/815848 | 14:58 |
| rosmaita | perhaps | 14:58 |
| simondodsley | lol | 14:59 |
| rosmaita | simondodsley: i'm in the middle of reviewing a patch, will promise to get to yours next | 15:06 |
| simondodsley | mine are small and will be a light relief for you | 15:09 |
| *** ianw_pto is now known as ianw | 19:00 | |
| opendevreview | Simon Dodsley proposed openstack/cinder master: [Pure Storage] Add check for NVMe-FC capable array https://review.opendev.org/c/openstack/cinder/+/815848 | 22:26 |
| opendevreview | rajadeepan proposed openstack/cinder master: Fixed copy-on-write mode in GPFS NFS driver https://review.opendev.org/c/openstack/cinder/+/813984 | 22:55 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!