| *** mhen_ is now known as mhen | 01:45 | |
| tonyb | croelandt: Are https://bugs.launchpad.net/glance/+bug/2121144 and https://review.opendev.org/c/openstack/requirements/+/959560 still relevant ? | 10:11 |
|---|---|---|
| croelandt | tonyb: yes, as we have not yet provided a fix | 13:28 |
| croelandt | tonyb: I should probably rebase the patch then, thanks for pointing that out | 13:28 |
| croelandt | tonyb: done | 14:00 |
| croelandt | #startmeeting glance | 14:01 |
| opendevmeet | Meeting started Thu Oct 23 14:01:04 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 |
| opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 14:01 |
| opendevmeet | The meeting name has been set to 'glance' | 14:01 |
| croelandt | #topic roll call | 14:01 |
| croelandt | #link https://etherpad.openstack.org/p/glance-team-meeting-agenda | 14:01 |
| rosmaita | o/ | 14:02 |
| mhen | o/ | 14:02 |
| dansmith | o/ (but multistasking) | 14:02 |
| croelandt | #topic Release/periodic job updates | 14:02 |
| croelandt | ALl good \o/ Except the fips one we know about | 14:02 |
| croelandt | #topic Important stable patches | 14:03 |
| croelandt | #topic https://review.opendev.org/c/openstack/glance_store/+/961870 | 14:03 |
| croelandt | There is a weird install issue for this one. This is annoying because it prevents us from merging the same patch in stable/2024.1 and the deadline is tomorrow :-( | 14:03 |
| croelandt | I think I should probably ask someone, but not sure who can help with these kind of install issues | 14:03 |
| croelandt | We're trying to install both glance-store===4.8.2 from the upper constraints, and 4.8.3dev1 from... the repo itself, I guess | 14:04 |
| croelandt | If anyone has input on how to fix this, I'm all ears :) | 14:04 |
| croelandt | (well, eyes since this is IRC) | 14:04 |
| rosmaita | this is just for that cross-glance-tox-functional job? | 14:05 |
| croelandt | yes | 14:05 |
| croelandt | and just on 2024.2 | 14:05 |
| croelandt | so, moving on, we can come back to this after the meeting | 14:08 |
| croelandt | #topic One easy patch per core dev | 14:08 |
| croelandt | #link https://review.opendev.org/c/openstack/glance/+/932768 | 14:08 |
| croelandt | I could probably Ninja approve this doc/sample patch, but always good to have more eyes on a patch :) | 14:09 |
| croelandt | #topic Glance specs | 14:09 |
| croelandt | #link https://review.opendev.org/q/project:openstack/glance-specs | 14:09 |
| croelandt | we can always review these before next week :) | 14:09 |
| croelandt | #topic Open Discussion | 14:09 |
| croelandt | Mostly about PTG this week | 14:09 |
| croelandt | #link https://etherpad.opendev.org/p/oct2025-ptg-glance | 14:09 |
| croelandt | #link https://etherpad.opendev.org/p/2026.1-ptg-glance-planning | 14:09 |
| croelandt | The proposed agenda. Please ping me if I missed anything, if a session should last longer, etc. | 14:10 |
| croelandt | After Friday EOD (my time) it'll be too late and you'll have to wait 6 months :D | 14:10 |
| croelandt | Anything else? | 14:10 |
| mhen | regarding image encryption, we have two sessions now correct? | 14:11 |
| rosmaita | sorry, i am in another meeting, will look at the functional test failure in 20 min or so | 14:12 |
| mhen | Thursday @ 18 UTC Glance+Nova | 14:12 |
| mhen | and the other one (Glance+Cinder) does not have a fixed time slot yet or did I miss something? | 14:13 |
| croelandt | mhen: there is one big Nova/Glance/Cinder session, for an hour | 14:13 |
| croelandt | did you want an hour with Nova and an hour with Cinder? | 14:13 |
| croelandt | and it's 16:00 UTC | 14:13 |
| croelandt | not sure where you see 18UTC | 14:13 |
| mhen | oh yea 16 sorry, messed up time conversion | 14:13 |
| mhen | I'm fine with a combined session with all teams, that's even better | 14:14 |
| mhen | I wonder if 1h is enough then though | 14:14 |
| rosmaita | yeah, we ususally lose some time with everybody showing up, getting into the right meeting room, etc | 14:15 |
| croelandt | More than an hour you usually lose people, I'd say | 14:15 |
| croelandt | We're going to spend between 10 and 20 hours in technical meetings this week | 14:15 |
| croelandt | it's draining | 14:15 |
| rosmaita | yeah, but you can say, we can have two 1 hour sessions, or 1 combined 90 min session | 14:15 |
| croelandt | I'd say if you need more than one hour to explain your topic, it's already not great | 14:16 |
| croelandt | Is there no way to discuss the main ideas in one hour | 14:16 |
| croelandt | and refine the details during the cycle, with specs & patches? | 14:16 |
| mhen | Josephine (Luzi) did already meet a bunch of people and had some early discussions last week so maybe we have enough of a headstart going into the discussion | 14:17 |
| rosmaita | ok, stick with an hour then | 14:17 |
| croelandt | I'm really worried spending more than an hour is going to lead to people being fed up with the topic and not fully engaged mentally | 14:18 |
| croelandt | but mhen knows better than me | 14:18 |
| croelandt | mhen: if more than an hour is needed, we need to make this happen *now* | 14:18 |
| mhen | the thing is: I actually don't know - up until recently this has been mostly between us, Glance and Cinder for the longest time | 14:19 |
| croelandt | I know :) It is tough to chair a session, especially on a super technical topic like yours | 14:20 |
| mhen | now Nova is joining the discussions more actively again (which is good) | 14:20 |
| mhen | but I don't know how many things will be brought up or how controversial the discussion will get now that we have 3 projects discussing it and maybe 3 different perspectives on it | 14:21 |
| croelandt | If this is such a huge topic, one of the key points to discuss is going to be "how do we keep in touch during the cycle?" and maybe that means we need someone from Nova and Cinder to join this meeting every week (not necessarily rosmaita and dansmith ) | 14:21 |
| croelandt | otherwise we'll discuss it once every 6 months at PTG and it might go stale | 14:21 |
| mhen | we did a weekly popup team IRC meeting for a year or two in the past but it slowly lost participants | 14:22 |
| mhen | maybe we can revive that | 14:22 |
| mhen | I will try to discuss this in the session briefly | 14:22 |
| croelandt | I think you can also prepare a detailed description of the topic to make it easier to follwo along and so everyone has a rough idea of how we're doing timewise | 14:24 |
| croelandt | Anyway, I think we can wrap up, unless there are any other topics? | 14:24 |
| rosmaita | release patches | 14:24 |
| croelandt | oh yeah | 14:25 |
| croelandt | patcheS, plural? | 14:25 |
| croelandt | I see https://review.opendev.org/c/openstack/releases/+/964227 | 14:25 |
| croelandt | which is soft-blocked by the failure discussed above | 14:25 |
| rosmaita | so, this situation is that one needs to be revised (update the hash) | 14:26 |
| croelandt | yes | 14:26 |
| croelandt | I would ideally manage to merge the stable/2024.2 and stable/2024.1 patch, and then change the hash | 14:27 |
| rosmaita | the more recent releases only need to include stuff that is being included in Caracal but hasn't been released yet in the newer branches | 14:27 |
| croelandt | I don't follow | 14:27 |
| rosmaita | we don't have to release from HEAD of the more recent stable branches | 14:28 |
| rosmaita | though for 2024.2 it looks like we do | 14:28 |
| rosmaita | ok, let me back up | 14:29 |
| rosmaita | this looks ready: https://review.opendev.org/c/openstack/releases/+/964240 | 14:30 |
| rosmaita | and so does this one: https://review.opendev.org/c/openstack/releases/+/964241 | 14:30 |
| rosmaita | so even if the 2024.2 gate is broken right now, we can still release | 14:30 |
| croelandt | yes | 14:31 |
| croelandt | my concern is more Caracal | 14:31 |
| croelandt | https://review.opendev.org/c/openstack/releases/+/964227 | 14:31 |
| croelandt | this is the *last* release for Caracal, right? | 14:31 |
| rosmaita | right | 14:32 |
| croelandt | so if we don't merge the patches in the 2024.1 branch, they will never be part of a release | 14:32 |
| rosmaita | what else did you want to get into it? | 14:32 |
| croelandt | the patch above that failed to merge for 2024.2 | 14:32 |
| croelandt | https://review.opendev.org/q/I436d56d0489b081822ed55108e10d43cc38512bb | 14:32 |
| abhishekk_ | @dansmith: o/ i have pushed new changes for s3 as per suggestions, please have a look when you get time | 14:36 |
| dansmith | abhishekk_: I know, I have it open...just keep getting interrupted :/ | 14:37 |
| croelandt | rosmaita: ok let's end the meeting and look at the failure "offline", ok? | 14:37 |
| rosmaita | ok | 14:38 |
| abhishekk_ | no problem, thanks for the inputs, it was better and covering all the scenarios | 14:38 |
| croelandt | #endmeeting | 14:38 |
| opendevmeet | Meeting ended Thu Oct 23 14:38:38 2025 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 14:38 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/glance/2025/glance.2025-10-23-14.01.html | 14:38 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/glance/2025/glance.2025-10-23-14.01.txt | 14:38 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/glance/2025/glance.2025-10-23-14.01.log.html | 14:38 |
| mhen | croelandt: regarding "prepare a detailed description of the topic" - is there a good way to reach most of the session participants beforehand? | 14:39 |
| mhen | I could prepare a short etherpad with a topic summary, the current state and things to discuss like a lightweight agenda | 14:39 |
| croelandt | mhen: update your topic in https://etherpad.opendev.org/p/2026.1-ptg-glance-planning#L128 and I'll reuse that in the agenda | 14:41 |
| abhishekk_ | mhen: you can reach via mail to openstack mailing list, keep [ptg] in subject line | 14:41 |
| mhen | croelandt: should I cram it directly in there or link to a dedicated etherpad? | 14:42 |
| croelandt | cram it in there, please | 14:42 |
| mhen | gotcha | 14:42 |
| croelandt | rosmaita: I approved your releases patches | 15:00 |
| rosmaita | not sure that's a good idea | 15:01 |
| croelandt | any idea about the test failure? Or do you someone who could help with that? | 15:01 |
| croelandt | rosmaita: DAMN IT | 15:01 |
| croelandt | why is that? :) | 15:01 |
| rosmaita | i'm not sure we caught all the changes ... trying to fiugre out the cross-functional failure now | 15:02 |
| rosmaita | actually, it will only be an issue if we get this change merged into caracal, i think | 15:03 |
| croelandt | I, once again, don't follow | 15:04 |
| rosmaita | well, if we get it into the final caracal release, it should also be in a dalmatian release, but the dalmatian release patch is pointing to a hash that doesn't include that change | 15:06 |
| croelandt | oh yeah | 15:07 |
| rosmaita | not sure what the epoxy situation is | 15:07 |
| croelandt | ok | 15:07 |
| croelandt | let's figure out the failure then | 15:07 |
| croelandt | and think about releases later | 15:07 |
| rosmaita | yep | 15:07 |
| rosmaita | so un-approve those patches, please | 15:08 |
| croelandt | yep, done | 15:11 |
| rosmaita | croelandt: i can't find any difference between that branch and the others with respect to that job ... last recheck was a week ago, i just issued another | 15:38 |
| rosmaita | and it failed | 15:48 |
| dansmith | abhishekk_: I think there's a problem, I commented, let me know if I'm wrong | 15:58 |
| dansmith | abhishekk_: much better structure of making the two paths the same though I think | 15:59 |
| croelandt | rosmaita: yeah maybe I should ask for help on openstack-infra? | 16:37 |
| rosmaita | i have asked in there for suggestions | 16:38 |
| rosmaita | i need to go grab lunch if you could keep an eye on the conversation | 16:40 |
| rosmaita | (and walk the dog, so i will be afk) | 16:41 |
| croelandt | oh rihgt | 16:41 |
| croelandt | well I'm gonna need torun myself, but I'll stay for a bit :) | 16:46 |
| croelandt | rosmaita: let's rename it to glancestore | 17:41 |
| rosmaita | a bit late for that, but yeah! | 17:42 |
| croelandt | ok I'm going to angrily run now | 17:42 |
| rosmaita | croelandt: ok, soon as this is fixed, i will backport that patch to Caracal and revise all the release patches | 17:43 |
| croelandt | so this patch is already up for Caracal | 17:43 |
| croelandt | https://review.opendev.org/q/I436d56d0489b081822ed55108e10d43cc38512bb | 17:44 |
| croelandt | My issue is that the deadline for Caracal is tomorrow | 17:44 |
| rosmaita | well, you know how deadlines are | 17:51 |
| opendevreview | Brian Rosmaita proposed openstack/glance_store stable/2024.2: Support multipart upload when image_size == 0 https://review.opendev.org/c/openstack/glance_store/+/961870 | 18:48 |
| opendevreview | Abhishek Kekane proposed openstack/glance master: Lazy update S3 URL on credential rotation https://review.opendev.org/c/openstack/glance/+/963875 | 21:21 |
| abhishekk | tonyb: i think both are still relevant however we have workaround if we don't want to pin the version, also we are going to discuss this in PTG, https://review.opendev.org/c/openstack/glance-specs/+/962248 | 21:23 |
| tonyb | Okay, so it's still relevant, and the work long term-fix is probably a little way off. So it may be worth adding a cap. | 21:38 |
| tonyb | Where do the environment variables need to be set? Could you add os.environ[xxx]='WHEN_REQUIRED' to the early start-up code for glance* and python-glanceclient? | 21:41 |
| abhishekk | that needs to be checked, I think croelandt can answer this better | 21:44 |
| tonyb | Okay I'll wait for them to come back online ... I'm guessing from the name they're in the EU | 21:46 |
| abhishekk | yeah | 21:48 |
| * abhishekk going offline | 21:49 | |
Generated by irclog2html.py 4.0.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!