Comment 2 for bug 1456115

Revision history for this message
Kiall Mac Innes (kiall) wrote :

[15:29:14] <Kiall> stanzgy_: BTW, re https://review.openstack.org/#/c/184432/ / bug 1456115
[15:29:41] <Kiall> I'm not 100% convinced that really is a bug, is_admin is just badly named these days now that oslo.policy is a thing
[15:30:19] <Kiall> is_admin is meant as a hint to olso.policy to "say yes to everything", rather than the other way around
[15:30:50] <Kiall> Nothing bar oslo.policy should ever look at the value of context.is_admin, if it does, it's bypassing policy and is a bug
[15:33:13] <Kiall> I'll post those comments in the bug report, seems stanzgy's real IRC handle already dropped off