| *** mhen_ is now known as mhen | 01:41 | |
| *** hemna9 is now known as hemna | 01:50 | |
| LarsErikP | melwitt: correct, but it failed creation because of the error I described | 06:01 |
|---|---|---|
| melwitt | LarsErikP: yeah, I think you have found a bug, unless this is some known issue that is documented somewhere. I did not find anything indicating that though | 06:03 |
| LarsErikP | Which I suspect happened because of the "require_image_format_match". Glance somehow interprets the "dummy"/empty image as raw, when nova creates it as qcow2 | 06:03 |
| LarsErikP | melwitt: I'll report a bug to glance then =) | 06:07 |
| melwitt | LarsErikP: right. glance used to not validate that the formats match (it is a relatively recent development) so in the past it went through without an error but now that glance is validating by default, it reveals the mismatch. nova does not actually set anything in the disk_format field AFAICT, it leaves it empty and maybe glance defaults to qcow2 if empty (I could be wrong). and as you found you can work around it by disabling the | 06:07 |
| melwitt | format validation. I'm not sure what/how we could fix this but it seems like a bug to me or at least if it's a known issue it should be documented somewhere | 06:07 |
| LarsErikP | yeah. I'll file a bug, and describe it as good as I can. I guess that'll help troubleshooting, and pin-pointing what's happening | 06:08 |
| melwitt | LarsErikP: it might be considered a nova bug. I'm not sure if there is a practical way nova could check the format and send the right one to glance. if we can do that then that would be the right fix I think | 06:08 |
| LarsErikP | Good point. Either that, or there is something that can be fixed in the format validation itself. I don't know enough details about how this is done. It may be worth mentioning that I tried to set the "snapshot_image_format" option in nova.conf to raw in all relevant nova-servers as well. Did not change the behaviour | 06:12 |
| LarsErikP | Which led me to belive that nova is just hardcoded to upload that "dummy" image as qcow2. I did not search in nova-code to confirm that though | 06:12 |
| melwitt | yeah, same. this is why I think nova isn't sending any disk_format to glance https://github.com/openstack/nova/blob/b99a882366251f88d145e27312b94692e0b2266f/nova/compute/api.py#L3465-L3466 | 06:13 |
| LarsErikP | ah. good catch. It's definitely not hardcoding qcow2 there ;) | 06:16 |
| LarsErikP | so.. do you think I should file the bug to nova or glance then? | 06:16 |
| melwitt | I'm thinking nova. we'll discuss it first and if we think it should be glance we can change it to glance | 06:17 |
| melwitt | I think nova is supposed to be sending the right format but it's not so clear when it's the volume-backed case where it involves the weird dummy image | 06:18 |
| LarsErikP | right. Something to chew on for the developers then.. I'll finnish my morning coffee and write up a bug report =) | 06:18 |
| melwitt | cool, thanks | 06:19 |
| LarsErikP | melwitt: https://bugs.launchpad.net/nova/+bug/2125730 =) | 06:58 |
| opendevreview | Arnaud Morin proposed openstack/nova master: Fix debug PciPassthroughFilter https://review.opendev.org/c/openstack/nova/+/907976 | 08:18 |
| opendevreview | Lajos Katona proposed openstack/nova master: Use SDK for Neutron extensions https://review.opendev.org/c/openstack/nova/+/962270 | 09:46 |
| gibi | dansmith: this is a small fix for the devstack fake-virt support due to stable-compute-uuid prevented the fake nova-compute restarts https://review.opendev.org/c/openstack/devstack/+/962300 | 12:02 |
| dansmith | gibi: would you rather just make it like ironic and not have to have the same rules? | 13:30 |
| gibi | dnaI tried that but | 13:35 |
| gibi | dansmith: I tried that but our own functional testing depends on the fake driver to test the compute_id logic | 13:36 |
| dansmith | ah okay | 13:38 |
Generated by irclog2html.py 4.0.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!