| ges | Good day! We seem to be hitting this issue: https://bugs.launchpad.net/glance/+bug/1983279 where the format_inspector doesn't want to "streamOptimized" VMDK. What would you all think about processing the streamOptimized VMDKs in the same way we handle the monolithicSparse variant? I ran a quick test with some images and it seems to agree with the size calculated by qemu-img. | 13:40 |
|---|---|---|
| ges | doesn't want to handle* | 13:41 |
| rosmaita | ges: i'm not sure why we support monolithicSparse format only ... your suggestion makes sense, maybe put it on the agenda for the next glance meeting? | 13:55 |
| rosmaita | ges: https://etherpad.opendev.org/p/glance-team-meeting-agenda | 13:58 |
| rosmaita | ges: make sure you mention that streamOptimized is a monolithic format (at least i think it is ... docs say "These disks are currently used only in the monolithic form") | 14:02 |
| rosmaita | also, make sure the docs i'm talking about are current: https://www.vmware.com/app/vmdk/?src=vmdk | 14:03 |
| ges | rosmaita: I read the same doc, and I think the same. I don't understand why we care about getting the variant from the descriptor for getting the virtual size. The virtual size seems to be one of the first things we can read from the VMDK file. | 15:44 |
| rosmaita | ges: i think the key issue is that the file needs to be monolithic so that each file is stand-alone | 15:45 |
| ges | rosmaita: I think we could add a check there, since we have acces to the "descriptor", we could ensure the VMDK consists of only one extend. I am not very familiar with of VMDK works in OpenStack, but I imagine it's not really possible to use a non-monolithic VMDK for example in Nova. | 15:50 |
| rosmaita | ges: that sounds about right, but the issue for Glance is that we don't want a user to upload an unusable image | 15:51 |
| ges | rosmaita: yeah, that would be strange. I added the item to the agenda for Thursday meeting, I think I could also quickly come-up with a patche that accepts streamOptimized and ensure we are working on a monolithic image. | 15:55 |
| rosmaita | ges: that sounds good | 15:55 |
| *** sfinucan is now known as stephenfin | 16:10 | |
| opendevreview | Brian Rosmaita proposed openstack/glance stable/yoga: [stable-only] Remove glance-code-constants-check https://review.opendev.org/c/openstack/glance/+/863622 | 18:06 |
| opendevreview | Dan Smith proposed openstack/glance master: DNM: Test cinder target devstack change https://review.opendev.org/c/openstack/glance/+/863926 | 19:20 |
| opendevreview | Brian Rosmaita proposed openstack/glance stable/xena: [stable-only] Remove glance-code-constants-check https://review.opendev.org/c/openstack/glance/+/863943 | 21:55 |
| opendevreview | Brian Rosmaita proposed openstack/glance stable/wallaby: [stable-only] Remove glance-code-constants-check https://review.opendev.org/c/openstack/glance/+/863944 | 22:06 |
| opendevreview | Brian Rosmaita proposed openstack/glance stable/victoria: [stable-only] Remove glance-code-constants-check https://review.opendev.org/c/openstack/glance/+/863945 | 22:09 |
| opendevreview | Brian Rosmaita proposed openstack/glance stable/ussuri: [stable-only] Remove glance-code-constants-check https://review.opendev.org/c/openstack/glance/+/863946 | 22:12 |
| opendevreview | Merged openstack/glance stable/yoga: [stable-only] Remove glance-code-constants-check https://review.opendev.org/c/openstack/glance/+/863622 | 22:13 |
| opendevreview | Brian Rosmaita proposed openstack/glance stable/train: [stable-only] Remove glance-code-constants-check https://review.opendev.org/c/openstack/glance/+/863947 | 22:15 |
| opendevreview | Cyril Roelandt proposed openstack/python-glanceclient master: Unhardcode the value of DEFAULT_PAGE_SIZE from the tests https://review.opendev.org/c/openstack/python-glanceclient/+/824480 | 23:06 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!