| rosmaita | croelandt: not a bother, i appreciate your concern, and i am glad that you are clearing out old reviews | 00:06 |
|---|---|---|
| opendevreview | Cyril Roelandt proposed openstack/glance master: Fix image name computation when deleting an image https://review.opendev.org/c/openstack/glance/+/925436 | 01:58 |
| tkajinam | croelandt, thanks and I'll check these | 02:27 |
| opendevreview | Pranali Deore proposed openstack/glance master: Replace futurist thread pool with concurrent.futures https://review.opendev.org/c/openstack/glance/+/992633 | 05:03 |
| opendevreview | Abhishek Kekane proposed openstack/python-glanceclient master: Add support for cache clean and prune APIs https://review.opendev.org/c/openstack/python-glanceclient/+/994846 | 06:19 |
| opendevreview | Takashi Kajinami proposed openstack/glance master: Drop unused management_url https://review.opendev.org/c/openstack/glance/+/973112 | 08:41 |
| opendevreview | Takashi Kajinami proposed openstack/glance master: Remove remaining logic for identity v1/v2 API https://review.opendev.org/c/openstack/glance/+/973102 | 08:41 |
| opendevreview | Takashi Kajinami proposed openstack/glance master: Use keystoneauth to get token https://review.opendev.org/c/openstack/glance/+/973195 | 08:41 |
| opendevreview | Takashi Kajinami proposed openstack/glance master: Validate enabled_import_methods https://review.opendev.org/c/openstack/glance/+/988853 | 09:07 |
| opendevreview | Maxim Sava proposed openstack/glance master: fix(rally-test): Update rally image location https://review.opendev.org/c/openstack/glance/+/994866 | 09:32 |
| opendevreview | Sahil Kumbhar proposed openstack/glance master: doc: document missing notification event types and payloads https://review.opendev.org/c/openstack/glance/+/977604 | 11:57 |
| opendevreview | Merged openstack/glance master: [doc] Mention Hyper-V Enlightenments for os_type https://review.opendev.org/c/openstack/glance/+/969999 | 11:59 |
| opendevreview | Takashi Kajinami proposed openstack/glance master: Remove remaining logic for identity v1/v2 API https://review.opendev.org/c/openstack/glance/+/973102 | 12:45 |
| opendevreview | Takashi Kajinami proposed openstack/glance master: Use keystoneauth to get token https://review.opendev.org/c/openstack/glance/+/973195 | 12:46 |
| croelandt | #startmeeting glance | 14:01 |
| opendevmeet | Meeting started Thu Jun 25 14:01:12 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 | #link https://etherpad.openstack.org/p/glance-team-meeting-agenda | 14:01 |
| croelandt | #topic roll call | 14:01 |
| abhishekk | o/ | 14:01 |
| croelandt | I think pdeore is out | 14:01 |
| mhen | o/ | 14:01 |
| abhishekk | yeah | 14:01 |
| jlejeune | o/ | 14:02 |
| croelandt | maybe rosmaita and dansmith are here | 14:02 |
| jlejeune | hello there | 14:02 |
| croelandt | jlejeune: hey, I added your topic at the end of the agenda, but the meeting should be quick :) | 14:02 |
| dansmith | I'm in another meeting, but kinda here | 14:02 |
| rosmaita | (i am in another meeting) | 14:02 |
| croelandt | #topic periodic jobs | 14:02 |
| croelandt | all good | 14:02 |
| croelandt | #topic S3 credentials | 14:02 |
| croelandt | #link https://review.opendev.org/c/openstack/glance/+/988385 | 14:02 |
| croelandt | I'll have to review this again, and we'll merge after releasing glance_store for M2 next week | 14:02 |
| opendevreview | Anatoli Barzev proposed openstack/glance master: Fix and refactor S3 URL parsing/constructing causing constant DB updates https://review.opendev.org/c/openstack/glance/+/994898 | 14:03 |
| abhishekk | sounds good | 14:03 |
| croelandt | #topic Parallel image import | 14:03 |
| croelandt | #link https://review.opendev.org/c/openstack/glance/+/989942 | 14:03 |
| croelandt | This I could re-review today :) | 14:03 |
| abhishekk | yeah | 14:03 |
| croelandt | or tomorrow when it's less hot | 14:03 |
| abhishekk | ack | 14:03 |
| croelandt | #topic Half deleted images | 14:03 |
| croelandt | jlejeune: you're up :) | 14:03 |
| croelandt | told you it'd be quick | 14:03 |
| croelandt | we're cruising | 14:03 |
| jlejeune | :) | 14:04 |
| jlejeune | what to say, I just discovered that bug opened since few years now, that I encountered on our side | 14:04 |
| abhishekk | SO we have this, https://review.opendev.org/c/openstack/glance/+/785552 | 14:05 |
| croelandt | so to sum it up | 14:05 |
| croelandt | sometimes images get deleted, but the domain delete and the db delete are not atomic | 14:05 |
| croelandt | so we can end up in a weird situation | 14:05 |
| abhishekk | this runs on service start | 14:05 |
| croelandt | abhishekk: yeah but you need to restart glance-api? | 14:05 |
| abhishekk | it takes care of orphan file from the staging area | 14:06 |
| croelandt | Can this run as a cron job? | 14:06 |
| abhishekk | yeah, because how much of this is really possible in production that image gets deleted while import is in progress? | 14:06 |
| mklejn_ | abhishekk: we don't want to restart glances too often, that would require connection draining and it's PITA to do when you have 200+ concurrent uploads | 14:06 |
| croelandt | abhishekk: not sure, but people seem to hit that | 14:07 |
| croelandt | could we clean up the staging area on a regular basis? | 14:07 |
| croelandt | sysadmins don't want to restart stuff, this is not Windows | 14:07 |
| abhishekk | that should be another topic for discussion I guess | 14:07 |
| croelandt | there is alos https://review.opendev.org/c/openstack/glance/+/925436 | 14:07 |
| abhishekk | I have pointed out what we have currently for the problem | 14:07 |
| croelandt | that has the same idea of wanting to clean up properly rather than relying on later housekeeping | 14:08 |
| croelandt | so maybe these could be discussed together for PTG? | 14:08 |
| abhishekk | I think so | 14:08 |
| croelandt | It's not critical because we can always cleanup, but 1) It would be amazing to do the right thing right away 2) Not having to restart the service to run housekeeping would be cool | 14:09 |
| jlejeune | agree | 14:09 |
| croelandt | I'll try to figure out if we can do 2) already | 14:09 |
| abhishekk | another option is we can have CLI based solution like glance-manage clean staging etc? | 14:09 |
| croelandt | yeah, so you could have that as a cron | 14:10 |
| croelandt | but imagine you're a sysadmin, you'd expect Glance to do the right thing :) | 14:10 |
| abhishekk | ack, may be I should become sysadmin :P | 14:10 |
| croelandt | it is quite fun that a lot of OpenStack devs are not sysadmins, to be honest :) | 14:11 |
| abhishekk | +1 | 14:11 |
| croelandt | #topic Open Discussion | 14:11 |
| croelandt | Anything else for today? | 14:11 |
| abhishekk | there are other patches pending for reviews as well | 14:11 |
| croelandt | yes, don't hesitate to put a list in the etherpad | 14:12 |
| abhishekk | cache clean and prune api calls and client support | 14:12 |
| croelandt | I've been doing a lot of reviews on tkajinam's patches that have been there for a while | 14:12 |
| abhishekk | ack, I will put it in next week | 14:12 |
| croelandt | oh I looked at those recently | 14:12 |
| mhen | there are a few open threads on the image encryption patchset | 14:12 |
| mhen | https://review.opendev.org/c/openstack/glance/+/926295?tab=comments | 14:12 |
| mhen | mostly by dansmith or croelandt | 14:12 |
| mhen | which I'd like to conclude soon | 14:12 |
| abhishekk | may be you can approve this quickly https://review.opendev.org/c/openstack/glance/+/993942 | 14:13 |
| croelandt | mhen: ok I'll check my comment | 14:13 |
| mhen | thank you! | 14:13 |
| croelandt | dansmith: If oyu got a bit of time, please answer mhen's comments on the encryption patch | 14:13 |
| croelandt | abhishekk: ok will do | 14:13 |
| abhishekk | cool, thank you | 14:13 |
| dansmith | croelandt: yeah, sorry, been a busy couple of weeks here | 14:14 |
| croelandt | I know :-( | 14:14 |
| dansmith | I'm also out week after next, so I need to try to do that this or next week | 14:14 |
| croelandt | ok | 14:14 |
| croelandt | I guess that's all | 14:14 |
| croelandt | thanks for joining, I'll go back to melting in the heatwave in front of gerrit | 14:15 |
| abhishekk | thanks | 14:15 |
| croelandt | see you next week! | 14:15 |
| croelandt | #endmeeting | 14:15 |
| opendevmeet | Meeting ended Thu Jun 25 14:15:19 2026 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 14:15 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/glance/2026/glance.2026-06-25-14.01.html | 14:15 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/glance/2026/glance.2026-06-25-14.01.txt | 14:15 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/glance/2026/glance.2026-06-25-14.01.log.html | 14:15 |
| *** vhari_ is now known as vhari | 15:01 | |
| opendevreview | Takashi Kajinami proposed openstack/glance master: Do not add version element to identity url https://review.opendev.org/c/openstack/glance/+/994932 | 15:45 |
| croelandt | amorin: I tried reviewing https://review.opendev.org/c/openstack/glance/+/967371 , honestly I think we'll need someone who understands Keystone :) | 15:52 |
| amorin | oh yes! | 15:53 |
| amorin | we are running this in prod since months, maybe years now | 15:53 |
| amorin | the thing with this is that, it make glance create a lot of trusts | 15:53 |
| amorin | which is what was expected by the initial code, but got broken due to the missing of option registration | 15:54 |
| amorin | about the int vs bool, I will double check, this is a good point | 15:54 |
| croelandt | the thing is, I don't doubt this works for you | 15:55 |
| croelandt | It just looks like a really weird fix | 15:55 |
| croelandt | so my concern here is more about long-term maintainability for the Glance team :) | 15:55 |
| croelandt | Do you define these options in glance-api.conf? | 15:55 |
| amorin | let me check, but I think I did not | 15:57 |
| amorin | confirmed, not set | 15:58 |
| croelandt | yeah I really don't want us to end up manually adding options that are part of other projects | 15:58 |
| croelandt | and seem a bit... unrelated? | 15:59 |
| croelandt | I'll try to find someone from Keystone who can help | 15:59 |
| amorin | don't you have the no such option error in logs on your deployments? | 15:59 |
| croelandt | I don't deploy OpenStack :) | 16:00 |
| croelandt | Red Hat customers have not reported it, as far as I know | 16:00 |
| croelandt | It happens to you when image upload takes a long time, right? With that Swift option regarding token expiration? | 16:01 |
| amorin | glance-664d55cc96-whcr8 glance 2026-06-25 16:01:46.761 1 INFO glance.api.v2.image_data [None req-7776d0de-3c75-4b79-bacc-bb446a1a6a57 527bed9674a04141a1a3004066113db6 352f8325e15b4f75a492df51313c9a4c - - default default] Unable to create trust: no such option collect_timing in group [keystone_authtoken] Use the existing user token. | 16:02 |
| amorin | something like this | 16:02 |
| amorin | it's happening on each image upload | 16:02 |
| amorin | even for small ones | 16:02 |
| amorin | it does not prevent the image to be uploaded correctly | 16:02 |
| croelandt | oh | 16:02 |
| croelandt | oh ok it uses the existing user token | 16:02 |
| croelandt | missed that | 16:02 |
| * croelandt is on 4 hours of sleep per night this week | 16:03 | |
| amorin | but yes, if the image is very big, then, it may finish after the token expired | 16:03 |
| croelandt | but that's a different issue | 16:03 |
| amorin | which is then failing to store the image | 16:03 |
| croelandt | ok honestly I don't understand anything about Keystone | 16:03 |
| croelandt | so tomorrow I'm gonna try to figure out if someone can help with that | 16:03 |
| amorin | ack, thanks | 16:03 |
| opendevreview | Merged openstack/glance master: Remove glance-grenade-centralized-cache zuul job https://review.opendev.org/c/openstack/glance/+/993942 | 16:41 |
| opendevreview | Anatoli Barzev proposed openstack/glance master: Fix and refactor S3 URL parsing/constructing causing constant DB updates https://review.opendev.org/c/openstack/glance/+/994898 | 23:05 |
Generated by irclog2html.py 4.1.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!