14:00:19 <mrjoshi> #startmeeting glance
14:00:19 <opendevmeet> Meeting started Thu Sep 12 14:00:19 2024 UTC and is due to finish in 60 minutes.  The chair is mrjoshi. Information about MeetBot at http://wiki.debian.org/MeetBot.
14:00:19 <opendevmeet> Useful Commands: #action #agreed #help #info #idea #link #topic #startvote.
14:00:19 <opendevmeet> The meeting name has been set to 'glance'
14:00:25 <mrjoshi> #topic roll call
14:00:25 <mrjoshi> #link https://etherpad.openstack.org/p/glance-team-meeting-agenda
14:00:27 <mrjoshi> o/
14:01:22 <dansmith> o/
14:01:49 <mrjoshi> PTL is not around today
14:02:47 <mrjoshi> let's wait for few min for other's to join
14:05:05 <mrjoshi> let's get started
14:05:13 <mrjoshi> #topic 2025.1/Epoxy PTG
14:05:42 <mrjoshi> As everyone knows 2025.1 virtual PTG is during October 21 - 25,
14:06:33 <mrjoshi> I would request you to please add the topics to the planning etherpad that you would like to discuss during the PTG
14:06:35 <mrjoshi> Glance 2025.1 PTG planning : https://etherpad.opendev.org/p/2025.1-ptg-glance-planning
14:07:07 <mrjoshi> moving to next
14:07:15 <mrjoshi> #topic release/periodic job updates
14:07:42 <mrjoshi> we have rc1 this week
14:07:44 <mrjoshi> https://review.opendev.org/c/openstack/releases/+/928526
14:08:08 <mrjoshi> Periodic jobs are all green
14:08:53 <mrjoshi> that's it from me, moving to open discussion
14:09:05 <mrjoshi> #topic Open Discussion
14:09:25 <mrjoshi> Are bot patches OK to ninja-approve?
14:09:33 <croelandt> I think so, but I wanted to confirm :)
14:10:16 <croelandt> I'm gonna say yes :)
14:10:40 <croelandt> and same for the next question, I think we'll give up on supporting UPPER_CONSTRAINTS since the bot keeps erasing it
14:10:53 <croelandt> and no one really cares apparently
14:11:21 <mrjoshi> I think release bot patches gets approved by release team and other bot patches glance cores can ninja approve
14:12:45 <croelandt> so the release team has the permissions to merge in glance?
14:14:40 <abhishekk> dansmith: are we going to merge glance --> oslo patch?
14:15:10 <dansmith> idk, it failed but not for the problem we were searching for right?
14:15:38 <abhishekk> yeah, that is a strange problem
14:15:39 <dansmith> I haven't had a chance to look at the fail yet this morning
14:16:25 <dansmith> but yeah up to you I guess. frustrating that we haven't repro'd the problem when it happened several times in a row earlier
14:16:41 <dansmith> I don't want to merge it and find the gate wedged, but it would only be the import jobs
14:17:42 <abhishekk> so since stable already cut, we need to backport it if we merge now
14:17:57 <dansmith> rc1 has cut?
14:18:06 <abhishekk> https://review.opendev.org/c/openstack/releases/+/928526
14:18:20 <dansmith> whoa.. man time is flying, okay
14:18:40 <dansmith> I guess that means we're toast
14:18:59 <abhishekk> yeah :/ I was also not aware about that
14:19:49 <abhishekk> also you have added recheck on wrong patch
14:20:11 <abhishekk> you need that recheck on retry patch and need to remove -w from format inspector patch
14:20:34 <dansmith> oh sorry yeah I had them both open
14:20:53 <abhishekk> np!
14:21:13 <abhishekk> We will tag rc2 and make if final rc
14:21:35 <dansmith> so, there is a "bug but really a feature" that glance will not have fixed as a result
14:21:44 <dansmith> not likely to matter much, but just for visibility,
14:22:00 <dansmith> https://bugs.launchpad.net/nova/+bug/2073413
14:22:10 <abhishekk> looking
14:22:19 <dansmith> this will affect glance too on import, if someone uses the very old (10+ year) original qcow2 format we won't consider it safe
14:22:30 <dansmith> it's fixed in oslo, but just FYI
14:22:39 <dansmith> that format is deprecated and recommended not to be used by qemu team anymore,
14:22:48 <dansmith> but there are lots of old images in the wild
14:23:10 <dansmith> it's trivial to pre-convert images to the new format as a workaround but just so you know in case someone complains
14:24:01 <abhishekk> hmm, thanks for the heads up
14:24:28 <abhishekk> So now we need a separate stable only fix for glance
14:24:57 <abhishekk> OR merged oslo-format-inspector and backport it?
14:25:03 <dansmith> no,
14:25:17 <dansmith> I think we don't fix it unless someone asks, I'm just heads-upping for visibility
14:25:25 <dansmith> nova specifically didn't fix it until the move to oslo
14:25:43 <dansmith> but we did have a downstream customer complain about it so we're backporting that change downstream-only
14:25:59 <abhishekk> Ok, I will make a note of it, yeah we need to do that as well
14:26:03 <dansmith> I'm specifically not fixing it in nova stable because it's more a feature than a bug (it's a grey area really)
14:26:11 <dansmith> since for glance it's only import, it's much less concerning I think
14:26:43 <abhishekk> ack, @croelandt I think we need to discuss this internally
14:27:20 <abhishekk> between, @dansmith thank you for your time and efforts around CVE and format inspector work
14:27:42 <dansmith> o7
14:28:05 <abhishekk> o7
14:28:06 <abhishekk> @mrjoshi you can move forward
14:28:19 <mrjoshi> cool
14:28:21 <mrjoshi> Anyone has anything else to highlight?
14:29:13 <abhishekk> yeah, I will be on PTO till October 10, starting from coming Tuesday
14:29:21 <dansmith> *gasp*
14:29:31 <abhishekk> but you can reach me on slack or mail
14:30:22 <mrjoshi> ack
14:30:24 <abhishekk> I will be able to reply post 1400 UTC during week
14:30:45 <croelandt> maybe enjoy the PTO :)
14:31:03 <abhishekk> I will, thank you :D
14:33:14 <mrjoshi> shall we wrap up, if there's nothing more to highlight?
14:33:32 <abhishekk> yeah,
14:33:47 <abhishekk> Bots patches can be ninja approved by core
14:34:03 * abhishekk nothing more to highlight
14:34:22 <mrjoshi> ack
14:34:25 <mrjoshi> Thanks everyone for joining!!
14:34:32 <croelandt> thanks!
14:34:32 <croelandt> ++
14:34:43 <mrjoshi> #endmeeting