opendevreview | Merged openstack/python-glanceclient master: Fix undesirable raw Python error https://review.opendev.org/c/openstack/python-glanceclient/+/803326 | 01:30 |
---|---|---|
opendevreview | Merged openstack/glance master: Check get_image(s) in the API https://review.opendev.org/c/openstack/glance/+/796067 | 02:16 |
opendevreview | Merged openstack/glance master: Add a member field to Image when appropriate https://review.opendev.org/c/openstack/glance/+/796066 | 06:39 |
*** akekane_ is now known as abhishekk | 06:41 | |
opendevreview | Abhishek Kekane proposed openstack/glance master: Refactor gateway auth layer for metadef APIs https://review.opendev.org/c/openstack/glance/+/799632 | 11:34 |
opendevreview | Abhishek Kekane proposed openstack/glance master: Move metadef namespace policy checks in the API https://review.opendev.org/c/openstack/glance/+/799633 | 11:34 |
opendevreview | Abhishek Kekane proposed openstack/glance master: Move metadef object policy checks in the API https://review.opendev.org/c/openstack/glance/+/799634 | 11:34 |
opendevreview | Abhishek Kekane proposed openstack/glance master: Move metadef resource type association policy checks in the API https://review.opendev.org/c/openstack/glance/+/799637 | 11:35 |
opendevreview | Abhishek Kekane proposed openstack/glance master: Move metadef property policy checks in the API https://review.opendev.org/c/openstack/glance/+/799635 | 11:35 |
opendevreview | Abhishek Kekane proposed openstack/glance master: Move metadef tag policy checks in the API https://review.opendev.org/c/openstack/glance/+/799636 | 11:35 |
opendevreview | Abhishek Kekane proposed openstack/glance master: Refactor gateway auth layer for task APIs https://review.opendev.org/c/openstack/glance/+/802243 | 12:09 |
opendevreview | Abhishek Kekane proposed openstack/glance master: Deprecate task specific policies https://review.opendev.org/c/openstack/glance/+/802244 | 12:09 |
opendevreview | Abhishek Kekane proposed openstack/glance master: Move Tasks policy checks in the API https://review.opendev.org/c/openstack/glance/+/803590 | 12:09 |
abhishekk | jokke_, rosmaita, dansmith, croelandt, weekly meeting in 10 minutes at #openstack-meeting | 13:51 |
rosmaita | abhishekk: ty | 13:51 |
abhishekk | np | 13:51 |
opendevreview | Merged openstack/glance master: Remove SSL configuration section from docs https://review.opendev.org/c/openstack/glance/+/802868 | 14:47 |
opendevreview | Erno Kuvaja proposed openstack/glance stable/wallaby: Remove SSL configuration section from docs https://review.opendev.org/c/openstack/glance/+/803618 | 15:03 |
opendevreview | Erno Kuvaja proposed openstack/glance stable/victoria: Remove SSL configuration section from docs https://review.opendev.org/c/openstack/glance/+/803619 | 15:05 |
opendevreview | Erno Kuvaja proposed openstack/glance stable/ussuri: Remove SSL configuration section from docs https://review.opendev.org/c/openstack/glance/+/803620 | 15:06 |
opendevreview | Erno Kuvaja proposed openstack/glance stable/victoria: Remove SSL configuration section from docs https://review.opendev.org/c/openstack/glance/+/803619 | 15:14 |
opendevreview | Erno Kuvaja proposed openstack/glance stable/ussuri: Remove SSL configuration section from docs https://review.opendev.org/c/openstack/glance/+/803620 | 15:17 |
* akekane_ going for dinner break | 16:01 | |
*** akekane_ is now known as abhishekk | 16:01 | |
croelandt | dansmith: have you lookd at https://review.opendev.org/c/openstack/glance/+/798073/26 ? It seems like a simple 403->404 missed in the functional tests | 16:11 |
dansmith | nope, but I ran tests locally and they passed. we've had a number of seemingly non-deterministic things with this stuff, | 16:25 |
dansmith | I wonder if we've got some global state that is causing these to be unpredictable | 16:25 |
croelandt | does the failing functional test actually pass, or is it skipped? | 16:27 |
dansmith | I'm re-running locally to check and see if I'm a liar | 16:27 |
dansmith | damn | 16:29 |
dansmith | I guess I'm a liar | 16:29 |
dansmith | hmm, that shouldn't be a 404 if that tenant can see the image, which the test seems to assert | 16:31 |
dansmith | ah, but the rbac job passes this test | 16:32 |
dansmith | okay yeah, so this test was previously asserting 403 because the auth layer would raise Forbidden regardless of whether or not you *should* see it, and now we fail to even get the image because the DB layer asserts different behavior I think | 16:41 |
dansmith | yet another example of all the layers disagreeing on what the behavior should be | 16:41 |
* abhishekk I think my entire time will go on metadefs only :/ | 16:43 | |
opendevreview | Merged openstack/glance master: Clean out deprecations from paste.ini https://review.opendev.org/c/openstack/glance/+/802867 | 17:34 |
opendevreview | Abhishek Kekane proposed openstack/glance master: Refactor gateway auth layer for metadef APIs https://review.opendev.org/c/openstack/glance/+/799632 | 19:02 |
opendevreview | Abhishek Kekane proposed openstack/glance master: Move metadef namespace policy checks in the API https://review.opendev.org/c/openstack/glance/+/799633 | 19:02 |
opendevreview | Abhishek Kekane proposed openstack/glance master: Move metadef object policy checks in the API https://review.opendev.org/c/openstack/glance/+/799634 | 19:02 |
opendevreview | Abhishek Kekane proposed openstack/glance master: Move metadef resource type association policy checks in the API https://review.opendev.org/c/openstack/glance/+/799637 | 19:02 |
opendevreview | Abhishek Kekane proposed openstack/glance master: Move metadef property policy checks in the API https://review.opendev.org/c/openstack/glance/+/799635 | 19:02 |
opendevreview | Abhishek Kekane proposed openstack/glance master: Move metadef tag policy checks in the API https://review.opendev.org/c/openstack/glance/+/799636 | 19:02 |
abhishekk | dansmith, lbragstad, croelandt this could be final revision for metadef policy refactoring | 19:03 |
dansmith | gah abhishekk, making me look bad | 19:03 |
abhishekk | :o | 19:03 |
abhishekk | My situation is, in dreams also I see policy checks :P | 19:04 |
dansmith | lol | 19:04 |
dansmith | I'm quite definitely NOT dreaming about policy checks | 19:16 |
lbragstad | abhishekk don't get too excited, they'll turn to nightmares soon | 19:25 |
abhishekk | haha | 19:25 |
abhishekk | dansmith, you are lucky | 19:26 |
-opendevstatus- NOTICE: The Gerrit service on review.opendev.org is going down for a quick restart to adjust its database connection configuration, and should return to service momentarily | 20:03 | |
abhishekk | :/ and i thought something is wrong with my network | 20:04 |
dansmith | dang, just saved a comment and got 503 | 20:05 |
abhishekk | :D | 20:12 |
abhishekk | looks like its up now | 20:13 |
opendevreview | Abhishek Kekane proposed openstack/glance master: Refactor gateway auth layer for task APIs https://review.opendev.org/c/openstack/glance/+/802243 | 20:17 |
opendevreview | Abhishek Kekane proposed openstack/glance master: Deprecate task specific policies https://review.opendev.org/c/openstack/glance/+/802244 | 20:17 |
opendevreview | Abhishek Kekane proposed openstack/glance master: Move Tasks policy checks in the API https://review.opendev.org/c/openstack/glance/+/802245 | 20:17 |
* abhishekk signing out for the day | 20:20 | |
abhishekk | now I can resume work on member APIs | 20:21 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!