14:01:14 <croelandt> #startmeeting glance
14:01:14 <opendevmeet> Meeting started Thu May 15 14:01:14 2025 UTC and is due to finish in 60 minutes.  The chair is croelandt. Information about MeetBot at http://wiki.debian.org/MeetBot.
14:01:14 <opendevmeet> Useful Commands: #action #agreed #help #info #idea #link #topic #startvote.
14:01:14 <opendevmeet> The meeting name has been set to 'glance'
14:01:23 <croelandt> #topic roll call
14:01:25 <abhishekk> o/
14:01:27 <croelandt> #link https://etherpad.openstack.org/p/glance-team-meeting-agenda
14:01:39 <croelandt> It's probably juste going to be the two of us :)
14:02:17 <croelandt> well, the *3* of us  \o/
14:02:22 <croelandt> #topic Release/periodic job updates
14:02:35 <croelandt> I only see a failure for glance-multistore-cinder-import-fips
14:02:54 <abhishekk> that is related to dependency between taskflow and one other library
14:03:09 <croelandt> ok I see the Python version for Nova is not the right one
14:03:19 <croelandt> so it's out of our hands?
14:03:37 <abhishekk> I think so, we can wait and keep watch on it
14:03:50 <croelandt> yes
14:03:57 <croelandt> #topic Important stable patches
14:04:05 <croelandt> #link https://review.opendev.org/c/openstack/glance/+/948810
14:04:07 <croelandt> #link https://review.opendev.org/c/openstack/glance/+/948812
14:04:22 <croelandt> So these are backports of Dan's fix for a bug that was holding the gate in master
14:04:28 <croelandt> the bug is also present in stable/2024.*
14:04:47 <croelandt> I think we should backport the fix so we can easily merge backports in stable/2024.* (that is, without hitting "recheck" 12 times)
14:04:51 <croelandt> any thoughts?
14:04:55 <abhishekk> sounds good
14:05:01 <abhishekk> I will ninja approve it one by one
14:05:06 <croelandt> ok, thanks a lot!
14:05:19 <abhishekk> np!
14:05:20 <croelandt> #topic glance-gaps patches
14:05:26 <mrjoshi> o/
14:05:38 <croelandt> I will take over the SDK/CLI patches next week since mrjoshi is going to leave us :-(
14:05:53 <croelandt> mrjoshi: anything we need to know regarding the work that is left to be done? Anything important you want to highlight?
14:06:56 <abhishekk> croelandt: all the best ;)
14:07:22 <mrjoshi> Thank you so croelandt , I have addressed all the comments and the patches are up, they are failing on zuul jobs, intermittent failure.
14:07:46 <abhishekk> are those related to glance or your patches?
14:09:09 <mrjoshi> No it's not related to my patch
14:09:27 <mrjoshi> it's been occuring in other sdk patches aswell
14:09:37 <croelandt> ok so that's fine :)
14:10:28 <abhishekk> cool
14:10:29 <croelandt> Well thanks a lot for all your work on the SDK/CLI, we made huge progress thanks to you
14:10:46 <abhishekk> ++, thank you for all your efforts mrjoshi!!
14:11:07 <croelandt> #topic Issues with secure hash
14:11:16 <croelandt> #link https://review.opendev.org/c/openstack/glance/+/949672
14:11:22 <croelandt> #link https://review.opendev.org/c/openstack/glance/+/949668
14:11:30 <croelandt> so I'm a bit out of the loop here
14:11:39 <croelandt> First of all, I think the doc patch should be merged with the code patch
14:11:48 <croelandt> so e don't risk having inaccurate docs
14:12:09 <croelandt> and then, I think we want to decide whether it's ok to rely on the "old" behaviour like some Tempest do
14:12:12 <croelandt> am I right?
14:12:13 <abhishekk> First of all, is it a good idea to flip the config value? rosmaita wdyt?
14:13:11 <abhishekk> This feature is still in early phase, and I am sure no one is still using it (talking about hash calculation in new location API)
14:14:08 <croelandt> :/
14:14:13 <abhishekk> We have enabled it by default, so that all the images/snapshot etc will have hash/checksum calculated
14:14:27 <croelandt> Maybe Tempest could use the right config value instead of us changing the default, right?
14:15:17 <abhishekk> that is why I have disabled it in the job, we don't need to calculate the hashes in tests IMO
14:16:07 <abhishekk> But, we need to disable it for all the jobs which are using ceph as backend
14:16:40 <abhishekk> Right now I have disabled it only for glance_store job, but we need to do it for nova-ceph-multistore job as well
14:16:55 <croelandt> so we'd drop Rajat's patches?
14:17:12 <abhishekk> We definitely need some input whether it is really good to flip the config value
14:18:02 <abhishekk> may be dansmith, rosmaita can comment here about the same
14:18:05 <croelandt> yeah I think it would solve our testing issues, but it might be worse for real users
14:18:35 <croelandt> ok so I think we need to get Dan, Brian, Rajat and the two of us in a 15 minute IRC meeting at some point
14:18:43 <croelandt> and figure this out
14:18:46 <abhishekk> ALSO you are the boss now, so whatever you say!!!
14:18:58 <dansmith> I haven't looked at this so I need time to digest
14:19:01 <croelandt> If I had it my way, we'd remove all features
14:19:13 <abhishekk> bang!!
14:19:21 <croelandt> I don't think it's urgent, so maybe we can try and find a time slot next week maybe?
14:19:28 <abhishekk> dansmith: ack
14:19:34 <croelandt> unless this gets resolved in Gerrit comments by then :)
14:20:58 <croelandt> ok moving on, then
14:21:08 <croelandt> #topic One easy patch per core dev
14:21:16 <croelandt> #link https://review.opendev.org/q/topic:%22bug/2110185%22
14:21:35 <croelandt> ok so I guess we'd want to have all these patches merged in the same release, so the behaviour is consistent across all drivers, right?
14:22:32 <abhishekk> yes, we need it for our main feature, pass image size in advance to glance
14:23:01 <croelandt> ok so I'll start reviewing those but I'll only push +W once they all seem good
14:23:09 <croelandt> ideally we would have them all in M2
14:23:37 <abhishekk> sounds good, we also need a release for glance store after that, so it will be good if we get it early as well
14:23:44 <Luzi> o/
14:24:22 <abhishekk> Luzi: hi
14:24:47 <croelandt> yes
14:24:53 <abhishekk> croelandt: I have approved your patch related to unused method
14:24:54 <croelandt> by the way we have a release for glacne_store M1
14:25:05 <croelandt> is there anything else M1-related I need to do as PTL?
14:25:20 <abhishekk> cool! I think that is enough
14:25:38 <croelandt> nice
14:25:46 <croelandt> #topic Open Discussion
14:25:53 <croelandt> Anything else to discuss?
14:25:58 <Luzi> hi
14:26:17 <Luzi> we do have some time to work on the image encryption again starting next week
14:26:28 <abhishekk> great!
14:27:14 <abhishekk> we are here if you guys need anything, just ping us!
14:27:29 <croelandt> yes don't hesitate to ping on #openstack-glance
14:27:46 <Luzi> were there any other points except for the copy image and glance_direct still missing in the patch?
14:28:23 <Luzi> i think markus answered a few open review questions - it would be good if someone could get over the patch again
14:28:39 <abhishekk> I think those are important, I will again have a look tomorrow and comment if something else is missing!
14:28:47 <Luzi> thank you :)
14:28:51 <abhishekk> np!!
14:29:22 <croelandt> ok, I think we can wrap up
14:31:10 <abhishekk> thank you!!
14:31:15 <croelandt> #endmeeting