| *** mhen_ is now known as mhen | 01:52 | |
| croelandt | #startmeeting glance | 14:01 |
|---|---|---|
| opendevmeet | Meeting started Thu Oct 16 14:01:00 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 | o/ | 14:01 |
| dansmith | o/ | 14:01 |
| msava | o/ | 14:01 |
| croelandt | #link https://etherpad.openstack.org/p/glance-team-meeting-agenda | 14:02 |
| croelandt | #topic Release/periodic job updates | 14:02 |
| croelandt | We're good on that front, some small issue with oslo-tips was fixed thanks to tkajinam | 14:02 |
| croelandt | #topic Important stable patches | 14:02 |
| croelandt | #link https://review.opendev.org/q/I436d56d0489b081822ed55108e10d43cc38512bb | 14:02 |
| croelandt | #link https://review.opendev.org/c/openstack/glance_store/+/962223 | 14:03 |
| croelandt | abhishek_: if you can help review some of those :) | 14:03 |
| croelandt | Or anyone with rights on the stable branches (not sure who that is :p) | 14:03 |
| croelandt | #topic Glance specs | 14:03 |
| croelandt | #link https://review.opendev.org/q/project:openstack/glance-specs | 14:04 |
| croelandt | Still one week to review those before we discuss them live at PTG \o/ | 14:04 |
| croelandt | #topic Open Discussion | 14:04 |
| croelandt | #link https://etherpad.opendev.org/p/2026.1-ptg-glance-planning | 14:04 |
| abhishek_ | ack | 14:04 |
| croelandt | Please add topics *now* since I'll try to find time slots for these next week | 14:04 |
| croelandt | and then we have a few patches that could use reviews | 14:05 |
| croelandt | #link https://review.opendev.org/c/openstack/glance/+/963868 | 14:05 |
| croelandt | #link https://review.opendev.org/c/openstack/glance/+/963875 | 14:05 |
| croelandt | Any other topics? | 14:05 |
| dansmith | um | 14:05 |
| abhishek_ | croelandt: I can see functional tests arre failing on stable patch related to s3 | 14:05 |
| dansmith | croelandt: I just saw your comment here, which I replied to: https://review.opendev.org/c/openstack/glance/+/963868 | 14:06 |
| abhishek_ | *unit | 14:06 |
| dansmith | croelandt: but that aside, are you good to merge that if we drop the depends-on? | 14:06 |
| croelandt | so why did you need depends-on in the first place? | 14:07 |
| dansmith | to test with a bootable iso | 14:08 |
| dansmith | I have that sample image queued, but we can't merge it until/unless glance gets support for it in import (or we exclude that in tempest) | 14:08 |
| croelandt | so if we drop depends-on, do the tests still pass for your glance patch? | 14:08 |
| dansmith | but I want to be able to merge the fix for upload, which is the common case and was easier to make work | 14:08 |
| dansmith | yes, it was just to get a testing data point (see my comments where I quote the iso-bootable) test | 14:09 |
| croelandt | then yeah, let's drop depends-on :) | 14:09 |
| dansmith | cool | 14:09 |
| croelandt | can you update the patch for pep8, remove depends-on, fix the = iso thing, and then I'll re-review it? | 14:10 |
| dansmith | I will of course :) | 14:10 |
| croelandt | perfect, I'll take another look soon then :) | 14:10 |
| abhishek_ | dansmith: if possible please look at s3 credential patch as well | 14:11 |
| dansmith | abhishek_: ack, I know.. I'm like buried this week | 14:12 |
| dansmith | sorry | 14:12 |
| abhishek_ | no problem, I can understand | 14:12 |
| croelandt | Any other topics for today? | 14:12 |
| rosmaita | yeah, caracal going to unmaintained: https://review.opendev.org/c/openstack/releases/+/963608 | 14:12 |
| rosmaita | there are some open reviews: https://review.opendev.org/q/(project:openstack/glance+OR+project:openstack/glance_store+OR+project:python-glanceclient)+status:open+branch:stable/2024.1 | 14:13 |
| abhishek_ | croelandt: I think we should sync offline for deciding final topics for PTG discussion | 14:13 |
| croelandt | rosmaita: ok abhishek_ and i can take a look at these patches | 14:13 |
| croelandt | abhishek_: yeah, I think we can discuss more topics than what we'll actually implement | 14:13 |
| croelandt | but no way we do everything | 14:13 |
| croelandt | so we should set up priorities | 14:13 |
| abhishek_ | right | 14:14 |
| croelandt | let me take a look at the list again | 14:14 |
| abhishek_ | So we will decide priority and then keep 1 hour to go through remaining during PTG | 14:14 |
| croelandt | something like that, yeah | 14:14 |
| croelandt | Anything else? | 14:15 |
| rosmaita | croelandt: there are a bunch of unreleased but merged changes in caracal, we should probably do a release before it goes to unmaintained | 14:15 |
| abhishek_ | ++ | 14:15 |
| rosmaita | i can put up a release patch, and then you can change the hash if you decide to merge more stuff | 14:16 |
| croelandt | yeah let's look at the patches first | 14:16 |
| croelandt | let's end the meeting so we can go review patches, ok? | 14:16 |
| abhishek_ | yep | 14:16 |
| croelandt | #endmeeting | 14:17 |
| opendevmeet | Meeting ended Thu Oct 16 14:17:01 2025 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 14:17 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/glance/2025/glance.2025-10-16-14.01.html | 14:17 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/glance/2025/glance.2025-10-16-14.01.txt | 14:17 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/glance/2025/glance.2025-10-16-14.01.log.html | 14:17 |
| opendevreview | Dan Smith proposed openstack/glance master: Handle images detected as ISO+GPT https://review.opendev.org/c/openstack/glance/+/963868 | 14:33 |
| rosmaita | croelandt: https://review.opendev.org/c/openstack/releases/+/964227 | 14:37 |
| dansmith | abhishek_: so the s3 patch has changed a lot since I first reviewed it.. can you summarize? | 14:38 |
| abhishek_ | Ok, I followed similar approach like cinder was doing, 3 cases as described in commit message | 14:39 |
| abhishek_ | 1. multistore but credential changes, single store (traditional approach) but creds changed, upgrade from singe to multistore to add store_name in metadata | 14:40 |
| dansmith | okay I guess it just seems like a lot of changed code and was hoping for a summary of _why_ the change and what specifically has changed but I guess I'll just have a look | 14:53 |
| croelandt | abhishek_: so https://review.opendev.org/c/openstack/glance/+/941291 seems somewhat important | 15:43 |
| croelandt | abhishek_: can we merge https://review.opendev.org/q/I436d56d0489b081822ed55108e10d43cc38512bb ? This would be nice for this patch to make the 2024.1 cut | 15:55 |
| opendevreview | Dan Smith proposed openstack/glance master: Handle images detected as ISO+GPT https://review.opendev.org/c/openstack/glance/+/963868 | 16:07 |
| croelandt | abhishek_: well only https://review.opendev.org/c/openstack/glance_store/+/961869 gotta fixthe other ones since they don't cherry-pick from the right patch | 16:08 |
| croelandt | I'll do it if you merge the 2025.1 patch :) | 16:08 |
| abhishek_ | croelandt: me not near laptop atm, will have a look | 16:17 |
| abhishek_ | ack | 16:19 |
| opendevreview | Merged openstack/glance_store stable/2025.1: Support multipart upload when image_size == 0 https://review.opendev.org/c/openstack/glance_store/+/961869 | 18:33 |
| opendevreview | Merged openstack/glance stable/2024.1: Update glance-store requirement to >= 4.7.0 https://review.opendev.org/c/openstack/glance/+/941291 | 19:07 |
| opendevreview | Cyril Roelandt proposed openstack/glance_store stable/2024.2: Support multipart upload when image_size == 0 https://review.opendev.org/c/openstack/glance_store/+/961870 | 19:59 |
| opendevreview | Cyril Roelandt proposed openstack/glance_store stable/2024.1: Support multipart upload when image_size == 0 https://review.opendev.org/c/openstack/glance_store/+/961871 | 20:00 |
| croelandt | abhishekk_: ^ I updated 2024.2 and 2024.1 \o/ | 20:00 |
Generated by irclog2html.py 4.0.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!