| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2022-01-27 | |||
| 15:48:17 | ade_lee__ | with my luck, I'll get rax three times in a row | |
| 15:51:05 | sean-k-mooney | ade_lee__: well staticially they used to be one of the larges ci providers so proably | |
| 15:53:48 | ade_lee__ | sean-k-mooney, seems like others have run into this -- https://review.opendev.org/c/openstack/devstack/+/779806 | |
| 15:54:21 | sean-k-mooney | ya | |
| 15:54:39 | sean-k-mooney | i was conisdering addign a env var to disable it a few weeks ago | |
| 15:55:35 | sean-k-mooney | actully did i push my one | |
| 15:55:59 | sean-k-mooney | i have been using a modifed install_pip on one of my hosts | |
| 15:57:50 | sean-k-mooney | ade_lee__: https://termbin.com/k0x9 | |
| 15:59:23 | sean-k-mooney | ade_lee__: i was modifying it to only install pip if it was not already there | |
| 16:00:38 | ade_lee__ | sean-k-mooney, makes sense -- anyways, for now added this as depends -- https://review.opendev.org/c/openstack/devstack/+/826714 - lets see how it goes | |
| 16:03:39 | pmonteir | should this be implemented? | |
| 16:03:39 | pmonteir | Hi everybody! I was checking the "live_migration_downtime" parameter and I there's a function that supposedly configures the max_downtime, but this function (migrate_configure_max_downtime in https://opendev.org/openstack/nova/src/commit/ca03fe87371d8c13fc2895f8ff3e7e1ca88cfe79/nova/virt/libvirt/guest.py) calls another one which is just a "pass", has anyone ever used this parameter? I think it's not working properly, | |
| 16:12:32 | pmonteir | ps: this function that's just a pass that I mentioned above, belongs to a "tests" folder (which makes me think that it shouldn't be used outside of a testcase). | |
| 16:15:25 | opendevreview | Balazs Gibizer proposed openstack/placement master: Add any-traits support for listing resource providers https://review.opendev.org/c/openstack/placement/+/826491 | |
| 16:15:26 | opendevreview | Balazs Gibizer proposed openstack/placement master: Remove unused compatibility code https://review.opendev.org/c/openstack/placement/+/826493 | |
| 16:15:26 | opendevreview | Balazs Gibizer proposed openstack/placement master: Add any-traits support for allocation candidates https://review.opendev.org/c/openstack/placement/+/826492 | |
| 16:15:27 | opendevreview | Balazs Gibizer proposed openstack/placement master: Add microversion 1.39 to support any-trait queries https://review.opendev.org/c/openstack/placement/+/826719 | |
| 16:17:55 | gibi | pmonteir: what you see is the libvirt test fixture we use for functional testing | |
| 16:18:30 | gibi | pmonteir: in production we use the real python binding for libvirt | |
| 16:18:32 | gibi | https://libvirt.org/python.html | |
| 16:18:44 | prometheanfire | uefi guests are failing to boot after upgrade to xena :| nova.exception.UEFINotSupported: UEFI is not supported | |
| 16:18:56 | prometheanfire | trying to find what in particular is triggering it | |
| 16:20:05 | gibi | pmonteir: so the the migrateSetMaxDowntime call is sent to libvirt | |
| 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 | 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:24 | sean-k-mooney | prometheanfire: can you paste the output of vrish capablities adn vrish domcaps | |
| 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 | sean-k-mooney | am let me reead the patch | |
| 17:24:51 | prometheanfire | is nova only supporting secure boot for uefi now? | |
| 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 | prometheanfire | you are only checking json there though? | |
| 17:33:35 | sean-k-mooney | well maybe that is not the issue actuly | |
| 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 | |