openstackgerrit | Abitha Palaniappan proposed openstack/cue: Validate uuid parameters at api layer https://review.openstack.org/240113 | 00:37 |
---|---|---|
openstackgerrit | OpenStack Proposal Bot proposed openstack/cue: Updated from global requirements https://review.openstack.org/253891 | 02:05 |
openstackgerrit | OpenStack Proposal Bot proposed openstack/cue: Updated from global requirements https://review.openstack.org/253891 | 02:28 |
*** ViswaV has quit IRC | 04:34 | |
*** ViswaV has joined #openstack-cue | 04:38 | |
*** openstackgerrit has quit IRC | 11:02 | |
*** openstackgerrit has joined #openstack-cue | 11:03 | |
*** miqui has joined #openstack-cue | 15:17 | |
*** openstackgerrit has quit IRC | 16:47 | |
*** openstackgerrit has joined #openstack-cue | 16:48 | |
*** ViswaV has quit IRC | 17:44 | |
*** ViswaV has joined #openstack-cue | 17:47 | |
*** openstackgerrit has quit IRC | 19:02 | |
*** openstackgerrit has joined #openstack-cue | 19:03 | |
krotscheck | vipul: Have you and min talked about the bug I found in cue's policy yet? https://review.openstack.org/#/c/253221/1/cue/tests/functional/api/test_policy.py | 19:16 |
*** harlowja has quit IRC | 19:38 | |
*** harlowja has joined #openstack-cue | 19:38 | |
vipul | krotscheck: hey what's up | 19:46 |
krotscheck | Hey there! | 19:46 |
vipul | what's the policy issue? | 19:46 |
krotscheck | vipul: I've outlined it in the unit tests I linked ^^ | 19:47 |
vipul | hmm.. multiple inits | 19:47 |
krotscheck | vipul: That's the first test. The second test is slightly different. | 19:48 |
vipul | krotscheck: ok i'll take a look .. | 19:53 |
vipul | krotscheck: is that impacting another patch you have? | 19:53 |
krotscheck | vipul: Yes, that second test suite highlights the bug that prevents https://review.openstack.org/#/c/237129/ from landing. | 19:54 |
vipul | krotscheck: we're making some changes to how we enforce the policy, currently happens at the DB layer.. do you happen to know / have an opinion on if these are useful for non-api based requests? ie. some backend service accessing a database object and doing an RBAC check | 19:57 |
krotscheck | vipul: Nope, no opinion. I live in the UI world, and expect the API to give me sane error messages when I violate access policies :) | 19:58 |
openstackgerrit | Bertrand Lallau proposed openstack/cue: Remove anyjson useless requirement https://review.openstack.org/254960 | 21:23 |
openstackgerrit | Bertrand Lallau proposed openstack/cue: Remove jsonpatch useless requirement https://review.openstack.org/254968 | 21:43 |
openstackgerrit | Abitha Palaniappan proposed openstack/cue: Fix for urllib2 import error in gate-cue-python34 https://review.openstack.org/254973 | 21:54 |
*** ViswaV has quit IRC | 22:18 | |
*** ViswaV has joined #openstack-cue | 22:20 | |
openstackgerrit | Abitha Palaniappan proposed openstack/cue: Fix for the import error in gate-cue-integration-dsvm-rabbitmq job https://review.openstack.org/254991 | 22:47 |
openstackgerrit | Saurabh Surana proposed openstack/cue: policy checks are not being done in the api layer https://review.openstack.org/254994 | 22:51 |
openstackgerrit | Saurabh Surana proposed openstack/cue: policy checks are not being done in the api layer https://review.openstack.org/254994 | 22:52 |
openstackgerrit | Saurabh Surana proposed openstack/cue: policy checks are now being done in the api layer https://review.openstack.org/254994 | 22:54 |
*** esmute has quit IRC | 22:55 | |
*** esmute_ has quit IRC | 22:56 | |
openstackgerrit | Saurabh Surana proposed openstack/cue: policy checks are now being done in the api layer https://review.openstack.org/254994 | 23:06 |
openstackgerrit | Abitha Palaniappan proposed openstack/cue: Fix import error in gate-cue-integration-dsvm-rabbitmq job https://review.openstack.org/254991 | 23:10 |
openstackgerrit | Abitha Palaniappan proposed openstack/cue: Fix for urllib2 import error in gate-cue-python34 https://review.openstack.org/254973 | 23:23 |
*** ViswaV_ has joined #openstack-cue | 23:50 | |
*** ViswaV has quit IRC | 23:51 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!