opendevreview | Abhishek Kekane proposed openstack/glance master: Update API reference to mention x-openstack-image-size header https://review.opendev.org/c/openstack/glance/+/956628 | 06:31 |
---|---|---|
croelandt | #startmeeting glance | 14:00 |
opendevmeet | Meeting started Thu Aug 7 14:00:52 2025 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 | #topic roll call | 14:00 |
croelandt | o/ | 14:01 |
abhishek_ | o/ | 14:02 |
croelandt | #link https://etherpad.openstack.org/p/glance-team-meeting-agenda | 14:05 |
croelandt | #topic Release/periodic job updates | 14:05 |
rosmaita | o/ | 14:05 |
dansmith | o | 14:05 |
croelandt | Not much change since last week, my CentOs patch still has not been merged, so our centos job is still failing :) | 14:05 |
croelandt | #topic Important stable patches | 14:05 |
croelandt | #link https://review.opendev.org/c/openstack/glance/+/953161 | 14:05 |
croelandt | this is a pretty simple stable patch, I think I could probably ninja approve it as PTL | 14:06 |
croelandt | #topic glance-gaps patches | 14:06 |
croelandt | There has been a bit of work on this topic this week | 14:06 |
croelandt | #link https://review.opendev.org/q/topic:%22glance-gaps%22%20status:open | 14:06 |
croelandt | "image tasks" and "metadef namespace import" look ready to me | 14:07 |
croelandt | thanks stephenfin for the reviews! | 14:07 |
croelandt | I don't think we'll merge a lot in time for tomorrow's release of OSC, but that's ok | 14:07 |
croelandt | #topic Open Discussion | 14:07 |
croelandt | Anything else we'd like to discuss this week? | 14:08 |
abhishek_ | @croelandt https://review.opendev.org/c/openstack/glance/+/948531 one more for you to review and approve | 14:08 |
croelandt | Maybe encryption | 14:08 |
croelandt | abhishek_: ok I'm keeping this open and will review it after the meeting | 14:08 |
abhishek_ | cool, thank you, I will then compile and see which tests are still needs to be migrated and work accordingly | 14:09 |
croelandt | thanks a lot for that | 14:09 |
croelandt | I think we could also review mhen's patches | 14:09 |
abhishek_ | np :) | 14:09 |
abhishek_ | ack, will also have a look | 14:10 |
mhen | I'd appreciate it :) | 14:10 |
croelandt | hehe | 14:10 |
mhen | dansmith: did you have the chance to look at https://review.opendev.org/c/openstack/nova/+/926326 yet? | 14:10 |
dansmith | no, it's probably still in my tab stack, sorry | 14:14 |
croelandt | we all have that "tab stack" :D | 14:15 |
croelandt | Ok I'll be away for the next two weeks | 14:15 |
croelandt | I'll let you self-organize :) | 14:16 |
croelandt | Anything else? | 14:16 |
abhishek_ | croelandt: what about releases? | 14:19 |
croelandt | M3 is on the 28th | 14:19 |
croelandt | and I'll be back on the 25th | 14:19 |
abhishek_ | when is the release for non-client libs and python-glanceclient? | 14:19 |
croelandt | so I can make the right patch | 14:19 |
croelandt | #link https://releases.openstack.org/flamingo/schedule.html | 14:19 |
croelandt | I wrote down | 14:19 |
croelandt | 2025-08-18 Last release for non libraries | 14:19 |
croelandt | 2025-08-28 M3 | 14:19 |
croelandt | 2025-09-08 RC1 | 14:19 |
abhishek_ | Next week is release for glance_store | 14:20 |
abhishek_ | I guess you can submit that patch now before going on leave | 14:20 |
croelandt | ok I don't see it in the planning | 14:20 |
croelandt | where do you see that? | 14:20 |
abhishek_ | 2025-08-18 Last release for non libraries | 14:20 |
abhishek_ | means glance_store for us | 14:20 |
croelandt | isn't glance_store a library? :) | 14:21 |
croelandt | $ git ll 5.0.0.. | 14:21 |
croelandt | 97cbdf3 (HEAD -> master, origin/master, origin/HEAD) Replace oslo_utils.encodeutils.exception_to_unicode | 14:21 |
croelandt | fb6d454 (gerrit/master) Imported Translations from Zanata | 14:21 |
abhishek_ | non client libraries it says | 14:21 |
croelandt | I only see these 2 patches though | 14:22 |
abhishek_ | Aug 18 - Aug 22 R-6 Final release for non-client libraries | 14:22 |
croelandt | oops | 14:22 |
abhishek_ | 14:22 | |
croelandt | yeah | 14:22 |
croelandt | ok so do we want a release for these 2 patches? | 14:22 |
rosmaita | croelandt: this may be helpful: https://review.opendev.org/dashboard/?title=Glance+Project+Releases&foreach=project:openstack/releases+(message:glance+OR+message:glance_store+OR+message:glanceclient)&Pending%20Release%20Patches=status:open&Merged+Release+Patches+(Last+4+Months)=status:merged+-age:4months | 14:22 |
dansmith | PTL'd | 14:22 |
abhishek_ | yes, we need final releasse | 14:22 |
croelandt | ok I'll send a patch for 5.0.1 I guess | 14:23 |
croelandt | is that fine? | 14:23 |
abhishek_ | yep | 14:25 |
croelandt | ok then | 14:25 |
croelandt | rosmaita: thanks for the link | 14:26 |
croelandt | tehre are 2024.1 unmerged patches... | 14:26 |
croelandt | gonna have to look at those :) | 14:26 |
croelandt | OK I think that was all | 14:26 |
croelandt | thanks everyone for joining | 14:26 |
croelandt | #endmeeting | 14:26 |
opendevmeet | Meeting ended Thu Aug 7 14:26:47 2025 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 14:26 |
opendevmeet | Minutes: https://meetings.opendev.org/meetings/glance/2025/glance.2025-08-07-14.00.html | 14:26 |
opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/glance/2025/glance.2025-08-07-14.00.txt | 14:26 |
opendevmeet | Log: https://meetings.opendev.org/meetings/glance/2025/glance.2025-08-07-14.00.log.html | 14:26 |
croelandt | abhishek_: so what was the issue with test_metadef_dont_accept_illegal_bodies? | 14:55 |
abhishek_ | I need to recollect it from the git history | 14:57 |
abhishek_ | croelandt: https://bugs.launchpad.net/glance/+bug/1475647 | 14:59 |
croelandt | hm ok and SynchronousAPIBase cannot handle that | 14:59 |
abhishek_ | yeah, you can see it was using wsgi, https://review.opendev.org/c/openstack/glance/+/207150/13/glance/common/wsgi.py | 15:00 |
croelandt | abhishek_: https://review.opendev.org/c/openstack/releases/+/956802 | 15:53 |
abhishek_ | croelandt: ack | 15:53 |
abhishek_ | I think you will get comment about the version number, it should be 5.1.0 | 15:54 |
croelandt | really? | 15:54 |
croelandt | yeah ok | 15:54 |
croelandt | honestly there is basically nothing to release so it's neither a minor release nor a patch one :D | 15:55 |
croelandt | updated :) | 15:55 |
abhishek_ | agree, but as it is final release of the cycle it should be 5.1.0 | 15:55 |
abhishek_ | thanks, verified hash | 15:57 |
croelandt | ok ok | 15:57 |
JayF | fwiw that's so backports can have room for version increments | 16:13 |
opendevreview | Merged openstack/glance master: Functional tests migration for metadef APIs https://review.opendev.org/c/openstack/glance/+/948531 | 16:20 |
stephenfin | croelandt: I think I've reviewed all of those SDK/OSC changes now. | 16:31 |
stephenfin | Approved one, and looked for small changes on two. I don't think we should proceed with the final one | 16:31 |
stephenfin | (final one being https://review.opendev.org/c/openstack/python-openstackclient/+/904345) | 16:32 |
croelandt | JayF: makes sense | 17:12 |
croelandt | ok so you merged "image_tasks" in the SDK | 17:14 |
croelandt | let me look at the client | 17:14 |
croelandt | Ok so regarding https://review.opendev.org/c/openstack/python-openstackclient/+/904345 , people could not use a JSON file directly, but would rather use the create command and get the fields using jq | 17:26 |
croelandt | makes sense to me | 17:26 |
croelandt | abhishek_: does that sound fine to you? | 17:26 |
Generated by irclog2html.py 4.0.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!