| croelandt | #startmeeting glance | 14:01 |
|---|---|---|
| opendevmeet | Meeting started Thu Oct 22 14:01:18 2026 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.opendev.org/p/glance-team-meeting-agenda | 14:01 |
| croelandt | abhishekk: ? | 14:03 |
| croelandt | pdeore_: ? | 14:03 |
| abhishekk | o/ | 14:03 |
| pdeore_ | o/ | 14:03 |
| abhishekk | sorry was busy in other stuff | 14:04 |
| croelandt | np np | 14:04 |
| croelandt | short agenda today but we've been in meeting all week last week so it's fine :) | 14:05 |
| croelandt | #topic Release/periodic job updates | 14:05 |
| croelandt | #link https://zuul.opendev.org/t/openstack/builds?project=openstack%2Fglance&project=openstack%2Fglance_store&project=openstack%2Fpython-glanceclient&pipeline=periodic | 14:05 |
| croelandt | it's all good except for FIPS | 14:05 |
| croelandt | and FIPS job has been abandonned a loooooong time ago | 14:05 |
| croelandt | everyone ok with me removing it? | 14:05 |
| abhishekk | sounds good | 14:05 |
| pdeore_ | +1 | 14:05 |
| croelandt | #topic stable patches | 14:06 |
| croelandt | #link https://review.opendev.org/c/openstack/glance/+/996357 seems important | 14:06 |
| croelandt | I have not had time to take a second look at it but it appears to fix a perforamnce issue in 2026.1 | 14:06 |
| croelandt | I'll review it and ideally someone else would do it too : | 14:07 |
| croelandt | :) | 14:07 |
| abhishekk | ack, as per our stable policy you can Ninja it if it looks all ok | 14:07 |
| croelandt | yeah but I'd like a second pair of eyes because it has the potential to break stuff, I think :) | 14:07 |
| croelandt | but anyway, regarding specs, abhishekk you have the mic | 14:08 |
| abhishekk | Yes, all those need reviews | 14:08 |
| abhishekk | The upload format is simple and easy to get in | 14:08 |
| abhishekk | Regarding s3 work we need to prioritise it as our Nova work is depends on that | 14:09 |
| croelandt | so do we call it multipart or multistream? | 14:09 |
| croelandt | I feel something else was multipart | 14:09 |
| abhishekk | yes multistream | 14:09 |
| abhishekk | but we have glance temp url as well | 14:10 |
| abhishekk | the important stuff I want to talk about is Tasks API removal | 14:10 |
| croelandt | ok | 14:11 |
| abhishekk | I have sent mail as discussed | 14:11 |
| abhishekk | https://lists.openstack.org/archives/list/openstack-discuss@lists.openstack.org/message/GVSCNKJ37HDEQVNDLVYKN7KATAU4D4CP/ | 14:11 |
| abhishekk | I got reply that we need to follow API-SIG guideline and instead of increasing minor version we should opt for /v3 | 14:11 |
| croelandt | how much do we hate that? | 14:12 |
| croelandt | I mean, when you think about it, removing an API, that is a big hcange | 14:12 |
| croelandt | so this might be a major one | 14:12 |
| abhishekk | I think this is going to be a major work which needs lots of cross project dicssions | 14:12 |
| abhishekk | I still remember we needed 2-3 years of discussion for planning for /v2 | 14:12 |
| abhishekk | Yes this is going to be major headache | 14:13 |
| croelandt | DO we have consumers for that API? We should not, but... | 14:13 |
| mklejn | wasn't glance /v3 api already existing somewhere in the past, which got later separated into Glare? | 14:14 |
| abhishekk | I am not sure anyone is using that API | 14:14 |
| abhishekk | yeah that was ages ago | 14:15 |
| abhishekk | I think glare is also now history? | 14:15 |
| croelandt | yeah glare is history | 14:16 |
| croelandt | have we ever removed an API before? | 14:16 |
| abhishekk | So if we have weekly meetings for TC then I think @croelandt you need to take this matter there and fight for us to save us from /v3 war | 14:16 |
| abhishekk | I think no | 14:16 |
| croelandt | hm | 14:16 |
| croelandt | so what is the actual issue with v3? Consumers would have to update their code to call v3 instead of v2? | 14:16 |
| mklejn | also, regarding 'temp url' -- ongoing, my draft is under internal review in attempt to catch anything stupid | 14:16 |
| abhishekk | yes | 14:17 |
| croelandt | you'd figure it's not that hard :p | 14:17 |
| croelandt | but it would mean some kind of transition period | 14:17 |
| abhishekk | exaclty, we need to bring all the consumers onboard 1st with this effort | 14:18 |
| abhishekk | mklejn: ack, we will discuss more if needed in open discussion for the same | 14:18 |
| croelandt | so maybe we should figure out who the consumers are, first | 14:18 |
| croelandt | I think the SDK is there | 14:19 |
| abhishekk | Tempest, Ironic, Nova, Cinder | 14:19 |
| abhishekk | Support in python-glanceclient as well | 14:20 |
| croelandt | glanceclient we control :) | 14:20 |
| abhishekk | yep | 14:20 |
| croelandt | maybe we should discuss with Dan & Brian? | 14:20 |
| croelandt | to figure out how hard it would be to switch to v3 in their projects | 14:20 |
| abhishekk | I can prepare a plan for v3 migration if required and we can discuss next week again | 14:20 |
| croelandt | I think Artem made sense | 14:21 |
| croelandt | (unfortunately for us) | 14:21 |
| croelandt | and while it's true others are breaking the API versioning contract... | 14:21 |
| abhishekk | when probably dansmith and rosmaita around? | 14:22 |
| croelandt | a bit later than this :) | 14:22 |
| abhishekk | So either move to v3 or keep deprecated tasks API forever until we are ready to move to v3 | 14:23 |
| rosmaita | i think we should only do a v3 if we rewrite glance in rust | 14:23 |
| dansmith | @omg please don't rev the api to remove an unused thing | 14:23 |
| croelandt | so we don't remove it, but is it disabled by default? | 14:24 |
| dansmith | no, we remove it | 14:24 |
| abhishekk | how? | 14:25 |
| dansmith | "every OpenStack service violated this X times" | 14:25 |
| abhishekk | :D | 14:25 |
| croelandt | hm | 14:25 |
| croelandt | not a bad point either :) | 14:25 |
| dansmith | and also, I bet the SDK doesn't even have support for this, so he's just making a point with no reasoning for this specific thing | 14:25 |
| croelandt | I'm wondering what actual issues it would cause | 14:25 |
| croelandt | dansmith: is there no support for task API at all? | 14:25 |
| dansmith | in sdk? I bet not, it's probably been deprecated since before SDK existed no/ | 14:26 |
| croelandt | we have image task list/show in OSC | 14:26 |
| croelandt | glance image-tasks is not implemented (get tasks associated with an image) | 14:26 |
| abhishekk | that is not related to old taks api | 14:26 |
| croelandt | glance task-create is WONTFIX | 14:26 |
| croelandt | oh is it only task-create? | 14:27 |
| abhishekk | probably | 14:27 |
| croelandt | hm, that changes everything | 14:27 |
| abhishekk | Ok, so action plan would be to check SDK and then revert to mail accordingly and proceed as per plan? | 14:28 |
| dansmith | here's a list of apis nova removed in a single shot for reference: https://docs.openstack.org/nova/latest/reference/api-microversion-history.html#microversion | 14:28 |
| dansmith | and we're far more stable with our APIs than glance is | 14:28 |
| dansmith | abhishekk: anything that SDK actually hits, just always return "409 Gone" and move on, IMHO | 14:28 |
| croelandt | abhishekk: sounds good | 14:29 |
| abhishekk | ack, so we should continue with our work then | 14:29 |
| croelandt | let's discuss the SDK after the meeting | 14:29 |
| croelandt | mklejn: you wanted to discuss your spec? | 14:30 |
| abhishekk | So tempest change is already posted, https://review.opendev.org/c/openstack/tempest/+/1011263 | 14:30 |
| abhishekk | yeah, we can move to open-discussion | 14:30 |
| mklejn | croelandt: I'll first post a refreshed draft and if there are questions that require higher bandwidth than gerrit comments I'll happily answer them next week | 14:31 |
| croelandt | ok, please ping us when you're ready for reviews | 14:32 |
| croelandt | Anything else? | 14:33 |
| abhishekk | mklejn: also related to multipart download work | 14:33 |
| abhishekk | should we discuss it next week, who is going to work on that? | 14:33 |
| abhishekk | I think we can conclude for the day | 14:35 |
| croelandt | ok, let's end this | 14:36 |
| croelandt | longer than expected | 14:36 |
| croelandt | #endmeeting | 14:36 |
| opendevmeet | Meeting ended Thu Oct 22 14:36:05 2026 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 14:36 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/glance/2026/glance.2026-10-22-14.01.html | 14:36 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/glance/2026/glance.2026-10-22-14.01.txt | 14:36 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/glance/2026/glance.2026-10-22-14.01.log.html | 14:36 |
| croelandt | abhishekk: so there is a create_task() method in openstack/image/v2/_proxy.py | 14:40 |
| abhishekk | ack | 14:41 |
| abhishekk | but do we really need to worry now about this? | 14:43 |
| dansmith | AFAIK, nobody can reasonably be using this, which means that there is no client/sdk behavior to salvage here | 14:45 |
| dansmith | so we can leave the URL and return a 409 if we want, but actually preserving it or bumping the entire API version for this is insanity, IMHO | 14:45 |
| dansmith | it's not like the offered example of cinder v2 and v3 where we were going from one supported widely-used thing to another | 14:45 |
| abhishekk | right | 14:47 |
| * abhishekk thinking that sending mail was a mistake | 14:49 | |
| croelandt | abhishekk: too late now | 15:19 |
| abhishekk | ;/ | 15:21 |
| tkajinam | maybe 403 forbidden or 410 gone can be better because the response does not depend on status of the target resource. | 15:40 |
| tkajinam | afair 401 is what nova used to shut down all apis for nova-net/xen | 15:40 |
| tkajinam | s/401/410/ | 15:40 |
| dansmith | tkajinam: sorry I meant 410 above | 15:44 |
| dansmith | "gone" is what I wanted, I just forgot the code ;D | 15:44 |
| dansmith | abhishekk: ^ | 15:45 |
| tkajinam | dansmith, oh, ok | 15:45 |
| tkajinam | then I totally agree | 15:45 |
| abhishekk | dansmith: ack | 15:46 |
| abhishekk | So we will keep endpoints and return 410 and remove the internal code | 15:46 |
| dansmith | yup | 15:47 |
| abhishekk | ack, sounds good | 15:48 |
| croelandt | dansmith: does Nova even use that Glance endpoint? | 16:15 |
| dansmith | no | 16:16 |
| *** zseguin is now known as Guest3574 | 19:13 | |
Generated by irclog2html.py 4.1.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!