| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2026-07-07 | |||
| 19:12:36 | sean-k-mooney | hum https://github.com/openstack/nova/blob/master/tox.ini#L142-L151 does not configure eventlet so i assume it is not disabeled by default | |
| 19:14:04 | sean-k-mooney | so not quite | |
| 19:14:06 | sean-k-mooney | https://github.com/openstack/nova/blob/master/nova/monkey_patch.py#L134-L144 | |
| 19:14:44 | sean-k-mooney | we will patch if the env var is not set because the behvior fo the new config option is to return auto | |
| 19:15:01 | sean-k-mooney | which means we take the else brnach | |
| 19:15:42 | sean-k-mooney | so until we change the default here https://github.com/openstack/nova/blob/master/nova/monkey_patch.py#L120 we still have not actully change the default in all code paths | |
| 19:16:25 | sean-k-mooney | so the unit and functional test and any tox env that does not set the evn var is currently using eventlet still | |
| 19:18:07 | gmaan | i think that was left because console proxy in this cycle still in eventlet mode as default | |
| 19:18:36 | sean-k-mooney | if that is the case then the unit test shoudl proably not be updated until next cycle | |
| 19:20:10 | sean-k-mooney | gmaan: we stil have a few test that dont pass in threaded mode as well https://github.com/openstack/nova/blob/master/threading_unit_test_excludes.txt | |
| 19:20:17 | gmaan | or we can do it for all other services | |
| 19:20:53 | sean-k-mooney | we cant have monkey patched and not in the same tox invocation | |
| 19:20:53 | gmaan | oh, i thought this list was cleared but yeah then we should wait probably | |
| 19:21:12 | sean-k-mooney | almost | |
| 19:21:35 | sean-k-mooney | if we get that to 0 it valid to swap the behvior up to RC1 | |
| 19:21:39 | sean-k-mooney | so we are not in a rush | |
| 19:22:48 | gmaan | yeah | |
| 21:14:23 | opendevreview | Merged openstack/nova master: Correct release name in concurrency_backend https://review.opendev.org/c/openstack/nova/+/996219 | |
| 21:50:36 | opendevreview | Ghanshyam Maan proposed openstack/nova master: Task tracking mechanism for graceful shutdown https://review.opendev.org/c/openstack/nova/+/996299 | |
| 22:27:15 | opendevreview | Brian Haley proposed openstack/nova master: Change neutron API calls to use project_id https://review.opendev.org/c/openstack/nova/+/994321 | |
| #openstack-nova - 2026-07-08 | |||
| 00:51:42 | opendevreview | Ghanshyam Maan proposed openstack/nova master: Fix intermittent 409 in TestGracefulShutdown tests https://review.opendev.org/c/openstack/nova/+/996381 | |
| 00:53:21 | gmaan | melwitt: sean-k-mooney ^^ i think this will fix the test_cold_migration_dest_compute_graceful_shutdown (i think revert resize test also have same issue ) failure | |
| 02:59:03 | opendevreview | Takashi Kajinami proposed openstack/nova master: Use domain capabilities to detect SEV-SNP support by libvirt/QEMU https://review.opendev.org/c/openstack/nova/+/996316 | |
| 05:22:23 | opendevreview | minwoo seo proposed openstack/nova master: Add availability_zone support for migration https://review.opendev.org/c/openstack/nova/+/976085 | |
| 05:25:23 | opendevreview | minwoo seo proposed openstack/nova master: Add availability_zone support for migration https://review.opendev.org/c/openstack/nova/+/976085 | |
| 05:31:01 | opendevreview | minwoo seo proposed openstack/nova master: Add availability_zone support for migration https://review.opendev.org/c/openstack/nova/+/976085 | |
| 05:31:36 | opendevreview | minwoo seo proposed openstack/nova master: Add availability_zone support for migration https://review.opendev.org/c/openstack/nova/+/976085 | |
| 05:32:29 | opendevreview | minwoo seo proposed openstack/nova-specs master: Add spec for cross-AZ migration support https://review.opendev.org/c/openstack/nova-specs/+/976202 | |
| 06:27:03 | opendevreview | Takashi Kajinami proposed openstack/nova master: Use domain capabilities to detect SEV-SNP support https://review.opendev.org/c/openstack/nova/+/996316 | |
| 06:29:03 | opendevreview | Takashi Kajinami proposed openstack/nova master: Use domain capabilities to detect SEV-SNP support https://review.opendev.org/c/openstack/nova/+/996316 | |
| 06:33:54 | ralonsoh | hello folks, if you have a few minutes, please check https://review.opendev.org/c/openstack/os-vif/+/995933 | |
| 06:33:57 | ralonsoh | thanks in advance! | |
| 06:34:36 | opendevreview | Takashi Kajinami proposed openstack/nova-specs master: Fix SEV-SNP feature detection in start up https://review.opendev.org/c/openstack/nova-specs/+/996394 | |
| 06:39:22 | opendevreview | Takashi Kajinami proposed openstack/nova-specs master: Use domain capability to detect SEV-SNP support https://review.opendev.org/c/openstack/nova-specs/+/996395 | |
| 08:13:18 | gibi | gmaan: sean-k-mooney: re threading as default in our unit test jobs. I'm OK in general to switch. The remaining disable unit tests needed DB fixture changes currently being developed in the functional test series. | |
| 08:37:34 | opendevreview | minwoo seo proposed openstack/nova master: Add availability_zone support for migration https://review.opendev.org/c/openstack/nova/+/976085 | |
| 09:08:50 | rubasov | hi nova folks: may I ask for a review on this bugfix (with a functional reproducer)? https://review.opendev.org/q/topic:bug/2051685 | |
| 09:40:29 | opendevreview | Takashi Kajinami proposed openstack/nova master: Use domain capabilities to detect SEV-SNP support https://review.opendev.org/c/openstack/nova/+/996316 | |
| 10:33:01 | opendevreview | Lajos Katona proposed openstack/nova master: Use SDK for Neutron networks https://review.opendev.org/c/openstack/nova/+/928022 | |
| 10:58:53 | opendevreview | Lajos Katona proposed openstack/nova master: Use SDK for Neutron networks https://review.opendev.org/c/openstack/nova/+/928022 | |
| 11:27:29 | tkajinam | mhen, can you share the full content of the firmware descriptor file in Ubuntu ? | |
| 11:27:59 | tkajinam | mhen, it's probably best to report a bug for ubuntu then add the link to it, just in case they fix that invalid descriptor | |
| 11:28:06 | tkajinam | instead of describing the whole details in the local doc | |
| 11:47:38 | opendevreview | Takashi Kajinami proposed openstack/nova master: Remove [libvirt] num_memory_encrypted_guests https://review.opendev.org/c/openstack/nova/+/995120 | |
| 11:47:38 | opendevreview | Takashi Kajinami proposed openstack/nova master: Update documentations for AMD SEV-SNP support https://review.opendev.org/c/openstack/nova/+/995090 | |
| 11:47:39 | opendevreview | Takashi Kajinami proposed openstack/nova master: Use domain capabilities to detect SEV-SNP support https://review.opendev.org/c/openstack/nova/+/996316 | |
| 11:51:44 | opendevreview | Takashi Kajinami proposed openstack/nova master: Update documentations for AMD SEV-SNP support https://review.opendev.org/c/openstack/nova/+/995090 | |
| 11:51:45 | opendevreview | Takashi Kajinami proposed openstack/nova master: Use domain capabilities to detect SEV-SNP support https://review.opendev.org/c/openstack/nova/+/996316 | |
| 11:51:45 | opendevreview | Takashi Kajinami proposed openstack/nova master: Remove [libvirt] num_memory_encrypted_guests https://review.opendev.org/c/openstack/nova/+/995120 | |
| 12:06:26 | opendevreview | Ashish Gupta proposed openstack/placement master: tests: Support file-backed SQLite URL in placement fixtures https://review.opendev.org/c/openstack/placement/+/993106 | |
| 12:21:04 | opendevreview | Ashish Gupta proposed openstack/nova master: tests: use file-backed Placement SQLite in functional threading https://review.opendev.org/c/openstack/nova/+/992581 | |
| 14:50:52 | opendevreview | Ashish Gupta proposed openstack/nova master: tests: Use per-database write locks instead of global lock https://review.opendev.org/c/openstack/nova/+/992862 | |
| 15:00:50 | Uggla | Reminder: Upstream bug triage in ~30mn. | |
| 15:15:03 | sean-k-mooney | gibi oh actlly https://review.opendev.org/c/opendev/zuul-providers/+/996449 merged an hour ag | |
| 15:15:09 | sean-k-mooney | *ago | |
| 15:15:40 | gibi | ack | |
| 15:15:41 | sean-k-mooney | so we shoudl not see failure in new jobs | |
| 15:34:09 | sean-k-mooney | Uggla: url? | |
| 15:34:26 | Uggla | https://meet.google.com/zjr-rxus-hzj | |
| 15:41:19 | opendevreview | Takashi Kajinami proposed openstack/nova master: libvirt: Remove redundant version check for virtio-fs https://review.opendev.org/c/openstack/nova/+/996496 | |
| 15:41:35 | tkajinam | ^^^ wondering if we should check the virt type instead | |
| 15:42:14 | sean-k-mooney | we woudl need to check for both qemu and kvm | |
| 15:42:15 | tkajinam | (this also makes me wonder if we should require kvm for sev support. it may not work for the other virt type really. | |
| 15:42:55 | sean-k-mooney | i.e virtio fs shoudl wor for qemu and kvm | |
| 15:43:10 | sean-k-mooney | sev maybe | |
| 15:43:25 | tkajinam | sean-k-mooney, yeah and I don't think it may work for lxc for example | |
| 15:43:35 | sean-k-mooney | i dont know if that works for qemu or just with kvm | |
| 15:44:19 | sean-k-mooney | well for lxc we are not actully using a vm so i dont think sev works for contianer in general | |
| 15:44:37 | sean-k-mooney | you can enabeld sev for all memoy at the host level but if you do that it used to break sriov | |
| 15:45:10 | tkajinam | sev definitely requires kvm because it relies on the implementations in kvm_amd module. I'm not too sure about virtiofs on the other hand | |
| 15:45:48 | sean-k-mooney | i dont think that requires kvm | |
| 15:46:05 | sean-k-mooney | but ya for sev you could add checkign teh vrit type to the supprot crtiria | |
| 15:46:10 | sean-k-mooney | if its not already there | |
| 15:46:20 | tkajinam | it's not, yet | |
| 15:46:40 | tkajinam | that's has never been added since sev support was first introduced. | |
| 15:47:02 | tkajinam | because that's an existing problem and may be trivial I'll look into that later, separately from sev-snp work | |
| 15:47:03 | sean-k-mooney | well if it does not work it would just fail to boot the vm | |
| 15:47:08 | tkajinam | yeah | |
| 15:47:26 | sean-k-mooney | so its not really a regression to not report sev supprot when virt type is qemu | |
| 15:47:57 | sean-k-mooney | tkajinam: if you can confim it i would just file a bug for it | |
| 15:48:48 | sean-k-mooney | https://review.opendev.org/c/openstack/nova/+/996496 looks ok ot me although im serpised we dont have any unit test coverage of the functions your removing | |
| 15:49:25 | tkajinam | sean-k-mooney, I'll file one by my end | |
| 15:49:43 | tkajinam | we might want to change that check to virtio-fs to account virt_type, instead of just removing it. | |
| 15:49:53 | tkajinam | (I'll record that as a separate bug, too | |
| 16:07:42 | opendevreview | Takashi Kajinami proposed openstack/nova master: Do not report SEV capability for non kvm virt_type https://review.opendev.org/c/openstack/nova/+/996498 | |
| 16:16:43 | gibi | sean-k-mooney: I think futurist is just slow to shut down an exector https://bugs.launchpad.net/futurist/+bug/2160159 | |
| 16:17:19 | sean-k-mooney | ah ok | |
| 16:17:50 | sean-k-mooney | i mean we could actully swap to concurrent.futures.ThreadPoolExecutor(max_workers=100) if needed | |
| 16:18:15 | sean-k-mooney | we have centralised the executor interaction in nova.utils | |
| 16:19:04 | sean-k-mooney | but cool that sound like a real bug | |
| 16:19:32 | gibi | yeah I will check if I can spot why | |
| 16:21:22 | sean-k-mooney | https://github.com/openstack/futurist/blob/master/futurist/_thread.py#L110-L147 | |
| 16:22:24 | sean-k-mooney | so its polling ever second https://github.com/openstack/futurist/blob/master/futurist/_thread.py#L98-L108 | |
| 16:22:29 | sean-k-mooney | when its waitign for the work to complete | |
| 16:23:35 | sean-k-mooney | but its also joining the work thread https://github.com/openstack/futurist/blob/master/futurist/_thread.py#L142 | |
| 16:24:01 | sean-k-mooney | but in your lamda case that should not be costly | |
| 16:24:16 | gibi | is it due to https://github.com/openstack/futurist/blob/master/futurist/_thread.py#L103 | |
| 16:24:19 | gibi | ? | |
| 16:24:27 | gibi | there is a clear 1 sec timeout passed there | |
| 16:25:00 | sean-k-mooney | that what i was wondering too | |