14:00:22 <pranali> #startmeeting glance
14:00:23 <opendevmeet> Meeting started Thu Feb 22 14:00:22 2024 UTC and is due to finish in 60 minutes.  The chair is pranali. Information about MeetBot at http://wiki.debian.org/MeetBot.
14:00:23 <opendevmeet> Useful Commands: #action #agreed #help #info #idea #link #topic #startvote.
14:00:23 <opendevmeet> The meeting name has been set to 'glance'
14:00:23 <pranali> #topic roll call
14:00:23 <pranali> #link https://etherpad.openstack.org/p/glance-team-meeting-agenda
14:00:29 <mklejn> o/
14:00:31 <pranali> o/
14:00:40 <mrjoshi> o/
14:01:12 <pranali> let's start, abhishek will not be able to join today
14:01:24 <pranali> #topic release/periodic jobs updates
14:01:49 <pranali> As we all know, M3 is in next week, so we need reviews on priority for Centralized cache DB & New location API changes
14:02:09 <pranali> and considering the random gate failures we might need to be prepare for FFE for the same if all the patches are approved till next Wednesday
14:02:36 <pranali> glance_store 4.7.0 release patch is submitted #link https://review.opendev.org/c/openstack/releases/+/909436
14:03:00 <pranali> Periodic jobs are all green except intermittent TIME_OUTs on fips jobs
14:03:11 <pranali> moving to next topic
14:03:13 <pranali> #topic Important Reviews
14:03:24 <pranali> Centralized cache DB - #link https://review.opendev.org/q/topic:%22centralized-cache-db%22
14:03:24 <pranali> New Location APIS -  #link https://review.opendev.org/q/topic:%22New-Location-Apis%22
14:03:41 <pranali> Kindly please review these patches on priority so that we can get those in before m3
14:04:08 <pranali> For new location apis, patches are updated by fixing the review comments and nova & cinder side PoCs are working as expected
14:04:48 <pranali> whoami-rajat,  has tested this from cinder side and it's working for New location api, there were issue with existing old location api reagrding incompatible location url which is also fixed in this patch
14:04:54 <pranali> #link https://review.opendev.org/c/openstack/glance/+/906359/8
14:05:37 <pranali> dansmith, we need your vote as well on these new location api patches, I have addressed all the suggestion given by you on location-import change
14:06:42 <pranali> that's it from me
14:06:51 <pranali> moving to next
14:06:53 <pranali> #topic Shelved instance image protection
14:06:58 <pranali> #link https://review.opendev.org/q/topic:%22shelved-image-protection%22
14:07:21 <pranali> There is a change proposed to add protection for removing images of shelved instances along with some nova side changes.
14:07:35 <croelandt> I'm a bit worried about that
14:07:40 <pranali> As abhishekk suggested I think we need a design spec first for this enhancement
14:07:46 <croelandt> It adds a config option, which is not "a simple fix"
14:08:03 <croelandt> and do we know whether they expect some kind of backport? Is it a feature or a bug in their mind?
14:08:04 <pranali> yeah right
14:08:25 <croelandt> especially since this config option must then be handled by ooo/NG
14:08:38 <croelandt> so yeah can they discuss it with us at PTG?
14:09:34 <pranali> I think mklejn is here
14:09:48 <mklejn> croelandt: I can join any discussion necessary. Please bare in mind that it's my first contribution, so I'm slightly lost
14:10:01 <croelandt> mklejn: oh ok :)
14:10:13 <croelandt> mklejn: so I don't think the patch is bad, at first glance (pun intended) it looks fine
14:10:34 <croelandt> but the way we do things, there is a lot of brueaucracy (sigh) and also real concerns about backporting & deployment
14:10:56 <croelandt> We have a meeting between April 8 - 12 that you could join
14:11:04 <croelandt> pranali: do we have an Etherpad with a list of topics already?
14:11:41 <pranali> croelandt, no not yet :) once it's ready I will make to add this topic there
14:11:53 <mklejn> croelandt: I've also made integration tests for this, but I didn't know which tempest is the right place to push it :)
14:11:56 <croelandt> ok :)
14:12:09 <croelandt> mklejn: how urgent would you say this is? Can this wait for the next cycle?
14:13:13 <mklejn> croelandt: yes, it can. don't see a problem with that
14:14:03 <croelandt> ok so probably we should discuss it during PTG in April and merge it during the next cycle
14:14:31 <pranali> mklejn, cool.. we are already at the end of this current cycle so we can discuss this during PTG and have it in next cycle along with spce
14:15:16 <mklejn> croelandt, pranali: sure, I'll join PTG then
14:15:35 <pranali> mklejn, gr8
14:15:44 <pranali> let's move to open discussions then
14:15:46 <pranali> #topic Open Discussion
14:15:51 <mklejn> and I'll do the spec in the meantime. about tempest: should I push tests to tempest repo or glance-tempest-plugin?
14:17:07 <croelandt> Ideally I'd say the plugin
14:17:08 <pranali> I think tempest would be ok, or you can add scenario test in the plugin
14:17:32 <mklejn> plugin it is then. thanks
14:18:53 <pranali> croelandt, could you please have a look at the location api patches again ? I have updated few things today
14:19:37 <croelandt> yeah I remember I had a few comments
14:19:41 <croelandt> I'll go throug them again
14:20:07 <pranali> tomorrow is Recharge Day, so I will submit the release related patches like, releasenotes & config refresh on monday
14:20:15 <pranali> croelandt, Thanks !
14:20:32 <pranali> so that's it from me, anyone has anything else to discuss ?
14:20:37 <croelandt> Can't wait to be on Monday to review that!
14:21:11 <pranali> :)
14:22:15 <pranali> shall we conclude for the day then ?
14:22:20 <croelandt> fine with me
14:22:24 <pranali> cool
14:22:39 <pranali> Thanks everyone for joining !!
14:22:50 <pranali> #endmeeting