| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2022-01-27 | |||
| 16:21:39 | prometheanfire | introduced in faad45b6323d7c52d35b7ccc45eacb5580b3b4d3 | |
| 16:21:49 | pmonteir | gibi: ohhh, I think I got it know... | |
| 16:22:59 | pmonteir | but shouln't it be virDomainMigrateGetMaxDowntime then? Don't know if I'm looking at the right place here > https://libvirt.org/html/libvirt-libvirt-domain.html#virDomainMigrateSetMaxDowntime | |
| 16:23:28 | pmonteir | I meant virDomainMigrateSetMaxDowntime* | |
| 16:24:45 | gibi | pmonteir: look at the naming rules at https://libvirt.org/python.html I think the python bindig call is generated to be MigrateSetMaxDowntime on teh domain object | |
| 16:25:14 | gibi | sorry migrateSetMaxDowntime | |
| 16:26:20 | prometheanfire | spatel: think this is a nova issue? (if so we should probably talk here) :D | |
| 16:27:16 | spatel | prometheanfire did you check your vm xml file and see what are the path you are seeing? | |
| 16:28:23 | opendevreview | Balazs Gibizer proposed openstack/nova master: DNM: run nova tests with any--traits placement feature https://review.opendev.org/c/openstack/nova/+/825914 | |
| 16:33:28 | prometheanfire | arch not specified, so hits the continue | |
| 16:33:46 | prometheanfire | spatel: the vm was shut down, so it's failing to boot on being defined | |
| 16:34:24 | prometheanfire | instance id is not listed in /etc/libvirt/qemu/ | |
| 16:35:41 | pmonteir | gibi: I think I got it now, thanks! | |
| 16:35:54 | prometheanfire | if I had the xml it'd be easier to debug | |
| 16:35:55 | gibi | pmonteir: cool | |
| 16:44:34 | gmann | dansmith: gibi can you check this backport centos8 nodeset usage update for stable/xena https://review.opendev.org/c/openstack/nova/+/825930 | |
| 16:45:02 | gmann | after that I will recheck on devstack patch and see if need to backport it for other stable too https://review.opendev.org/c/openstack/devstack/+/824290 | |
| 16:45:21 | dansmith | gmann: by "check" you mean "rubber stamp" right? | |
| 16:45:37 | gmann | dansmith: yes :) | |
| 16:45:55 | gibi | gmann: looks good to me, but I only have the small stamp on stable | |
| 16:46:07 | gmann | oh did not know | |
| 16:46:08 | dansmith | gibi: any concern with me ninjaing? | |
| 16:46:15 | gibi | dansmith: go for it | |
| 16:51:04 | prometheanfire | I think nova uefi booting only works with redhat now | |
| 17:11:35 | sean-k-mooney | prometheanfire: it shoudl work on ubuntu 20. | |
| 17:11:40 | sean-k-mooney | 20.04 | |
| 17:11:49 | sean-k-mooney | since that is what the dev and testign was down with | |
| 17:12:54 | prometheanfire | debian buster doesn't seem to be working at least, still trying to figure out what in particular is throwing the flag | |
| 17:13:59 | prometheanfire | sean-k-mooney: if you have suggestions on how to debug https://github.com/openstack/nova/commit/faad45b6323d7c52d35b7ccc45eacb5580b3b4d3#diff-67d0163175a798156def4ec53c18fa2ce6eba79b6400fa833a9219d3669e9a11R1465 I'd appreciate it :D | |
| 17:14:08 | prometheanfire | atm I'm about to put in a bunch of debug statements | |
| 17:16:20 | sean-k-mooney | what is the errror you are seeing | |
| 17:16:45 | prometheanfire | traceback nova.exception.UEFINotSupported: UEFI is not supported | |
| 17:17:02 | prometheanfire | which tracks down to just that get_loader function | |
| 17:17:10 | gmann | dansmith: one more easy one https://review.opendev.org/c/openstack/osc-placement/+/819203 | |
| 17:17:35 | sean-k-mooney | prometheanfire: but that is coming form use parsing libvirt | |
| 17:17:52 | sean-k-mooney | prometheanfire: so that impleis qemu is sayign secure boot is not supproted on debian | |
| 17:18:24 | sean-k-mooney | prometheanfire: can you paste the output of vrish capablities adn vrish domcaps | |
| 17:18:24 | prometheanfire | the ovmf files are there, it used to work, I'll look some more, I didn't see any errors on the libvirt-daemon side | |
| 17:18:29 | prometheanfire | k | |
| 17:18:38 | dansmith | gmann: done | |
| 17:18:54 | gmann | dansmith: thanks | |
| 17:22:29 | prometheanfire | sean-k-mooney: capabilities https://gist.github.com/prometheanfire/adb89403ef70a11301b9cc45b040cd4e | |
| 17:23:40 | sean-k-mooney | ack looks liek the loader info is not in the main caps proably dom caps | |
| 17:23:41 | prometheanfire | sean-k-mooney: domcaps https://gist.github.com/prometheanfire/49bf656542cacbf8e37c275cb08ca27b | |
| 17:23:54 | sean-k-mooney | ok yes | |
| 17:24:01 | sean-k-mooney | https://gist.github.com/prometheanfire/49bf656542cacbf8e37c275cb08ca27b#file-gistfile1-txt-L10-L25 | |
| 17:24:16 | sean-k-mooney | so ya libvirt said secure no | |
| 17:24:22 | sean-k-mooney | https://gist.github.com/prometheanfire/49bf656542cacbf8e37c275cb08ca27b#file-gistfile1-txt-L23 | |
| 17:24:30 | prometheanfire | but I don't want secure boot | |
| 17:24:33 | sean-k-mooney | because the files it looked at are not the secure boot ones | |
| 17:24:44 | sean-k-mooney | oh right | |
| 17:24:51 | prometheanfire | is nova only supporting secure boot for uefi now? | |
| 17:24:51 | sean-k-mooney | am let me reead the patch | |
| 17:24:55 | prometheanfire | ack | |
| 17:25:19 | sean-k-mooney | well yes secure boot is only supprote with uefi but no you shoudl be able to use uefi without secure boot | |
| 17:25:32 | prometheanfire | cool | |
| 17:26:30 | prometheanfire | so, domcaps says I should support uefi guests, but only without secure boot, nova is saying that's not good enough, I think | |
| 17:26:37 | prometheanfire | (to summarize) | |
| 17:28:16 | sean-k-mooney | ya still readign the code it was out of my cache | |
| 17:28:22 | sean-k-mooney | but that i belvie is incorect | |
| 17:28:31 | sean-k-mooney | we should not require secure boot unless you ask for it | |
| 17:29:02 | opendevreview | Merged openstack/osc-placement master: Updating python testing as per Yoga testing runtime https://review.opendev.org/c/openstack/osc-placement/+/819203 | |
| 17:29:46 | prometheanfire | elsewhere in debug logs it says UEFI support detected | |
| 17:32:10 | sean-k-mooney | prometheanfire: ok so i don tthink the get_loader part is the issue | |
| 17:32:20 | sean-k-mooney | it _get_loaders i think | |
| 17:32:38 | sean-k-mooney | https://github.com/openstack/nova/commit/faad45b6323d7c52d35b7ccc45eacb5580b3b4d3#diff-67d0163175a798156def4ec53c18fa2ce6eba79b6400fa833a9219d3669e9a11R97 | |
| 17:32:49 | sean-k-mooney | you have <value>/usr/share/OVMF/OVMF_CODE.fd</value> | |
| 17:33:00 | sean-k-mooney | we are checking | |
| 17:33:02 | sean-k-mooney | '/usr/share/qemu/firmware', | |
| 17:33:04 | sean-k-mooney | '/etc/qemu/firmware', | |
| 17:33:35 | sean-k-mooney | well maybe that is not the issue actuly | |
| 17:33:35 | prometheanfire | you are only checking json there though? | |
| 17:34:07 | prometheanfire | logs don't show the error message either | |
| 17:36:06 | sean-k-mooney | prometheanfire: bacicly im curently looking back in that patch to see where we determin if its supported or not | |
| 17:36:10 | sean-k-mooney | uefi that is | |
| 17:36:49 | prometheanfire | ack | |
| 17:37:02 | sean-k-mooney | prometheanfire: we really should have test that assert this behavior using xml by the way | |
| 17:37:26 | sean-k-mooney | if we dont we can use your gist as the sameple data and and see if we can repoduce | |
| 17:37:58 | prometheanfire | ack | |
| 17:43:06 | prometheanfire | I think this is a problem in OSA for xena on buster (and maybe bullseye), ovmf in debian buster-backports includes json and secboot files | |
| 17:43:13 | prometheanfire | instance booted with that | |
| 17:44:07 | prometheanfire | sean-k-mooney: https://gist.github.com/prometheanfire/1e2414828d7904daa2c568653c70cd15 | |
| 17:44:30 | prometheanfire | spatel: ^ I think ovmf needs to be installed from backports on buster at least | |
| 17:45:08 | spatel | hmm | |
| 17:47:42 | spatel | sean-k-mooney i had this issue in nova error logs when i was trying to do secure boot - nova error Secure boot requires SMM feature enabled | |
| 17:48:23 | sean-k-mooney | well SMM is enabled by default be libvirt | |
| 17:48:28 | sean-k-mooney | and we do not specify it at all | |
| 17:48:35 | sean-k-mooney | so that seams like a libvirt bug | |
| 17:48:46 | sean-k-mooney | what prometheanfire is reporting is more extream | |
| 17:49:01 | sean-k-mooney | e.g. uefi just does not work on non redhat distos | |
| 17:49:39 | EugenMayer | i have an instance which has a 'stuck image backup task' - is there any way to clean this up? | |
| 17:49:39 | prometheanfire | sean-k-mooney: I THINK that the missing json file may be a cause | |
| 17:50:02 | prometheanfire | /usr/share/qemu/firmware/60-edk2-x86_64.json in the backports version vs not in the main version | |
| 17:50:57 | sean-k-mooney | prometheanfire: ya i think kashyap had a converation with the ubuntu deves at somepoint | |
| 17:51:14 | sean-k-mooney | looking at the code it does seam to be tryign to pasrse that instead fo gettign suff from libvirt | |
| 17:51:19 | sean-k-mooney | but im not really sure | |
| 18:03:30 | opendevreview | Lee Yarwood proposed openstack/nova master: func: Allow compute_driver to be set and used by _IntegratedTestBase https://review.opendev.org/c/openstack/nova/+/764484 | |
| 18:03:31 | opendevreview | Lee Yarwood proposed openstack/nova master: libvirt: Improve creating images INFO log https://review.opendev.org/c/openstack/nova/+/826524 | |
| 18:03:31 | opendevreview | Lee Yarwood proposed openstack/nova master: block_device_info: Add swap to inline https://review.opendev.org/c/openstack/nova/+/826523 | |
| 18:03:32 | opendevreview | Lee Yarwood proposed openstack/nova master: libvirt: Remove defunct comment https://review.opendev.org/c/openstack/nova/+/826525 | |