| *** mhen_ is now known as mhen | 02:27 | |
| opendevreview | Merged openstack/glance_store master: Skip cross-glance-tox-functional for irrelevant files https://review.opendev.org/c/openstack/glance_store/+/968030 | 06:44 |
|---|---|---|
| croelandt | #startmeeting glance | 14:00 |
| 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 |
| opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 14:00 |
| opendevmeet | The meeting name has been set to 'glance' | 14:00 |
| croelandt | #topic roll call | 14:00 |
| croelandt | o/ | 14:00 |
| croelandt | abhishekk: ? | 14:02 |
| abhishekk | o/ | 14:02 |
| croelandt | rosmaita: are you unable to join? | 14:02 |
| croelandt | #topic Release/periodic job updates | 14:02 |
| croelandt | All good, there was a failure 2 weeks ago but it was solved | 14:03 |
| croelandt | #topic mportant stable patches - http://tiny.cc/glance-maintained | 14:03 |
| 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 |
| croelandt | #topic Glance download image from specific store | 14:03 |
| croelandt | #link https://review.opendev.org/c/openstack/glance-specs/+/963239 | 14:03 |
| croelandt | so yeah, using "oneof" was not great | 14:04 |
| abhishekk | yeah we need to finalise the params so that I can proceed | 14:04 |
| croelandt | &stores=ceph,s3&store-selection=[require/prefer] sounds good to me | 14:05 |
| croelandt | rosmaita: if you want to weigh in on that later ^ | 14:05 |
| croelandt | dansmith: ^ | 14:05 |
| abhishekk | OK, I will wait for tonight and then go ahead with ^ if there are no objections? | 14:06 |
| croelandt | sounds good | 14:06 |
| croelandt | #topic Decompression plugin | 14:06 |
| abhishekk | I think both are not available today due to US holiday | 14:06 |
| croelandt | abhishekk: you have the floor :) | 14:06 |
| croelandt | oh yeah | 14:06 |
| croelandt | ok so maybe wait for Monday | 14:06 |
| croelandt | tomorrow is Black Friday :D | 14:07 |
| abhishekk | yeah | 14:07 |
| croelandt | ok let's move on to decompression plugin | 14:07 |
| croelandt | can you explain the issue? | 14:07 |
| abhishekk | Yeah | 14:07 |
| abhishekk | there are 3 issues which are dependent | 14:07 |
| abhishekk | We support decompression for gzip, lha and zip | 14:08 |
| abhishekk | out of which gzip works with web-download which does not use stage api internally | 14:08 |
| abhishekk | but gzip with glance-direct fails as later it compares size and mismatches | 14:09 |
| abhishekk | then lha and zip has different failures | 14:09 |
| abhishekk | So all 3 are valid issues and we need those | 14:09 |
| croelandt | ok but they are unrelated? As in, we can fix them one by one? | 14:10 |
| abhishekk | the size mismatch fix needs to be land first | 14:11 |
| abhishekk | https://review.opendev.org/c/openstack/glance/+/968351 | Update image size after image decompression (968351) · Gerrit Code Review | 14:11 |
| abhishekk | this one needs to go 1st | 14:11 |
| abhishekk | then LHA and then zip | 14:11 |
| croelandt | I see | 14:12 |
| croelandt | glad to see that no one uses this plugin :D | 14:12 |
| abhishekk | The first one is required because other two will fail without it | 14:12 |
| croelandt | or at least no one is complaining | 14:12 |
| abhishekk | May be they are using gzip with web-download :D | 14:13 |
| croelandt | maybe | 14:13 |
| croelandt | #topic One easy patch per core dev | 14:13 |
| croelandt | #link https://review.opendev.org/c/openstack/glance/+/939684 | 14:13 |
| croelandt | yes we should definitely review this | 14:13 |
| * croelandt is guilty of not reviewing this | 14:13 | |
| abhishekk | its not easy though :D | 14:13 |
| croelandt | haha | 14:13 |
| croelandt | you lied! | 14:13 |
| croelandt | will take a look after the meeting | 14:13 |
| croelandt | #topic Open Discussion | 14:13 |
| abhishekk | thank you!! | 14:13 |
| croelandt | Anything else? | 14:14 |
| rajiv_ | Hi | 14:14 |
| abhishekk | Yeah, image encryption | 14:14 |
| croelandt | rajiv_: hello | 14:14 |
| abhishekk | I think we need to give priority to that | 14:14 |
| croelandt | abhishekk: the issue is that it is OK from the Glance side | 14:15 |
| 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 |
| croelandt | but Nova/Cinder had more concerns | 14:15 |
| croelandt | my take from PTG is that we need people from Nova/Cinder to commit to reviewing and helping Markus & Josephine | 14:15 |
| abhishekk | ack | 14:15 |
| abhishekk | Lets discuss this with nova and cinder next time then, I will try to bring rajat from cinder for this | 14:16 |
| croelandt | abhishekk: good | 14:16 |
| croelandt | rajiv_: so this is only what is listed tyhrough "tox -l" | 14:16 |
| croelandt | you can still run tox -epy312 | 14:17 |
| rajiv_ | yes, locally i was testing via -e py312, but was wondering why it was not added to envlist | 14:17 |
| croelandt | honestly it's annoying to change all the time | 14:18 |
| croelandt | and I don't personnally run tox -l | 14:18 |
| 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:18 |
| croelandt | do pre-prod testing :D | 14:19 |
| rajiv_ | haha sure :) | 14:19 |
| croelandt | I don't remember anything | 14:19 |
| croelandt | everything important should be in the release notes | 14:19 |
| abhishekk | No new db changes introduced since damaltian I thing, centralized databse for cache is pre damaltian i guess | 14:20 |
| rajiv_ | previously, when an image was uploaded, it used to pick visibility private but now its shared, is this expected ? | 14:20 |
| croelandt | abhishekk: do you remember if this is a change we introduced? | 14:21 |
| abhishekk | I don't remember we done this | 14:21 |
| rajiv_ | okay, lastly, regarding https://bugs.launchpad.net/glance/+bug/1916052 which i raised, was reverted, is comment. #16 valid ? | 14:23 |
| abhishekk | This is what I found https://review.opendev.org/c/openstack/glance/+/422897/7/releasenotes/notes/image-visibility-changes-fa5aa18dc67244c4.yaml | 14:23 |
| abhishekk | rajiv_: frankly no idea about this bug atm | 14:24 |
| 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:24 |
| rajiv_ | abhishekk: okay, can i follow up next week ? | 14:25 |
| abhishekk | I am not sure I can manage to have look at it due to workload i am having | 14:25 |
| rajiv_ | oops ok | 14:26 |
| croelandt | rajiv_: there was a fix and it was reverted? | 14:26 |
| rajiv_ | croelandt: yes | 14:26 |
| croelandt | ok | 14:27 |
| croelandt | so does #16 work for you? | 14:27 |
| rajiv_ | i havent tested it since its a critical change, idk if glance would run without keystone_auth | 14:28 |
| croelandt | well | 14:28 |
| croelandt | testing patches really helps :) | 14:28 |
| rajiv_ | sure, wanted to know if it was tested before i try! | 14:29 |
| rajiv_ | thanks, thats all it from my side | 14:30 |
| croelandt | nope | 14:30 |
| croelandt | we don't really test much on "real world scenarios" to be honest | 14:30 |
| rajiv_ | ah ok | 14:30 |
| croelandt | so if users who report issues can say "yeah that patch fixes my issue" it's great | 14:30 |
| croelandt | not sure about abhishekk, but I don't run OpenStack "for real", I just build it :) | 14:31 |
| abhishekk | :D | 14:31 |
| rajiv_ | oh ok, croelandt can we try revert the reverted patch :D ? | 14:31 |
| abhishekk | The mentioned patch in comment 16 has -1 from zuul | 14:31 |
| abhishekk | rajiv_: there might be reason why that patch got reverted, do you know it? | 14:32 |
| croelandt | I think rajiv_ is talking about the patch reverted by Erno | 14:32 |
| croelandt | see #4 | 14:32 |
| abhishekk | Yeah but there should be reason for that | 14:32 |
| abhishekk | ack | 14:32 |
| 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 |
| rajiv_ | that was the reason on the revert msg | 14:33 |
| abhishekk | yeah, this needs careful eyes we can not just rerevert it | 14:34 |
| rajiv_ | okay | 14:35 |
| croelandt | rajiv_: but if you got a lab where you can test #16, that helps :) | 14:35 |
| rajiv_ | sure, will update the thread accordingly | 14:36 |
| croelandt | thanks! | 14:37 |
| croelandt | I think we can end it for today | 14:37 |
| croelandt | thanks everyone for joining | 14:37 |
| croelandt | #endmeeting | 14:37 |
| opendevmeet | Meeting ended Thu Nov 27 14:37:23 2025 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 14:37 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/glance/2025/glance.2025-11-27-14.00.html | 14:37 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/glance/2025/glance.2025-11-27-14.00.txt | 14:37 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/glance/2025/glance.2025-11-27-14.00.log.html | 14:37 |
| masahito | hi glance team, please review the bug fix commit for the image list API query parameter. thanks. https://review.opendev.org/c/openstack/glance/+/934922 | 14:47 |
| opendevreview | Abhishek Kekane proposed openstack/glance master: Fix FileNotFoundError when registering task operations https://review.opendev.org/c/openstack/glance/+/968711 | 14:50 |
Generated by irclog2html.py 4.0.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!