opendevreview | Merged openstack/glance master: Release notes for Zed Milestone 3 https://review.opendev.org/c/openstack/glance/+/854496 | 06:15 |
---|---|---|
*** akekane_ is now known as abhishekk | 08:33 | |
opendevreview | Pranali Deore proposed openstack/python-glanceclient master: Add missing tests for glance-download import method https://review.opendev.org/c/openstack/python-glanceclient/+/855494 | 10:18 |
opendevreview | Ghanshyam proposed openstack/glance-tempest-plugin master: Update Image APIs tests to check system user is not permitted https://review.opendev.org/c/openstack/glance-tempest-plugin/+/855440 | 12:00 |
pdeore | gmann, is this glance RBAC patch - dropping system scope is expected for m3 ? https://review.opendev.org/c/openstack/glance/+/855435 | 12:13 |
pdeore | abhishekk, rosmaita, jokke_, dansmith, croelandt, mrjoshi glance weekly meeting in 10 minutes at #openstack-meeting | 13:50 |
abhishekk | ty! | 13:50 |
pdeore | np !! | 13:50 |
dansmith | is there anything I need to pay attention to? | 13:51 |
dansmith | I'm sprinting for a nova FFE thing | 13:51 |
pdeore | dansmith, yeah, we are planning to tag m3 today but there is one patch up regarding RBAC work - "Dropping system scope" https://review.opendev.org/c/openstack/glance/+/855435 | 13:53 |
pdeore | so is this expected to have in m3 or we can have this in rc1 ? | 13:53 |
dansmith | should be in m3 if at all possible | 13:55 |
abhishekk | it has -w that needs to be lifted | 13:55 |
abhishekk | or else we need to push m3 next week | 13:56 |
dansmith | yeah I dunno why he -Wd it | 13:57 |
dansmith | it looks right to me based on the nova one | 13:58 |
dansmith | we'll see what he says | 13:59 |
pdeore | yeah, even the glance-tempest-plugin patch also has -W, not sure why | 13:59 |
rosmaita | sorry, missing today's meeting :( | 14:07 |
abhishekk | np! | 14:07 |
*** pdeore is now known as pdeore|afk | 14:25 | |
abhishekk | dansmith, we need to tag m3 today | 14:36 |
abhishekk | gmann, is not around yet so I think he needs to follow the FFE process later | 14:36 |
abhishekk | that is what I heard from release team (we need to tag it today) | 14:37 |
dansmith | okay, the tc meeting is in 20 minutes, so I'm pretty sure he'll be around soon | 14:37 |
dansmith | it's a community goal, so hopefully it'll get your FFE blessing if needed :) | 14:37 |
abhishekk | cool, so we will wait for next 30 mins | 14:38 |
abhishekk | unfortunately I need to go out for couple of hours, but if he lifts -1 then you can go ahead and assume my +2 on the patch | 14:38 |
dansmith | ack | 14:39 |
abhishekk | thank you | 14:39 |
opendevreview | Ghanshyam proposed openstack/glance-tempest-plugin master: Update Image APIs tests to check system user is not permitted https://review.opendev.org/c/openstack/glance-tempest-plugin/+/855440 | 14:46 |
gmann | pdeore|afk: abhishek dansmith yeah I am little late on pushing those RBAC changes, but I was working on the glance tempest plugin test updates ^^ | 14:46 |
gmann | hope it will pass this time | 14:46 |
dansmith | gmann: ack, so if passes, we should get into m3? | 14:46 |
gmann | is there option if we can get review and merge it before m-3 tag? I am ok for FFE request too | 14:46 |
gmann | dansmith: yeah, I was holding to ping you for that. | 14:47 |
gmann | once rbac job is passing I will ping you | 14:48 |
dansmith | gmann: okay | 14:48 |
*** pdeore|afk is now known as pdeore | 15:59 | |
opendevreview | Ghanshyam proposed openstack/glance-tempest-plugin master: Update Image APIs tests to check system user is not permitted https://review.opendev.org/c/openstack/glance-tempest-plugin/+/855440 | 16:02 |
dansmith | gmann: tests are passing now, good to drop -W? | 16:56 |
gmann | dansmith: done. please check both https://review.opendev.org/c/openstack/glance/+/855435 https://review.opendev.org/c/openstack/glance-tempest-plugin/+/855440 | 16:58 |
dansmith | gmann: all set | 17:02 |
gmann | dansmith: thanks | 17:02 |
gmann | dansmith: can you check this too, making consistent use of release number in specs dir - https://review.opendev.org/c/openstack/glance-specs/+/855409 | 17:03 |
dansmith | oh yeah, but pdeore should probably give the first ack on that | 17:06 |
gmann | ok | 17:10 |
pdeore | gmann, done | 17:25 |
* pdeore will be back in an hr | 17:25 | |
*** pdeore is now known as pdeore|afk | 17:25 | |
gmann | pdeore|afk: thanks, dansmith ^^ | 17:25 |
dansmith | gmann: ah I guess abhi will have to merge that, only PTL can in glance, but I +2d | 17:27 |
gmann | sure. | 17:27 |
gmann | thanks | 17:27 |
opendevreview | Merged openstack/glance-specs master: Using release number as primary identifier https://review.opendev.org/c/openstack/glance-specs/+/855409 | 17:49 |
opendevreview | Merged openstack/glance master: RBAC updates: drop system scope https://review.opendev.org/c/openstack/glance/+/855435 | 19:15 |
opendevreview | Merged openstack/glance-tempest-plugin master: Update Image APIs tests to check system user is not permitted https://review.opendev.org/c/openstack/glance-tempest-plugin/+/855440 | 20:21 |
dansmith | gmann: rbac patch merged | 20:31 |
dansmith | I thought abhi was coming back to do the release, but I doubt it at this point | 20:31 |
dansmith | aha, pdeore|afk got it :) | 20:32 |
gmann | dansmith: yeah its in https://review.opendev.org/c/openstack/releases/+/855480 | 20:55 |
dansmith | yup | 20:55 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!