14:00:15 <croelandt> #startmeeting glance
14:00:15 <opendevmeet> Meeting started Thu Nov 27 14:00:15 2025 UTC and is due to finish in 60 minutes.  The chair is croelandt. Information about MeetBot at http://wiki.debian.org/MeetBot.
14:00:15 <opendevmeet> Useful Commands: #action #agreed #help #info #idea #link #topic #startvote.
14:00:15 <opendevmeet> The meeting name has been set to 'glance'
14:00:18 <croelandt> #topic roll call
14:00:20 <croelandt> o/
14:02:23 <croelandt> abhishekk: ?
14:02:30 <abhishekk> o/
14:02:32 <croelandt> rosmaita: are you unable to join?
14:02:53 <croelandt> #topic Release/periodic job updates
14:03:02 <croelandt> All good, there was a failure 2 weeks ago but it was solved
14:03:10 <croelandt> #topic mportant stable patches - http://tiny.cc/glance-maintained
14:03:35 <croelandt> Some patches by Bence Romsic, the tests fail because they require another patch to be backported. I'll have to discuss taht with the author
14:03:44 <croelandt> #topic Glance download image from specific store
14:03:50 <croelandt> #link https://review.opendev.org/c/openstack/glance-specs/+/963239
14:04:02 <croelandt> so yeah, using "oneof" was not great
14:04:19 <abhishekk> yeah we need to finalise the params so that I can proceed
14:05:09 <croelandt> &stores=ceph,s3&store-selection=[require/prefer] sounds good to me
14:05:18 <croelandt> rosmaita: if you want to weigh in on that later ^
14:05:19 <croelandt> dansmith: ^
14:06:05 <abhishekk> OK, I will wait for tonight and then go ahead with ^ if there are no objections?
14:06:29 <croelandt> sounds good
14:06:45 <croelandt> #topic  Decompression plugin
14:06:47 <abhishekk> I think both are not available today due to US holiday
14:06:50 <croelandt> abhishekk: you have the floor :)
14:06:51 <croelandt> oh yeah
14:06:55 <croelandt> ok so maybe wait for Monday
14:07:04 <croelandt> tomorrow is Black Friday :D
14:07:07 <abhishekk> yeah
14:07:16 <croelandt> ok let's move on to decompression plugin
14:07:20 <croelandt> can you explain the issue?
14:07:36 <abhishekk> Yeah
14:07:45 <abhishekk> there are 3 issues which are dependent
14:08:08 <abhishekk> We support decompression for gzip, lha and zip
14:08:39 <abhishekk> out of which gzip works with web-download which does not use stage api internally
14:09:03 <abhishekk> but gzip with glance-direct fails as later it compares size and mismatches
14:09:21 <abhishekk> then lha and zip has different failures
14:09:32 <abhishekk> So all 3 are valid issues and we need those
14:10:19 <croelandt> ok but they are unrelated? As in, we can fix them one by one?
14:11:19 <abhishekk> the size mismatch fix needs to be land first
14:11:40 <abhishekk> https://review.opendev.org/c/openstack/glance/+/968351 | Update image size after image decompression (968351) · Gerrit Code Review
14:11:47 <abhishekk> this one needs to go 1st
14:11:55 <abhishekk> then LHA and then zip
14:12:14 <croelandt> I see
14:12:20 <croelandt> glad to see that no one uses this plugin :D
14:12:46 <abhishekk> The first one is required because other two will fail without it
14:12:51 <croelandt> or at least no one is complaining
14:13:08 <abhishekk> May be they are using gzip with web-download :D
14:13:20 <croelandt> maybe
14:13:23 <croelandt> #topic One easy patch per core dev
14:13:29 <croelandt> #link https://review.opendev.org/c/openstack/glance/+/939684
14:13:35 <croelandt> yes we should definitely review this
14:13:40 * croelandt is guilty of not reviewing this
14:13:42 <abhishekk> its not easy though :D
14:13:45 <croelandt> haha
14:13:46 <croelandt> you lied!
14:13:52 <croelandt> will take a look after the meeting
14:13:57 <croelandt> #topic Open Discussion
14:13:59 <abhishekk> thank you!!
14:14:01 <croelandt> Anything else?
14:14:17 <rajiv_> Hi
14:14:30 <abhishekk> Yeah, image encryption
14:14:33 <croelandt> rajiv_: hello
14:14:42 <abhishekk> I think we need to give priority to that
14:15:17 <croelandt> abhishekk: the issue is that it is OK from the Glance side
14:15:18 <rajiv_> I am planning to upgrade Dalmatian to Flamingo, i see Flamingo runs on Ubuntu Noble, which supports only python 3.12, hence should we add python 3.12 here : https://opendev.org/openstack/glance/src/branch/stable/2025.2/tox.ini#L4
14:15:26 <croelandt> but  Nova/Cinder had more concerns
14:15:50 <croelandt> my take from PTG is that we need people from Nova/Cinder to commit to reviewing and helping Markus & Josephine
14:15:58 <abhishekk> ack
14:16:27 <abhishekk> Lets discuss this with nova and cinder next time then, I will try to bring rajat from cinder for this
14:16:32 <croelandt> abhishekk: good
14:16:58 <croelandt> rajiv_: so this is only what is listed tyhrough "tox -l"
14:17:06 <croelandt> you can still run tox -epy312
14:17:43 <rajiv_> yes, locally i was testing via -e py312, but was wondering why it was not added to envlist
14:18:00 <croelandt> honestly it's annoying to change all the time
14:18:08 <croelandt> and I don't personnally run tox -l
14:18:59 <rajiv_> ah ok, secondly, i dont see any change in alembic version, any other important changes i need to be aware of before upgrading :) i did refer the release notes
14:19:27 <croelandt> do pre-prod testing :D
14:19:36 <rajiv_> haha sure :)
14:19:46 <croelandt> I don't remember anything
14:19:52 <croelandt> everything important should be in the release notes
14:20:01 <abhishekk> No new db changes introduced since damaltian I thing, centralized databse for cache is pre damaltian i guess
14:20:20 <rajiv_> previously, when an image was uploaded, it used to pick visibility private but now its shared, is this expected ?
14:21:03 <croelandt> abhishekk: do you remember if this is a change we introduced?
14:21:26 <abhishekk> I don't remember we done this
14:23:04 <rajiv_> okay, lastly, regarding https://bugs.launchpad.net/glance/+bug/1916052 which i raised, was reverted, is comment. #16 valid ?
14:23:37 <abhishekk> This is what I found https://review.opendev.org/c/openstack/glance/+/422897/7/releasenotes/notes/image-visibility-changes-fa5aa18dc67244c4.yaml
14:24:47 <abhishekk> rajiv_: frankly no idea about this bug atm
14:24:54 <rajiv_> this is an important fix for us, since we upload very large images but have very less token validity, hence token renewal during uploads via trust was a saviour for us
14:25:13 <rajiv_> abhishekk: okay, can i follow up next week ?
14:25:41 <abhishekk> I am not sure I can manage to have look at it due to workload i am having
14:26:05 <rajiv_> oops ok
14:26:08 <croelandt> rajiv_: there was a fix and it was reverted?
14:26:20 <rajiv_> croelandt: yes
14:27:11 <croelandt> ok
14:27:16 <croelandt> so does #16 work for you?
14:28:16 <rajiv_> i havent tested it since its a critical change, idk if glance would run without keystone_auth
14:28:53 <croelandt> well
14:28:57 <croelandt> testing patches really helps :)
14:29:54 <rajiv_> sure, wanted to know if it was tested before i try!
14:30:00 <rajiv_> thanks, thats all it from my side
14:30:00 <croelandt> nope
14:30:11 <croelandt> we don't really test much on "real world scenarios" to be honest
14:30:21 <rajiv_> ah ok
14:30:32 <croelandt> so if users who report issues can say "yeah that patch fixes my issue" it's great
14:31:06 <croelandt> not sure about abhishekk, but I don't run OpenStack "for real", I just build it :)
14:31:24 <abhishekk> :D
14:31:31 <rajiv_> oh ok, croelandt can we try revert the reverted patch :D ?
14:31:41 <abhishekk> The mentioned patch in comment 16 has -1 from zuul
14:32:04 <abhishekk> rajiv_: there might be reason why that patch got reverted, do you know it?
14:32:33 <croelandt> I think rajiv_ is talking about the patch reverted by Erno
14:32:49 <croelandt> see #4
14:32:58 <abhishekk> Yeah but there should be reason for that
14:32:59 <abhishekk> ack
14:33:18 <rajiv_> The Swift driver not being able to use Trusts had nothing to do     about the store driver itself nor that keystoneauth1 would have     broken the feature, but rather it not having the functionality     in the first place and us not catching that on reviews.      We should figure out how to test this before we try to replace     this code again.
14:33:36 <rajiv_> that was the reason on the revert msg
14:34:44 <abhishekk> yeah, this needs careful eyes we can not just rerevert it
14:35:46 <rajiv_> okay
14:35:54 <croelandt> rajiv_: but if you got a lab where you can test #16, that helps :)
14:36:36 <rajiv_> sure, will update the thread accordingly
14:37:11 <croelandt> thanks!
14:37:15 <croelandt> I think we can end it for today
14:37:19 <croelandt> thanks everyone for joining
14:37:23 <croelandt> #endmeeting