| *** tkajinam_ is now known as tkajinam | 10:13 | |
| *** rosmaita1 is now known as rosmaita | 12:56 | |
| opendevreview | Sebastian Krott proposed openstack/glance_store stable/2025.2: Fix final chunk's content_length for Swift image uploads. https://review.opendev.org/c/openstack/glance_store/+/977319 | 13:28 |
|---|---|---|
| opendevreview | Sebastian Krott proposed openstack/glance_store stable/2025.2: Fix final chunk's content_length for Swift uploads https://review.opendev.org/c/openstack/glance_store/+/977319 | 13:29 |
| croeland1 | #startmeeting glance | 14:00 |
| opendevmeet | Meeting started Thu Feb 19 14:00:07 2026 UTC and is due to finish in 60 minutes. The chair is croeland1. 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 |
| croeland1 | #link https://etherpad.openstack.org/p/glance-team-meeting-agenda | 14:00 |
| croeland1 | #topic roll call | 14:00 |
| croeland1 | o/ | 14:00 |
| pdeore | o/ | 14:01 |
| croeland1 | is it only us? :) | 14:01 |
| pdeore | I thought it's canceled, did see anything in the agenda :P | 14:01 |
| pdeore | yeah | 14:01 |
| rajiv_ | hi | 14:01 |
| croeland1 | pdeore: yeah i'm lazy and write the agenda at the very last minute | 14:01 |
| croeland1 | no rajiv is here! | 14:01 |
| croeland1 | I think rosmaita is in another call | 14:01 |
| croeland1 | there is not muc hto the agenda anyway | 14:02 |
| pdeore | yeha | 14:02 |
| croeland1 | The main thing this week was merging a ton of store patches and releasing M2 | 14:02 |
| croeland1 | so congrats to us on that \o/ | 14:02 |
| croeland1 | #topic Release/periodic job updates | 14:02 |
| pdeore | \o/ | 14:02 |
| croeland1 | All good \o/ | 14:02 |
| croeland1 | #topic Important stable patches | 14:02 |
| croeland1 | so we have a few stable patches in the queue, ready to be merged in Glance | 14:02 |
| croeland1 | but there is a failure in the CI that will be resovled by | 14:03 |
| croeland1 | #link https://review.opendev.org/c/openstack/requirements/+/976227 | 14:03 |
| croeland1 | I'm keeping an eye on this and once it merges I'll recheck other patches | 14:03 |
| croeland1 | #topic eventlet patches | 14:03 |
| croeland1 | we merged all the eventlet patches under review, we have very few uses of eventlet left, so we're almost there | 14:03 |
| croeland1 | I'll try to talk to hberaud about this again and see what we can do | 14:04 |
| croeland1 | #topic PTG | 14:04 |
| croeland1 | #link https://etherpad.opendev.org/p/2026.2-ptg-glance-planning | 14:04 |
| croeland1 | You can add topics there ^ | 14:04 |
| croeland1 | I see Abhishek has been working on this | 14:04 |
| croeland1 | #topic Open Discussion | 14:04 |
| croeland1 | And do we have anything else? | 14:04 |
| rajiv_ | yes | 14:05 |
| pdeore | nothing from me | 14:05 |
| rajiv_ | i upgraded to flamingo with swift backend | 14:05 |
| rajiv_ | the last chunk fails to update, i receive a 499 | 14:05 |
| croeland1 | Interesting | 14:05 |
| rajiv_ | https://review.opendev.org/c/openstack/glance_store/+/977319 could this patch help fix this ? it works locally | 14:05 |
| rajiv_ | we never had this issue in dalmatian | 14:06 |
| mhen | o/ | 14:06 |
| croeland1 | so yes, but it should be submitted in master first | 14:06 |
| rajiv_ | hey abhishekk, based on yesterdays issue discussion, we tested this locally and it works : https://review.opendev.org/c/openstack/glance_store/+/977319 | 14:07 |
| rajiv_ | ah ok ok, i will change the branch | 14:07 |
| mhen | regarding image encryption, I saw dansmith and melwitt reacting to my latest comments on the glance spec yesterday and I will review and answer as soon as possible | 14:07 |
| croeland1 | $ git ll --no-merges origin/stable/2024.2..origin/stable/2025.2 glance_store/_drivers/swift/ | 14:08 |
| croeland1 | 97cbdf3 (tag: 5.1.0) Replace oslo_utils.encodeutils.exception_to_unicode | 14:08 |
| croeland1 | d71ba18 swift: Drop support for v1/v2 auth | 14:08 |
| croeland1 | 7b5817a [swift] Add size validation for image uploads | 14:08 |
| croeland1 | rajiv_: ^ these are the patches between D (2024.2) and F (2025.2) that directly affect swift | 14:08 |
| croeland1 | I wonder if anything here caused the issue | 14:08 |
| croeland1 | oh yeah | 14:08 |
| croeland1 | you mentionned 7b5817a in the commit log | 14:08 |
| croeland1 | ok so if you can confirm that reverting 7b5817a helps, yeah, we should do something about it | 14:09 |
| rajiv_ | yes, considering swift service never been updated in the past, we were looking into previous commits | 14:09 |
| rajiv_ | it works locally, strange thing is the image upload works well with glancclient and fails with openstackclient | 14:09 |
| croeland1 | ouch this is interesting | 14:10 |
| croeland1 | so is the store at fault or the client? :) | 14:10 |
| tkajinam | maybe chunk size is handled differently in client side | 14:10 |
| * tkajinam is glad that he is not blamed for his changes, seeing the other two are his. :-P | 14:10 | |
| tkajinam | it seems the problem is caused by chunk alignment (causing failure if the last chunk size does not match the fixed one) | 14:11 |
| rajiv_ | i was initially confused if it was the clients, considering a similar commit from Dan for openstackclient wasnt merged. | 14:11 |
| rajiv_ | tkajinam: yes | 14:11 |
| croeland1 | the glanceclient is going away, so... | 14:11 |
| croeland1 | so yeah please submit your fix to master, than Abhishek and I will take a look | 14:11 |
| rajiv_ | sure, thanks | 14:12 |
| croeland1 | and make sure we fix your bug and fix the bug 7b5817a was supposed to fix | 14:12 |
| croeland1 | this kind of code where we loop over chunks and handle multiple weird cases is headache-inducing | 14:12 |
| * tkajinam changed the status of the bug because its fix has not yet merged to master | 14:13 | |
| tkajinam | (fix committed indicates that the fix is merged but is not yet released | 14:13 |
| croeland1 | thaks | 14:13 |
| croeland1 | mhen: also thanks for your work on the spec | 14:13 |
| croeland1 | anything else? | 14:13 |
| croeland1 | well, thanks everyone for joining | 14:18 |
| croeland1 | see you next week | 14:18 |
| croeland1 | #endmeeting | 14:18 |
| opendevmeet | Meeting ended Thu Feb 19 14:18:06 2026 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 14:18 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/glance/2026/glance.2026-02-19-14.00.html | 14:18 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/glance/2026/glance.2026-02-19-14.00.txt | 14:18 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/glance/2026/glance.2026-02-19-14.00.log.html | 14:18 |
| *** croeland1 is now known as croelandt | 14:18 | |
| tkajinam | oh, one thing which might help you. py310 job is broken now (you may have seen POST_FAILURE) and we are waiting for new images built and deployed in CI infra | 14:19 |
| * tkajinam runs away | 14:19 | |
| opendevreview | Sebastian Krott proposed openstack/glance_store master: Fix final chunk's content_length for Swift uploads https://review.opendev.org/c/openstack/glance_store/+/977323 | 14:28 |
| opendevreview | Rajiv Mucheli proposed openstack/glance_store master: Fix final chunk's content_length for Swift image uploads. https://review.opendev.org/c/openstack/glance_store/+/977323 | 14:34 |
| *** vhari_ is now known as vhari | 14:59 | |
| dansmith | mhen: for a bootable LUKSv1-encrypted disk created by cinder, what does the actual disk layout look like? LUKS at the beginning of the disk | 15:56 |
| dansmith | ? | 15:56 |
| dansmith | just refreshing myself on the LUKS header format and.. it doesn't leave room for a bootloader that I know of, so surely a bootable disk needs to be LUKS only in a partition | 15:58 |
| dansmith | but if cinder is wrapping the disk as seen by the guest in LUKS, that can never be directly bootable (like a real machine would) and only the _contents_ are bootable if the outer layer is being stripped - correct? | 15:59 |
| mhen | dansmith: here is one example of Cinder's implementation (Ceph driver): https://github.com/openstack/cinder/blob/master/cinder/volume/drivers/rbd.py#L1127-L1135 | 16:04 |
| mhen | The output file of that qemu-img command when queried via `file ...` is: "LUKS encrypted file, ver 1 [aes, xts-plain64:sha256, sha256] ..." | 16:05 |
| mhen | LUKS is directly at the beginning | 16:05 |
| dansmith | ack, okay | 16:05 |
| mhen | The guest never sees the LUKS stuff, only its payload | 16:06 |
| dansmith | yeah, I understand that part in this scheme | 16:06 |
| mhen | Nova either uses QEMU's way to directly unwrap the LUKS during attachment (via LibVirt Secrets) in case of KVM or as a fallback, dm-crypt with /dev/mapper endpoint and passing that decrypted endpoint into the VM iirc | 16:07 |
| dansmith | yeah, I understand | 16:08 |
| dansmith | I just wanted to confirm that the layout of things is sort of the opposite of what would work on a real system.. i.e. that we don't have gpt->luks (in partition 1)->gpt | 16:09 |
| mhen | right, usually you would not use LUKS on the whole raw block device in real systems, but have partitions, raid/LVM etc. before that first | 16:10 |
| dansmith | you would/could on a data-only volume, just not on something you expected to be bootable | 16:11 |
Generated by irclog2html.py 4.0.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!