14:00:06 <pdeore> #startmeeting glance
14:00:06 <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:06 <opendevmeet> Useful Commands: #action #agreed #help #info #idea #link #topic #startvote.
14:00:06 <opendevmeet> The meeting name has been set to 'glance'
14:00:06 <pdeore> #topic roll call
14:00:06 <pdeore> #link https://etherpad.openstack.org/p/glance-team-meeting-agenda
14:00:13 <pdeore> o/
14:00:17 <abhishekk> o/
14:00:18 <dansmith> o/
14:00:25 <jokke_> o/
14:00:28 <mrjoshi> o/
14:00:35 <croelandt> o/
14:00:54 <rosmaita> o/
14:01:02 <pdeore> everyone is here .. let's start :)
14:01:12 <pdeore> #topic release/periodic jobs
14:01:24 <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:37 <pdeore> but we are going to release glance-store for sure
14:02:03 <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:09 <pdeore> #link https://review.opendev.org/c/openstack/oslo.config/+/840592
14:02:22 <pdeore> the glance side fixes are submitted for the same
14:02:27 <pdeore> #link https://review.opendev.org/c/openstack/glance/+/841350
14:02:28 <pdeore> #link https://review.opendev.org/c/openstack/openstack-zuul-jobs/+/841368
14:02:38 <pdeore> Kindly please have a look
14:02:47 <abhishekk> former patch is dependent on later one
14:03:03 <abhishekk> so be in touch with infra core team for the same
14:03:12 <pdeore> ack
14:03:34 <pdeore> moving ahead
14:03:42 <pdeore> #topic Spec repo maintenance
14:03:52 <pdeore> not sure who added this topic :)
14:04:00 <pdeore> croelandt, is that you?
14:05:09 <abhishekk> I have approved 1st 3 specs
14:05:52 <abhishekk> will visit 2.2 after the meeting and take appropriate actions
14:06:11 <pdeore> ack
14:06:20 <pdeore> let's move ahead
14:06:29 <pdeore> #topic Specs For Review
14:06:30 <croelandt> pdeore: no
14:06:46 <pdeore> Most of the specs are merged, now just need some attention at
14:06:52 <abhishekk> might be rosmaita have added
14:06:53 <pdeore> Expanding store details - https://review.opendev.org/c/openstack/glance-specs/+/835606 (two +2)
14:06:53 <pdeore> Update proposal for duplicate image download - https://review.opendev.org/c/openstack/glance-specs/+/734683
14:07:04 <pdeore> croelandt, ohh i thought it's you :)
14:07:36 <pdeore> kindly have a look at these specs ^
14:07:48 <abhishekk> ack
14:07:58 <pdeore> moving ahead
14:08:02 <pdeore> #topic Missing specs
14:08:25 <pdeore> Need spec for Inject metadata for upload - https://review.opendev.org/c/openstack/glance/+/835462
14:08:57 <abhishekk> dansmith, have you got any response from tobias (hope this is right pronunciation)
14:09:20 <abhishekk> Or should I go ahead a write a spec for it?
14:09:47 <dansmith> abhishekk: nope haven't heard, but we should ping him
14:09:47 <abhishekk> dansmith, is buys with infra team atm, I will ping him later
14:09:58 <abhishekk> where can we find him :D
14:10:28 <abhishekk> he is present in cinder channel, I will ping him later
14:10:33 <dansmith> ack
14:11:01 <pdeore> ok, moving ahead
14:11:07 <pdeore> #topic Secure RBAC
14:11:25 <pdeore> We have discussed the glance side queries regarding system scope implemetation in last policy popup meeting
14:11:47 <rosmaita> (sorry, i am stuck running another meeting)
14:11:51 <pdeore> kindly have a look for more details, #link https://etherpad.opendev.org/p/glance-rbac-system-scope
14:11:58 <abhishekk> ok, anything specific to share?
14:12:05 <abhishekk> share/highlight?
14:12:55 <pdeore> yeah, regarding owner issue for system scope, how we should handle system scope for all create/update metadef apis?
14:13:24 <dansmith> we can already create those with a specific owner right?
14:13:25 <pdeore> abhishekk, should i explain the details from etherpad?
14:13:33 <dansmith> like, the admin can create them on behalf of another yser?
14:13:39 <abhishekk> yes you can pass --owner as input
14:14:20 <dansmith> yeah, so making those APIs system-scope and always requiring --owner is one option
14:14:24 <abhishekk> but we need to add the validation if it is missing from the request body
14:14:29 <dansmith> since they're already admin-only anyway
14:14:49 <abhishekk> yeah, I have suggested that option as well
14:15:25 <pdeore> ok, and should keep those for both system & project admin? or just system admin?
14:15:31 <abhishekk> this is more feasible I guess
14:15:39 <pdeore> yeah it seems so
14:15:43 <abhishekk> just for sysadmin
14:15:47 <dansmith> pdeore: we should not have it under both scopes
14:15:49 <pdeore> ok
14:15:57 <dansmith> unless absolutely necessary, and this is not a suitable case I think
14:16:04 <abhishekk> agree
14:16:08 <pdeore> ohh ok
14:16:14 <dansmith> because what we want is to say "only operators should call these apis"
14:16:15 <abhishekk> pdeore, I think you can write one spec for this change
14:16:32 <pdeore> abhishekk, sure I will do that
14:17:02 <abhishekk> cool, that will help us to finalize the approach before actually start working on it
14:17:13 <pdeore> yeah right!
14:17:59 <pdeore> ok, let's move ahead
14:18:04 <pdeore> #topic glance-download import method
14:18:12 <pdeore> spec : https://review.opendev.org/c/openstack/glance-specs/+/836132 ( few minor comments)
14:18:37 <abhishekk> I don't think the owner of the spec is present today for the discussion
14:18:38 <dansmith> merge and allow follow-up?
14:18:51 <abhishekk> I think that is good option
14:19:21 <abhishekk> will try to find them till tomorrow else will allow them to modify it as a followup
14:19:43 <pdeore> ack
14:20:07 <pdeore> ok, let's move to Open Discussion
14:20:09 <pdeore> #topic Open Discussion
14:20:41 <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:50 <pdeore> https://review.opendev.org/c/openstack/glance-tempest-plugin/+/802792/20
14:20:50 <pdeore> https://review.opendev.org/c/openstack/glance-tempest-plugin/+/802794/15
14:20:50 <pdeore> https://review.opendev.org/c/openstack/glance-tempest-plugin/+/802795
14:20:50 <pdeore> https://review.opendev.org/c/openstack/glance-tempest-plugin/+/802795
14:21:08 <dansmith> ack, sorry
14:21:18 <abhishekk> alistarle, is here
14:21:19 <pdeore> alistarle, is here :)
14:21:57 <abhishekk> alistarle, there are some small suggestions on glance-download spec, once you fix it we can merge it right away
14:22:13 <pdeore> dansmith, np :) but I think we should get those merge in this milestone at least :)
14:22:43 <alistarle> Hello there, oh nice, we will fix that then
14:23:17 <abhishekk> cool, thank you
14:23:44 <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:57 <abhishekk> I have also shared coding standards we follow with your colleague
14:24:49 <abhishekk> jokke_, ack, noted
14:25:02 <jokke_> And ^^^ is not to point fingers, just reminding everyone to be careful with these things
14:25:21 <abhishekk> yep
14:25:29 <pdeore> jokke_, sure
14:25:44 <abhishekk> pslestang, do you have any questions related to comments on your spec?
14:26:23 <abhishekk> Once you submit the new patch for spec, we will get it merged quickly as everything is sorted out
14:27:16 <pdeore> so that's is from me, anyone has anything else to discuss or we should wrap up?
14:27:30 <abhishekk> nothing from me as well
14:27:42 <jokke_> One quick question
14:28:08 <pslestang> abhishekk: nope it should be good
14:28:18 <abhishekk> pslestang, cool
14:28:26 <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:42 <abhishekk> timeline is M2
14:29:11 <abhishekk> you can post your comment on the spec and I can invite him for discussion during our next meeting
14:29:57 <abhishekk> no spec will be approved post M2
14:30:03 <jokke_> Sounds good ... I'll try to remember put my notes in the review tomorrow so there is next week to preo
14:30:06 <jokke_> prep
14:30:07 <abhishekk> M2 --> Milestone 2
14:30:20 <abhishekk> sounds good
14:31:41 <jokke_> that's all from me for now, thanks
14:31:56 <pdeore> cool, let's wrap up for today!
14:32:07 <abhishekk> thank you!
14:32:07 <pdeore> Thanks everyone for joining !!
14:32:19 <abhishekk> have a nice weekend o/~
14:32:20 <pdeore> #endmeeting