| opendevreview | Sahil Kumbhar proposed openstack/glance-specs master: [Spec] S3 store: store location without credentials in the URI https://review.opendev.org/c/openstack/glance-specs/+/980244 | 05:39 |
|---|---|---|
| *** sakumbha_ is now known as sakumbha | 07:08 | |
| elodilles | croelandt: it is not allowed to update the hash of an existing release o:) do you want to cut a release before EOL tagging? then a separate release patch needs to be proposed for 29.2.1 for glance | 07:59 |
| elodilles | croelandt: anyway, i've created a release patch for that single bug o:) https://review.opendev.org/c/openstack/releases/+/987595/1 | 08:09 |
| elodilles | the patch on top of it should get Verified+1 when the final stable release is merged. | 08:10 |
| croelandt | elodilles: ah ok I messed up | 14:01 |
| croelandt | I wanted to update the hash for 2024.2-eol | 14:01 |
| dansmith | meeting? | 14:02 |
| croelandt | elodilles: thanks for taking care of that | 14:02 |
| croelandt | dansmith: I see you're eager to look at a list of urgent patches | 14:02 |
| croelandt | good. | 14:02 |
| croelandt | #startmeeting glance | 14:02 |
| opendevmeet | Meeting started Thu May 7 14:02:28 2026 UTC and is due to finish in 60 minutes. The chair is croelandt. Information about MeetBot at http://wiki.debian.org/MeetBot. | 14:02 |
| opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 14:02 |
| opendevmeet | The meeting name has been set to 'glance' | 14:02 |
| croelandt | #link https://etherpad.openstack.org/p/glance-team-meeting-agenda | 14:02 |
| croelandt | #topic roll call | 14:02 |
| croelandt | o/ | 14:02 |
| abhishekk | o/ | 14:02 |
| sakumbha | o/ | 14:02 |
| dansmith | o/ (but scowling) | 14:02 |
| croelandt | :-( | 14:03 |
| croelandt | pdeore_: around? | 14:03 |
| pdeore_ | croelandt, yes o/ | 14:03 |
| croelandt | #topic Release/periodic job updates | 14:03 |
| croelandt | All good \o/ | 14:03 |
| opendevreview | Abhishek Kekane proposed openstack/glance-specs master: [Spec] Parallel image import https://review.opendev.org/c/openstack/glance-specs/+/967046 | 14:03 |
| croelandt | #topic Important stable patches | 14:03 |
| croelandt | #link https://review.opendev.org/q/Id87c3aa0d0d44a731cc5e2439e0b464bb92460a2 | 14:04 |
| croelandt | Some people want to backport this | 14:04 |
| croelandt | it is a bugfix/workaround but also introduces a new config option | 14:04 |
| croelandt | so it both fits and does not fit the criteria for backporting | 14:04 |
| croelandt | I'd be in favor of backporting it | 14:04 |
| croelandt | Anyone agrees/disagrees? | 14:04 |
| abhishekk | *+1 | 14:04 |
| croelandt | good, I'll review those and probably merge them | 14:05 |
| croelandt | #topic Easy patches | 14:05 |
| croelandt | #link https://review.opendev.org/c/openstack/glance/+/986379 | 14:05 |
| croelandt | This is the usual constant migration thingy | 14:05 |
| croelandt | abhishekk: can you +2? :) | 14:05 |
| abhishekk | done | 14:06 |
| croelandt | thanks | 14:06 |
| croelandt | #topic M1 release | 14:06 |
| croelandt | so M1 is next week already | 14:06 |
| croelandt | I feel like we're just out of the PTG | 14:06 |
| croelandt | So, two specs to ideally merge | 14:06 |
| croelandt | #link https://review.opendev.org/c/openstack/glance-specs/+/980244 | 14:06 |
| croelandt | #link https://review.opendev.org/c/openstack/glance-specs/+/967046/ | 14:07 |
| abhishekk | I have updated the parallel image import spec, included why engine=parallel is not ideal for our usecase | 14:07 |
| croelandt | I had comments on this one | 14:07 |
| abhishekk | s3 credential less URL is straightforward and you can merge it today itself | 14:08 |
| croelandt | can you take a look at them after the meeting? | 14:08 |
| abhishekk | sakumbha: Remember, when you resolve the comments given on your spec, you need to address them in review and reply accordingly | 14:08 |
| croelandt | well, there is a bunch of stuff regarding S3 credentials | 14:08 |
| croelandt | it's not "easy" | 14:08 |
| croelandt | There is single/multi store support, lazy update, proper testing | 14:09 |
| croelandt | I see abhishekk +2ed,I'll take a second look today | 14:09 |
| abhishekk | I mean proposal is straight forward | 14:09 |
| croelandt | yes | 14:09 |
| croelandt | and non controversial | 14:09 |
| croelandt | #link https://review.opendev.org/c/openstack/glance/+/939684 | 14:10 |
| abhishekk | ;) | 14:10 |
| croelandt | I +2ed this patch a while ago | 14:10 |
| croelandt | it would be nice to have a second pair of eyes though | 14:10 |
| croelandt | pdeore_: dansmith: maybe one of you? :) | 14:10 |
| dansmith | oof, I'll have to build some context there | 14:11 |
| abhishekk | just FYI API and doc reference will be in a followup patch | 14:11 |
| dansmith | (re-build I mean) | 14:11 |
| pdeore_ | yeah I started looking at it but had to leave in between, will do it after meeting | 14:11 |
| croelandt | ok, thanks | 14:11 |
| croelandt | #link https://review.opendev.org/c/openstack/glance/+/986467 | 14:11 |
| croelandt | This I briefly discussed (before sending the patch) with dansmith | 14:11 |
| abhishekk | dansmith: we have centralized db for caching now, so if we pass image id to an API call then it will show on which glance node that image is cached | 14:12 |
| croelandt | it's teh last functional test taht needs to spawn a server | 14:12 |
| croelandt | and we don't care about IPv4/v6 anymore | 14:12 |
| croelandt | so if we could remove that test | 14:12 |
| croelandt | then it opens up the way for removing a bunch of stuff in the actual code :) | 14:12 |
| croelandt | basically, most of wsgi.py iirc | 14:12 |
| croelandt | would be nice to review this otherwise we'll still have eventlet patches when I retire in 30 years time | 14:12 |
| abhishekk | https://review.opendev.org/c/openstack/glance/+/975118 | 14:13 |
| croelandt | can you prefix it with "eventlet-removal"? | 14:15 |
| croelandt | in the topic, I mean | 14:16 |
| croelandt | so we can find it easily | 14:16 |
| abhishekk | ack | 14:16 |
| croelandt | so I assume tomorrow is a bank holiday for most of us? | 14:17 |
| croelandt | Or is it not? | 14:17 |
| dansmith | not me | 14:17 |
| abhishekk | neither do we | 14:18 |
| croelandt | oh | 14:18 |
| croelandt | you gotta strike to get the "end of WW2" bank holiday | 14:19 |
| croelandt | So these patches I've listed are the ones we need for a successful M1 | 14:19 |
| abhishekk | :P | 14:19 |
| croelandt | + encryption spec | 14:19 |
| abhishekk | ack | 14:19 |
| dansmith | yeah was going to say.. encryption spec needs approval right? | 14:19 |
| croelandt | If we don't merge them we're gonna need a review party :D | 14:19 |
| croelandt | but I don't really want to review a bunch of specs in a meeting | 14:19 |
| croelandt | #link https://review.opendev.org/c/openstack/glance-specs/+/964755 | 14:20 |
| croelandt | ah, you are happy with it | 14:20 |
| abhishekk | You can say that I don't see any objections on the spec then I am going to merge those :P | 14:20 |
| croelandt | I think as it's quite important both abhishekk and I should take another look | 14:20 |
| mhen | that would be much appreciated, thank you | 14:20 |
| abhishekk | ack | 14:21 |
| croelandt | ok so ideally we all look at the patches that were linked here | 14:21 |
| croelandt | otherwise I'll annoy everyone in the Red Hat meeting on Monday :D | 14:21 |
| croelandt | #topic Open Discussion | 14:22 |
| croelandt | Anything else for today? | 14:22 |
| abhishekk | nothing from me for now | 14:22 |
| croelandt | ok, let's go review patches, and I'll see you on Monday | 14:23 |
| croelandt | anything urgent, you should ask me in the next 1.5 hours | 14:23 |
| abhishekk | enjoy | 14:23 |
| croelandt | :D | 14:23 |
| croelandt | #endmeeting | 14:24 |
| opendevmeet | Meeting ended Thu May 7 14:24:35 2026 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 14:24 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/glance/2026/glance.2026-05-07-14.02.html | 14:24 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/glance/2026/glance.2026-05-07-14.02.txt | 14:24 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/glance/2026/glance.2026-05-07-14.02.log.html | 14:24 |
| opendevreview | Merged openstack/glance master: Change DB migration constant to 2026_2 https://review.opendev.org/c/openstack/glance/+/986379 | 18:03 |
Generated by irclog2html.py 4.1.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!