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