| opendevreview | Maxim Sava proposed openstack/glance master: Remove admin role from service_api policy https://review.opendev.org/c/openstack/glance/+/1001324 | 10:53 |
|---|---|---|
| fprzewozn | croelandt would you be able to have a look at https://review.opendev.org/c/openstack/glance/+/1005674 ? | 13:36 |
| croelandt | fprzewozn: it's on my list of topics for today's meeting | 14:00 |
| croelandt | #startmeeting glance | 14:00 |
| opendevmeet | Meeting started Thu Sep 17 14:00:38 2026 UTC and is due to finish in 60 minutes. The chair is croelandt. Information about MeetBot at http://wiki.debian.org/MeetBot. | 14:00 |
| opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 14:00 |
| opendevmeet | The meeting name has been set to 'glance' | 14:00 |
| croelandt | #link https://etherpad.openstack.org/p/glance-team-meeting-agenda | 14:00 |
| croelandt | #topic roll call | 14:00 |
| croelandt | o/ | 14:00 |
| abhishekk_ | o/ | 14:01 |
| pdeore__ | o/ | 14:01 |
| *** pdeore__ is now known as pdeore | 14:01 | |
| sebkro | o/ | 14:01 |
| croelandt | #topic periodic jobs | 14:02 |
| croelandt | all good \o/ | 14:02 |
| croelandt | #topic stable patches | 14:02 |
| croelandt | #link https://review.opendev.org/c/openstack/glance/+/1005674 | 14:02 |
| croelandt | abhishekk_: this seems fairly simple, could you take a look at it? | 14:02 |
| abhishekk_ | I will take a look later today | 14:03 |
| croelandt | ok thanks | 14:04 |
| croelandt | #topic Easy patches | 14:04 |
| croelandt | #link https://review.opendev.org/c/openstack/glance/+/1005913 this is the DB migration patch | 14:04 |
| croelandt | I could push it myself, but... :) | 14:04 |
| croelandt | #link https://review.opendev.org/c/openstack/glance/+/1003982 <- And I think you can abandon this, abhishekk_ ? | 14:04 |
| abhishekk_ | Will approve it | 14:05 |
| abhishekk_ | Ack | 14:05 |
| croelandt | thanks | 14:06 |
| croelandt | #topic PTG | 14:06 |
| croelandt | #link https://etherpad.opendev.org/p/2027.1-ptg-glance-planning | 14:06 |
| croelandt | if you want something discussed at PTG, this is becoming urgent :) | 14:07 |
| croelandt | #topic Open Discussion | 14:07 |
| croelandt | anything else for today? | 14:07 |
| abhishekk_ | Yes | 14:07 |
| dansmith | I'm here (but have nothing) :) | 14:07 |
| seunghunlee | Hello. I'm new here. Can I ask a question? | 14:07 |
| abhishekk_ | Cycle highlights? | 14:07 |
| croelandt | abhishekk_: oh I sent the patch | 14:07 |
| croelandt | https://review.opendev.org/c/openstack/releases/+/1005912 | 14:07 |
| croelandt | it was merged | 14:07 |
| abhishekk_ | Gr8 | 14:07 |
| croelandt | seunghunlee: once abhishekk_ is done :) | 14:07 |
| croelandt | I think abhishekk_ is done | 14:08 |
| croelandt | seunghunlee: you have hte mic | 14:08 |
| abhishekk_ | Yeah | 14:08 |
| seunghunlee | Thanks. is Glance team interested in the feature that adds option to prevent images from shelved instances? | 14:08 |
| seunghunlee | I'm currently planning the development but wanted to check with the community first | 14:09 |
| abhishekk_ | What’s the usecase? | 14:09 |
| croelandt | prevent images from shelved instances from doing what? | 14:09 |
| dansmith | yeah I'm confused :) | 14:09 |
| seunghunlee | Oops sorry didn;t finish the sentence. I meant prevent deleting the images that are the snapshot of the shelved instances | 14:09 |
| croelandt | seunghunlee: so is this useful in one of your deployments? | 14:09 |
| seunghunlee | Not one, many deployments but yes this would be useful preventing operators making mistakes | 14:10 |
| dansmith | seunghunlee: what's the mechanism by which they will be prevented from deletion? | 14:11 |
| seunghunlee | Currently my idea is to check properties of a image to identify if the image is a snapshot of shelved instance. Raise the exception if that's the case. This may involve Nova patch too (To add properties to snapshot images on shelving for example). | 14:14 |
| seunghunlee | But there's no firm mechanism at this time. | 14:16 |
| dansmith | so some special property that nova sets on the image to prevent deletion? | 14:17 |
| dansmith | nova uses the user's token to create and delete images, so the user would be able to remove the " | 14:17 |
| dansmith | special" property as well | 14:17 |
| seunghunlee | Hmm that's true | 14:18 |
| dansmith | if it's just an extra step to help prevent deletion, that's fine I guess, it just needs to be clear that it's something the user could arbitrarily remove | 14:18 |
| dansmith | many of the other services have a "locked-by: nova for shelved instance $uuid" kind of thing, so this would be similar | 14:19 |
| dansmith | TBH, I've not heard of this being a problem but I'm not necessarily opposed in principle | 14:19 |
| croelandt | so if other services have a way to warn users, we can probably reuse that mechanism | 14:20 |
| dansmith | no standard mechanism that I know of, they just have similar ideas | 14:20 |
| seunghunlee | I'll have a look at other services and see if similar can be done in Glance | 14:20 |
| dansmith | so like manila has a way for nova to lock a share as "you can't delete this because it's in use by nova instance $uuid" | 14:21 |
| croelandt | dansmith: and similarly, the user can work around this? | 14:21 |
| dansmith | croelandt: I don't think so because I think we use nova's service credentials to set it, but it's a full separate api not just a property among other properties | 14:22 |
| dansmith | I guess I'd prefer we not say "service users can set/delete that property" for something like this | 14:22 |
| dansmith | the user owns the instance and the image and they should be able to delete it at any time, IMHO | 14:22 |
| croelandt | ok | 14:22 |
| dansmith | a second step that is sort of "you know this belongs to an instance and you'll break it if you delete this right?" step may be okay | 14:22 |
| croelandt | seunghunlee: I think you should put a topic in the PTG etherpad I linked above | 14:22 |
| croelandt | and try to look into the mechanisms dansmith is discussing, see if that can help | 14:23 |
| seunghunlee | Yeah that sounds sensible. I'll write some on the etherpad | 14:23 |
| seunghunlee | I think that's it for now. Thank you. | 14:24 |
| croelandt | ok | 14:26 |
| dansmith | seunghunlee: when you come to ptg to talk about it, can you have some real-world use cases? like users deleting those images because they don't recognize them, or think they are not needed, or .. etc? | 14:26 |
| croelandt | well thanks everyone for joining | 14:26 |
| croelandt | yes | 14:26 |
| croelandt | real use case | 14:26 |
| seunghunlee | Yes, sure | 14:27 |
| croelandt | and imagine the Glance devs do not run their own OpenStack cluster for fun on the weekend | 14:27 |
| dansmith | all of those things are possible I'm sure but hearing about actual cases or why people are confused would be interesting | 14:27 |
| croelandt | (shocking, I know) | 14:27 |
| dansmith | seunghunlee: thanks! | 14:27 |
| croelandt | See you all next week! | 14:27 |
| croelandt | #endmeeting | 14:27 |
| opendevmeet | Meeting ended Thu Sep 17 14:27:43 2026 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 14:27 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/glance/2026/glance.2026-09-17-14.00.html | 14:27 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/glance/2026/glance.2026-09-17-14.00.txt | 14:27 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/glance/2026/glance.2026-09-17-14.00.log.html | 14:27 |
| opendevreview | Thomas Goirand proposed openstack/python-glanceclient master: Support Python 3.15 hashlib error message change https://review.opendev.org/c/openstack/python-glanceclient/+/1006063 | 15:09 |
| opendevreview | Merged openstack/glance master: Use common environment classifier https://review.opendev.org/c/openstack/glance/+/1004175 | 16:40 |
| opendevreview | Merged openstack/glance stable/2025.2: Fix glance service policy rule https://review.opendev.org/c/openstack/glance/+/1005674 | 18:14 |
Generated by irclog2html.py 4.1.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!