| *** soniya is now known as soniya|ruck | 05:04 | |
| *** soniya29 is now known as soniya29|ruck | 11:08 | |
| *** soniya29|ruck is now known as soniya29|ruck|brb | 11:10 | |
| *** soniya is now known as soniya|ruck | 12:21 | |
| *** dasm|off is now known as dasm | 13:15 | |
| pdeore | #startmeeting glance | 14:00 |
|---|---|---|
| opendevmeet | Meeting started Thu May 12 14:00:06 2022 UTC and is due to finish in 60 minutes. The chair is pdeore. Information about MeetBot at http://wiki.debian.org/MeetBot. | 14:00 |
| opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 14:00 |
| opendevmeet | The meeting name has been set to 'glance' | 14:00 |
| pdeore | #topic roll call | 14:00 |
| pdeore | #link https://etherpad.openstack.org/p/glance-team-meeting-agenda | 14:00 |
| pdeore | o/ | 14:00 |
| abhishekk | o/ | 14:00 |
| dansmith | o/ | 14:00 |
| jokke_ | o/ | 14:00 |
| mrjoshi | o/ | 14:00 |
| croelandt | o/ | 14:00 |
| rosmaita | o/ | 14:00 |
| pdeore | everyone is here .. let's start :) | 14:01 |
| pdeore | #topic release/periodic jobs | 14:01 |
| pdeore | Milestone 1 release is just one week away and we will verify whether we are going to tag the M1 or not | 14:01 |
| pdeore | but we are going to release glance-store for sure | 14:01 |
| pdeore | Periodic jobs all green except POST_FAILURE for fips job and 2 oslo tips jobs failure for python3.6 because of dropping py36/37 support and updated to py38 in oslo.config | 14:02 |
| pdeore | #link https://review.opendev.org/c/openstack/oslo.config/+/840592 | 14:02 |
| pdeore | the glance side fixes are submitted for the same | 14:02 |
| pdeore | #link https://review.opendev.org/c/openstack/glance/+/841350 | 14:02 |
| pdeore | #link https://review.opendev.org/c/openstack/openstack-zuul-jobs/+/841368 | 14:02 |
| pdeore | Kindly please have a look | 14:02 |
| abhishekk | former patch is dependent on later one | 14:02 |
| abhishekk | so be in touch with infra core team for the same | 14:03 |
| pdeore | ack | 14:03 |
| pdeore | moving ahead | 14:03 |
| pdeore | #topic Spec repo maintenance | 14:03 |
| pdeore | not sure who added this topic :) | 14:03 |
| pdeore | croelandt, is that you? | 14:04 |
| abhishekk | I have approved 1st 3 specs | 14:05 |
| abhishekk | will visit 2.2 after the meeting and take appropriate actions | 14:05 |
| pdeore | ack | 14:06 |
| pdeore | let's move ahead | 14:06 |
| pdeore | #topic Specs For Review | 14:06 |
| croelandt | pdeore: no | 14:06 |
| pdeore | Most of the specs are merged, now just need some attention at | 14:06 |
| abhishekk | might be rosmaita have added | 14:06 |
| pdeore | Expanding store details - https://review.opendev.org/c/openstack/glance-specs/+/835606 (two +2) | 14:06 |
| pdeore | Update proposal for duplicate image download - https://review.opendev.org/c/openstack/glance-specs/+/734683 | 14:06 |
| pdeore | croelandt, ohh i thought it's you :) | 14:07 |
| pdeore | kindly have a look at these specs ^ | 14:07 |
| abhishekk | ack | 14:07 |
| pdeore | moving ahead | 14:07 |
| pdeore | #topic Missing specs | 14:08 |
| pdeore | Need spec for Inject metadata for upload - https://review.opendev.org/c/openstack/glance/+/835462 | 14:08 |
| abhishekk | dansmith, have you got any response from tobias (hope this is right pronunciation) | 14:08 |
| abhishekk | Or should I go ahead a write a spec for it? | 14:09 |
| dansmith | abhishekk: nope haven't heard, but we should ping him | 14:09 |
| abhishekk | dansmith, is buys with infra team atm, I will ping him later | 14:09 |
| abhishekk | where can we find him :D | 14:09 |
| abhishekk | he is present in cinder channel, I will ping him later | 14:10 |
| dansmith | ack | 14:10 |
| pdeore | ok, moving ahead | 14:11 |
| pdeore | #topic Secure RBAC | 14:11 |
| pdeore | We have discussed the glance side queries regarding system scope implemetation in last policy popup meeting | 14:11 |
| rosmaita | (sorry, i am stuck running another meeting) | 14:11 |
| pdeore | kindly have a look for more details, #link https://etherpad.opendev.org/p/glance-rbac-system-scope | 14:11 |
| abhishekk | ok, anything specific to share? | 14:11 |
| abhishekk | share/highlight? | 14:12 |
| pdeore | yeah, regarding owner issue for system scope, how we should handle system scope for all create/update metadef apis? | 14:12 |
| dansmith | we can already create those with a specific owner right? | 14:13 |
| pdeore | abhishekk, should i explain the details from etherpad? | 14:13 |
| dansmith | like, the admin can create them on behalf of another yser? | 14:13 |
| abhishekk | yes you can pass --owner as input | 14:13 |
| dansmith | yeah, so making those APIs system-scope and always requiring --owner is one option | 14:14 |
| abhishekk | but we need to add the validation if it is missing from the request body | 14:14 |
| dansmith | since they're already admin-only anyway | 14:14 |
| abhishekk | yeah, I have suggested that option as well | 14:14 |
| pdeore | ok, and should keep those for both system & project admin? or just system admin? | 14:15 |
| abhishekk | this is more feasible I guess | 14:15 |
| pdeore | yeah it seems so | 14:15 |
| abhishekk | just for sysadmin | 14:15 |
| dansmith | pdeore: we should not have it under both scopes | 14:15 |
| pdeore | ok | 14:15 |
| dansmith | unless absolutely necessary, and this is not a suitable case I think | 14:15 |
| abhishekk | agree | 14:16 |
| pdeore | ohh ok | 14:16 |
| dansmith | because what we want is to say "only operators should call these apis" | 14:16 |
| abhishekk | pdeore, I think you can write one spec for this change | 14:16 |
| pdeore | abhishekk, sure I will do that | 14:16 |
| abhishekk | cool, that will help us to finalize the approach before actually start working on it | 14:17 |
| pdeore | yeah right! | 14:17 |
| pdeore | ok, let's move ahead | 14:17 |
| pdeore | #topic glance-download import method | 14:18 |
| pdeore | spec : https://review.opendev.org/c/openstack/glance-specs/+/836132 ( few minor comments) | 14:18 |
| abhishekk | I don't think the owner of the spec is present today for the discussion | 14:18 |
| dansmith | merge and allow follow-up? | 14:18 |
| abhishekk | I think that is good option | 14:18 |
| abhishekk | will try to find them till tomorrow else will allow them to modify it as a followup | 14:19 |
| pdeore | ack | 14:19 |
| pdeore | ok, let's move to Open Discussion | 14:20 |
| pdeore | #topic Open Discussion | 14:20 |
| pdeore | dansmith, I have updated the glance-tempest plugin rbac patches with your suggetsion, kindly please have a look when you have time | 14:20 |
| pdeore | https://review.opendev.org/c/openstack/glance-tempest-plugin/+/802792/20 | 14:20 |
| pdeore | https://review.opendev.org/c/openstack/glance-tempest-plugin/+/802794/15 | 14:20 |
| pdeore | https://review.opendev.org/c/openstack/glance-tempest-plugin/+/802795 | 14:20 |
| pdeore | https://review.opendev.org/c/openstack/glance-tempest-plugin/+/802795 | 14:20 |
| dansmith | ack, sorry | 14:21 |
| abhishekk | alistarle, is here | 14:21 |
| pdeore | alistarle, is here :) | 14:21 |
| abhishekk | alistarle, there are some small suggestions on glance-download spec, once you fix it we can merge it right away | 14:21 |
| pdeore | dansmith, np :) but I think we should get those merge in this milestone at least :) | 14:22 |
| alistarle | Hello there, oh nice, we will fix that then | 14:22 |
| abhishekk | cool, thank you | 14:23 |
| jokke_ | Just a quick reminder. Changes like https://review.opendev.org/c/openstack/glance/+/840409 is API breaking, so please, be careful and lets try to get the responses right on the first time. We absolutely cannot break the stable branches backporting minor looking fixes like that | 14:23 |
| abhishekk | I have also shared coding standards we follow with your colleague | 14:23 |
| abhishekk | jokke_, ack, noted | 14:24 |
| jokke_ | And ^^^ is not to point fingers, just reminding everyone to be careful with these things | 14:25 |
| abhishekk | yep | 14:25 |
| pdeore | jokke_, sure | 14:25 |
| abhishekk | pslestang, do you have any questions related to comments on your spec? | 14:25 |
| abhishekk | Once you submit the new patch for spec, we will get it merged quickly as everything is sorted out | 14:26 |
| pdeore | so that's is from me, anyone has anything else to discuss or we should wrap up? | 14:27 |
| abhishekk | nothing from me as well | 14:27 |
| jokke_ | One quick question | 14:27 |
| pslestang | abhishekk: nope it should be good | 14:28 |
| abhishekk | pslestang, cool | 14:28 |
| jokke_ | There is spec proposal from Rajat about new locations APIs. Do we want to discuss that here or just keep the comments going on the review and what's the timeline for that? | 14:28 |
| abhishekk | timeline is M2 | 14:28 |
| abhishekk | you can post your comment on the spec and I can invite him for discussion during our next meeting | 14:29 |
| abhishekk | no spec will be approved post M2 | 14:29 |
| jokke_ | Sounds good ... I'll try to remember put my notes in the review tomorrow so there is next week to preo | 14:30 |
| jokke_ | prep | 14:30 |
| abhishekk | M2 --> Milestone 2 | 14:30 |
| abhishekk | sounds good | 14:30 |
| jokke_ | that's all from me for now, thanks | 14:31 |
| pdeore | cool, let's wrap up for today! | 14:31 |
| abhishekk | thank you! | 14:32 |
| pdeore | Thanks everyone for joining !! | 14:32 |
| abhishekk | have a nice weekend o/~ | 14:32 |
| pdeore | #endmeeting | 14:32 |
| opendevmeet | Meeting ended Thu May 12 14:32:20 2022 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 14:32 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/glance/2022/glance.2022-05-12-14.00.html | 14:32 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/glance/2022/glance.2022-05-12-14.00.txt | 14:32 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/glance/2022/glance.2022-05-12-14.00.log.html | 14:32 |
| *** soniya|ruck is now known as soniya|out | 16:11 | |
| *** dasm is now known as dasm|off | 21:11 | |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!