| *** mhen_ is now known as mhen | 02:17 | |
| opendevreview | Max proposed openstack/python-glanceclient master: feat: multipart_data download https://review.opendev.org/c/openstack/python-glanceclient/+/969491 | 10:48 |
|---|---|---|
| mhen | hello | 13:14 |
| mhen | quick question concerning the image encryption spec: | 13:15 |
| mhen | currently, we aimed to introduce `os_decrypt_size` | 13:15 |
| mhen | see https://review.opendev.org/c/openstack/glance-specs/+/964755/3/specs/2026.1/approved/glance/standardized_image_encryption.rst#109 | 13:15 |
| mhen | while looking at how to process this in Cinder I stumbled upon `virtual_size`: https://answers.launchpad.net/glance/+question/683437 | 13:16 |
| mhen | do you think we could drop `os_decrypt_size` from the spec and simply refer to `virtual_size` to represent the resulting decrypted size of the image? | 13:17 |
| mhen | Cinder already has handling to check this against the volume size, so that would simplify things | 13:18 |
| mhen | as I understand the Launchpad answer there don't seem to be many cases where `virtual_size` is used currently | 13:19 |
| mhen | oh wait | 13:25 |
| mhen | turns out that property is read-only, you can't specify it as a client when creating an image | 13:25 |
| mhen | I was misled by "The virtual_size can be populated by hand if you want to give it a value" in the Launchpad post but turns out this is not true | 13:26 |
| mhen | bummer | 13:26 |
| abhishekk | mhen: yes, virtual size is read only, however now you can specify size while creating the image | 13:27 |
| mhen | abhishekk: you mean min-disk? | 13:28 |
| abhishekk | no | 13:28 |
| abhishekk | --size | 13:29 |
| mhen | "ERROR: --size was given, which is an Image v1 option that is no longer supported in Image v2" | 13:29 |
| mhen | that is what my client tells me | 13:30 |
| mhen | don't see it in https://docs.openstack.org/api-ref/image/v2/#create-image either | 13:30 |
| mhen | I believe Glance calculates it on its own when consuming the chunks, no? | 13:30 |
| abhishekk | mhen: which version are you using? | 13:34 |
| abhishekk | https://review.opendev.org/q/topic:%22spec/set-size-on-upload%22+status:merged | 13:36 |
| mhen | I'm running aDevStack on master from Nov 14th | 13:37 |
| mhen | but anyway, I don't think that is relevant to my case | 13:37 |
| mhen | one of the patchset says "if actual image size mismatches with the size proivded in request header then the process will be aborted" | 13:38 |
| mhen | I want to encode the size an image would consume when decrypted but Glance only handles the encrypted form, so it would mismatch anyway | 13:38 |
| abhishekk | https://docs.openstack.org/api-ref/image/v2/#upload-binary-image-data | 13:38 |
| abhishekk | ack | 13:39 |
| mhen | it would only work with virtual_size I think | 13:39 |
| abhishekk | ack, we added this size matching validation in last cycle | 13:40 |
| croelandt | stephenfin: I see upload_image() is marked as deprecated in the SDK and that I should switch to using "create_image". I'm not sure I can use create_image to upload data to an existing image though | 15:33 |
| croelandt | Also I could use the import workflow, but I'd like for the data upload to work in an env where that is not configured | 15:33 |
| croelandt | any idea? | 15:33 |
| croelandt | well actually it's more of an SDK question than a glance question, I should ask taht on the sdk channel | 15:44 |
| dansmith | seems weird for upload to be deprecaeted | 15:45 |
| dansmith | I wonder if that's a misunderstanding about import being a replacement for upload | 15:45 |
| opendevreview | Fernando Ferraz proposed openstack/glance_store master: [WIP] Fix race conditions on mount/umount operations https://review.opendev.org/c/openstack/glance_store/+/967626 | 16:31 |
| opendevreview | Mitsuhiro Tanino proposed openstack/glance master: Add option to enforce mandatory image properties https://review.opendev.org/c/openstack/glance/+/969967 | 16:33 |
| opendevreview | Merged openstack/glance stable/2025.2: Fix FileNotFoundError when registering task operations https://review.opendev.org/c/openstack/glance/+/969017 | 19:24 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/glance master: [doc] Mention Hyper-V Enlightenments for os_type https://review.opendev.org/c/openstack/glance/+/969999 | 20:34 |
Generated by irclog2html.py 4.0.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!