| opendevreview | Cyril Roelandt proposed openstack/glance master: Remove the last reference to OsloDBDeprecationWarning https://review.opendev.org/c/openstack/glance/+/961565 | 02:14 |
|---|---|---|
| opendevreview | Koya Watanabe proposed openstack/glance master: releasenotes: Epoxy known issue for S3-compatible storage checksum behavior https://review.opendev.org/c/openstack/glance/+/958196 | 07:11 |
| opendevreview | Koya Watanabe proposed openstack/glance stable/2025.1: releasenotes: Epoxy known issue for S3-compatible storage checksum behavior https://review.opendev.org/c/openstack/glance/+/961597 | 07:26 |
| *** EugenMayer4401809 is now known as EugenMayer440180 | 08:23 | |
| frickler | croeland1: stephenfin: dansmith: please be mindful of feature freeze. like https://review.opendev.org/c/openstack/glance/+/961465 certainly shouldn't merge before branching. I'm not 100% sure about 899951, but it also seems to be very late for it right now | 08:44 |
| stephenfin | frickler: Agreed on 961465, and there's no chance of that merging before then. Per above though, I'd be eager to merge 899951 before branch since I'm concerned about releasing something that may well break within its supported lifespan. It's also very low risk given it's been done in mostly all other service projects now | 09:58 |
| croeland1 | not sure why 961465 is really unsuitable right now, but ok, I'll just look at 899951 | 13:15 |
| croeland1 | ok merged | 13:18 |
| *** croeland1 is now known as croelandt | 13:21 | |
| croelandt | Not sure if https://review.opendev.org/c/openstack/glance/+/961465 is fine to merge right now, any thoughts frickler ? | 13:22 |
| dansmith | oh I'm sorry, is glance not branched yet? | 13:29 |
| croelandt | we do not have stable/2025.2 yet | 13:33 |
| croelandt | is it something the PTL has to do? :) | 13:33 |
| frickler | croelandt: yes, review https://review.opendev.org/c/openstack/releases/+/961240 please | 13:35 |
| frickler | croelandt: IMO you should delay 961465 until the branch has been made | 13:35 |
| croelandt | ok | 13:36 |
| croelandt | stephenfin: should I branch after 899951? | 13:36 |
| croelandt | or can we just backport that patch to stable/2025.2 if needed? | 13:36 |
| stephenfin | either-or | 13:36 |
| stephenfin | whatever is easier for you | 13:36 |
| stephenfin | once it's there | 13:37 |
| croelandt | ok so I guess I'll just branch at rc1, then mertge 961465 (I got some comments though) | 13:37 |
| croelandt | if we start having failures in stable/2025.2, we'll backport 899951 | 13:38 |
| croelandt | #startmeeting glance | 14:02 |
| opendevmeet | Meeting started Thu Sep 18 14:02:06 2025 UTC and is due to finish in 60 minutes. The chair is croelandt. Information about MeetBot at http://wiki.debian.org/MeetBot. | 14:02 |
| opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 14:02 |
| opendevmeet | The meeting name has been set to 'glance' | 14:02 |
| croelandt | #topic roll call | 14:02 |
| croelandt | o/ | 14:02 |
| mhen | o/ | 14:03 |
| croelandt | #link https://etherpad.openstack.org/p/glance-team-meeting-agenda | 14:05 |
| croelandt | #topic Release/periodic job updates | 14:05 |
| croelandt | Currently our oslo-tips jobs are failing | 14:06 |
| croelandt | #link https://review.opendev.org/c/openstack/glance/+/961565 | 14:06 |
| croelandt | ^ This should fix the issue | 14:06 |
| croelandt | dansmith: ^ | 14:06 |
| croelandt | rosmaita: ^ | 14:06 |
| croelandt | if you ever got a minute :) | 14:06 |
| croelandt | #topic Open Discussion | 14:06 |
| croelandt | #link https://etherpad.opendev.org/p/2026.1-ptg-glance-planning | 14:06 |
| croelandt | PTG planning is still open | 14:06 |
| rosmaita | croelandt: will look at the tips patch | 14:07 |
| croelandt | #link https://review.opendev.org/c/openstack/glance_store/+/961488 | 14:07 |
| croelandt | rosmaita: dansmith: if you got a minute, this is a patch for S3 that looks simple enough | 14:07 |
| croelandt | rosmaita: thanks | 14:07 |
| croelandt | mhen: you have the floor | 14:07 |
| dansmith | sorry, finishing up another meeting | 14:07 |
| mhen | added a comment here: https://review.opendev.org/c/openstack/glance/+/926295/comment/e5033260_5dac08d4/ | 14:08 |
| mhen | so basically we are stuck with the image encryption until we agree on how to handle the inner disk format (encrypted payload) | 14:09 |
| mhen | my proposal is to make os_decrypt_disk_format mandatory (as dansmith suggested) but still allowing it to be set to "raw" for the time being | 14:10 |
| croelandt | for backwards-compatibility reasons? | 14:10 |
| mhen | no, because we have open questions and cinder <-> glance interaction to address in order to be able to properly validate any format within the encryption (payload) | 14:10 |
| dansmith | I'm not really prepared to have a long discussion about this right now (other meeting still going on) and probably deserves some PTG time, | 14:11 |
| dansmith | but we also really need the cinder side of this to be figured out, | 14:11 |
| dansmith | both for image encryption and the rest of the glance-as-defender work that is basically waiting for that stuff | 14:12 |
| croelandt | ok so this must be one (or multiple) sessions for PTG | 14:12 |
| croelandt | could either of you fill out the PTG planning etherpad with a detailed description of our issues? | 14:13 |
| mhen | I can do that | 14:13 |
| mhen | basically my proposal is to keep the scope of the current image encryption patchset limited and focused, implementing further hardening when the other stuff is ready in a dedicated followup patchset | 14:13 |
| dansmith | I talked to rosmaita about this last(?) week, I dunno if he has plans to write up a cross-project session for that stuff or what | 14:13 |
| croelandt | rosmaita: can you help with this? | 14:14 |
| rosmaita | yes | 14:14 |
| croelandt | We've got people who know both Glance and Cinder very well (Rajat, rosmaita for instance) but we need to make sure we know what we want to achieve | 14:15 |
| croelandt | ok so ideally we'd have a draft by next week | 14:15 |
| rosmaita | mhen: lmk when you have a description in the etherpad and i will add stuff | 14:15 |
| croelandt | we still have a few weeks before PTG but I'd rather not have to organize everything in a rush and then realize we forgot half the stuff we needed to discuss :D | 14:15 |
| rosmaita | planning ahead is good | 14:16 |
| mhen | agreed | 14:16 |
| croelandt | ok perfect | 14:18 |
| croelandt | mhen: sorry for the delay on the feature | 14:18 |
| croelandt | but we'd rather delay it and having something perfect :)à | 14:18 |
| croelandt | ok anything else? | 14:19 |
| croelandt | nope, let's close this meeting then :) | 14:21 |
| croelandt | #endmeeting | 14:21 |
| opendevmeet | Meeting ended Thu Sep 18 14:21:45 2025 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 14:21 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/glance/2025/glance.2025-09-18-14.02.html | 14:21 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/glance/2025/glance.2025-09-18-14.02.txt | 14:21 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/glance/2025/glance.2025-09-18-14.02.log.html | 14:21 |
| croelandt | fungi: hey, do you have an opinon on what the license field should be in pyproject.toml? See https://review.opendev.org/c/openstack/glance/+/961466/2/pyproject.toml#12 . I think using "{ text = ... }" is a syntax from earlier versions of the PEP | 15:11 |
| fungi | croelandt: i do, but it's a lengthy discussion with necessary citations, and i'm not around today but i flagged the change to add my feedback tomorrow | 15:30 |
| croelandt | ok looking forward to that :) | 15:39 |
| opendevreview | Merged openstack/glance master: Add pyproject.toml to support pip 23.1 https://review.opendev.org/c/openstack/glance/+/899951 | 15:48 |
| opendevreview | OpenStack Release Bot proposed openstack/glance stable/2025.2: Update .gitreview for stable/2025.2 https://review.opendev.org/c/openstack/glance/+/961727 | 16:04 |
| opendevreview | OpenStack Release Bot proposed openstack/glance stable/2025.2: Update TOX_CONSTRAINTS_FILE for stable/2025.2 https://review.opendev.org/c/openstack/glance/+/961728 | 16:04 |
| opendevreview | OpenStack Release Bot proposed openstack/glance master: Update master for stable/2025.2 https://review.opendev.org/c/openstack/glance/+/961729 | 16:04 |
| opendevreview | Merged openstack/glance master: Remove the last reference to OsloDBDeprecationWarning https://review.opendev.org/c/openstack/glance/+/961565 | 22:36 |
| opendevreview | Eimad Zamrik proposed openstack/glance master: Add configurable qemu-img process limits https://review.opendev.org/c/openstack/glance/+/952950 | 23:00 |
| opendevreview | Eimad Zamrik proposed openstack/glance master: Add configurable qemu-img process limits https://review.opendev.org/c/openstack/glance/+/952950 | 23:02 |
| opendevreview | Eimad Zamrik proposed openstack/glance master: Add configurable qemu-img process limits https://review.opendev.org/c/openstack/glance/+/952950 | 23:07 |
Generated by irclog2html.py 4.0.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!