| *** mhen_ is now known as mhen | 01:45 | |
| gksk | Hi Team, I am getting below error while creating a snapshot of a vm | 12:06 |
|---|---|---|
| gksk | Glance showing OSError: unable to receive chunked part when uploading | 12:06 |
| gksk | can someone help with workaround to fix this issue | 12:06 |
| gksk | ERROR glance.common.wsgi OSError: unable to receive chunked part | 12:08 |
| gksk | Hi Team, I am getting below error while creating a snapshot of a vm | 12:21 |
| gksk | ERROR glance.common.wsgi OSError: unable to receive chunked part | 12:21 |
| gksk | can someone help with workaround to fix this issue | 12:21 |
| croelandt | #startmeeting glance | 14:00 |
| opendevmeet | Meeting started Thu Jul 31 14:00:24 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:00 |
| mhen | o/ | 14:03 |
| croelandt | abhishek_: dansmith: rosmaita: around? | 14:03 |
| croelandt | hm | 14:04 |
| croelandt | mhen: did you have anything specific to discuss regarding encryption? | 14:04 |
| mhen | just wanted to poke dansmith to review https://review.opendev.org/c/openstack/nova/+/926326 | 14:05 |
| mhen | and in general ask if anything is missing | 14:05 |
| abhishek_ | o/ | 14:06 |
| croelandt | oh abhishek_ is here \o/ | 14:06 |
| croelandt | #topic Release/periodic job updates | 14:06 |
| croelandt | #link https://review.opendev.org/c/openstack/devstack/+/937251 | 14:06 |
| croelandt | ^ is still under review and I need to figure oiut how to get reviews :) | 14:06 |
| croelandt | #topic Important stable patches | 14:07 |
| croelandt | #link https://review.opendev.org/c/openstack/glance/+/950853/10 | 14:07 |
| croelandt | abhishek_: so, about this | 14:07 |
| croelandt | we should probably do it if the bug is in other versions :) | 14:07 |
| croelandt | what branches should we target? | 14:07 |
| croelandt | 2025.1 and 2024.2? | 14:07 |
| abhishek_ | yes it was | 14:08 |
| abhishek_ | I will initiate the backport, we need to backport 2 patches in each branch | 14:09 |
| croelandt | ok, please ping me and I'll review that | 14:09 |
| croelandt | does it require other patches in Cinder/Nova? | 14:09 |
| abhishek_ | sure, thank you!! | 14:09 |
| abhishek_ | no as of now | 14:09 |
| croelandt | ok | 14:10 |
| croelandt | #topic Zuul patches | 14:10 |
| croelandt | #link https://review.opendev.org/c/openstack/python-glanceclient/+/953512 | 14:10 |
| croelandt | I got this, which should be pretty easy :) | 14:10 |
| croelandt | #topic glance-gaps patches | 14:10 |
| croelandt | I updated a few SDK patches | 14:10 |
| croelandt | #link https://review.opendev.org/q/topic:%22glance-gaps%22+status:open+project:openstack/openstacksdk | 14:10 |
| croelandt | stephenfin will help review those | 14:11 |
| croelandt | I'm not sure I can be the +2 from Glance nowthat I'm also the author :) | 14:11 |
| abhishek_ | I will have a look | 14:11 |
| croelandt | ok so now | 14:12 |
| croelandt | #topic image encryption | 14:12 |
| croelandt | mhen: I've been away, where do we stand? Are we ready to merge? | 14:13 |
| mhen | most of the patchsets have favorable reviews and I'm currently not aware of any major blockers but the Nova patchset has not received a review yet | 14:14 |
| mhen | but the Glance side does not depend on it | 14:14 |
| croelandt | ok | 14:15 |
| croelandt | abhishek_: do we try to review and merge the Glance part before the next meeting? | 14:15 |
| croelandt | does it make sense to go ahead on our side? | 14:16 |
| abhishek_ | I will have a closer look again | 14:17 |
| abhishek_ | It does look good to me though | 14:18 |
| croelandt | ok | 14:18 |
| croelandt | so let's try and get this merged | 14:18 |
| mhen | note that there is also https://review.opendev.org/c/openstack/glance/+/926905 as a separate patchset for db migration of old values | 14:18 |
| croelandt | and then d othe same in otherp rojects | 14:18 |
| croelandt | mhen: yes | 14:19 |
| croelandt | mhen: there is a fallback though, right? | 14:19 |
| mhen | yes, it's only a deprecation for now, so the old values are still fully supported | 14:19 |
| mhen | the db migration is not required for anything to work | 14:19 |
| croelandt | might as well merge it, though :) | 14:21 |
| croelandt | #topic Open Discussion | 14:21 |
| croelandt | anything else we should discuss? | 14:21 |
| mhen | no other topics from my side | 14:21 |
| croelandt | ok, thanks | 14:23 |
| croelandt | see you next week | 14:23 |
| croelandt | #endmeeting | 14:23 |
| opendevmeet | Meeting ended Thu Jul 31 14:23:25 2025 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 14:23 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/glance/2025/glance.2025-07-31-14.00.html | 14:23 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/glance/2025/glance.2025-07-31-14.00.txt | 14:23 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/glance/2025/glance.2025-07-31-14.00.log.html | 14:23 |
| mhen | thank you!# | 14:23 |
| dansmith | croelandt: sorry I got pulled away right before it started | 14:34 |
| dansmith | I assume we're too close to FF to land anything too substantial wrt to the encryption stuff, right? Also looks like it depends on a massive unmerged (but +2d) cinder change) | 14:36 |
| dansmith | I can try to take a quick pass through the review linked above | 14:36 |
| croelandt | we still have a month before M3 | 14:56 |
| croelandt | abhishek_: as our most experienced PTL, what do you think? | 14:56 |
| abhishek_ | I think FF is still 3 weeks away, if it looks solid we can merge it | 14:59 |
| abhishek_ | @dansmith Sorry to bother you, but the client release is just two weeks away. Could you please review the patch that adds the --size option to the image creation commands at your convenience? | 15:03 |
| dansmith | oye | 15:04 |
| dansmith | link me | 15:04 |
| abhishek_ | https://review.opendev.org/c/openstack/python-glanceclient/+/949787 | 15:04 |
| * dansmith never gets to do anything other than review code these days | 15:04 | |
| dansmith | that's more important than the enc spec I think | 15:04 |
| * abhishek_ enc is also important because not sure these people will find more time in upcoming cycle to work on it | 15:05 | |
| croelandt | dansmith: congrats, you now only read code like Linus & friends! | 15:38 |
| abhishek_ | dansmith: regarding your comment on popping size from the args is because scheam defined at glance API side, size is read only property, if not removed it, this will cause schema validation failure at glance side | 15:56 |
| dansmith | abhishek_: right, I'm saying you should not just pop and ignore, but fail visibly to the user if they try to provide it | 15:56 |
| abhishek_ | so you mean to say fail if no --file or data is not provided using stdin? | 15:57 |
| dansmith | no, I mean if I try to create an image with --size=3, before the upload, currently you just pop size out of the properties and create the image anyway, right? | 15:59 |
| abhishek_ | yes | 16:01 |
| abhishek_ | but here for image-create command if you pass --file then it internally calls upload image as well | 16:02 |
| abhishek_ | and before that it fails for image create call saying size is readonly | 16:02 |
| dansmith | so I'm saying don't do that.. if they provide --property size=3 to create, don't just silently ignore it, fail and say "no you can't provide size a a property" | 16:02 |
| abhishek_ | OK, so you want --size as property only for image-upload or image-stage command | 16:04 |
| dansmith | no | 16:18 |
| abhishek_ | ok, i will update the patch as per my understanding | 16:39 |
| dansmith | I am creating a devstack system so I can explain | 16:40 |
| abhishek_ | ack | 16:45 |
| opendevreview | Abhishek Kekane proposed openstack/python-glanceclient master: Add support for passing image size to Glance API https://review.opendev.org/c/openstack/python-glanceclient/+/949787 | 16:45 |
| dansmith | abhishek_: looks like you updated it to what I was expecting (still need to look) but | 17:59 |
| dansmith | this is the regression in behavior you were introducing before: | 17:59 |
| dansmith | https://paste.opendev.org/show/bnqD9dzH2LkYIbL96GH7/ | 17:59 |
| dansmith | if I passed --property size=123 before, I was properly told that's not allowed.. after your (original) patch, you would just silently ignore me and set size=None | 18:00 |
| *** tosky_ is now known as tosky | 22:50 | |
Generated by irclog2html.py 4.0.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!