| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2026-07-07 | |||
| 18:48:06 | opendevreview | Takashi Kajinami proposed openstack/nova master: WIP: Use domain capabilities to detect SEV-SNP support by libvirt/QEMU https://review.opendev.org/c/openstack/nova/+/996316 | |
| 18:48:11 | gmaan | sambork: gibi: after seeing openstack-tox-py313 and openstack-tox-py314 unit test jobs failing one of my test in eventlet mode (which i need to fix to handle both mode), i realized should not be run all our default unit tests jobs openstack-tox-py313, openstack-tox-py314 in threading mode and convert nova-tox-py313-threading->nova-tox-py313-eventlet ? | |
| 18:48:36 | gmaan | mainly openstack-tox-py314 or any new higher python versions testing can be done in threading mode | |
| 19:02:46 | sean-k-mooney | well we will next cycle right | |
| 19:03:03 | sean-k-mooney | the goal is to default to threaded mdoe this cycle | |
| 19:03:48 | sean-k-mooney | but form next cycel we defintly should | |
| 19:04:10 | sean-k-mooney | for this cycle if we now have everything defaulting to threaded | |
| 19:04:34 | sean-k-mooney | we could update the job/tox config i guess | |
| 19:05:03 | sean-k-mooney | gmaan: we are technaly supproting eventlet on 3.14 | |
| 19:05:20 | sean-k-mooney | so the eventlet supprot is more tied to the nova version then the python version | |
| 19:06:01 | gmaan | yeah, i mean we can run 3.14 version on eventlet but rest all jobs running on threading mode | |
| 19:06:03 | sean-k-mooney | although i dont know if 3.15 will ever be suproted by eventlet so that might eb threaded only as a side effect | |
| 19:07:01 | sean-k-mooney | well really your talking about changing this to true https://github.com/openstack/nova/blob/master/tox.ini#L66 | |
| 19:07:10 | gmaan | kind of matching the code default which is threading mode to what we tests in most of the jobs and have a single eventlet job running on latest py version until we need to keep the support | |
| 19:07:18 | sean-k-mooney | ya | |
| 19:07:28 | sean-k-mooney | i think that woudl make sense | |
| 19:08:14 | sean-k-mooney | so remove OS_NOVA_DISABLE_EVENTLET_PATCHING=False form the default tox unit and functional targets | |
| 19:08:21 | gmaan | yeah | |
| 19:08:32 | sean-k-mooney | and rename the -threading evne and update the zuul jobs to match | |
| 19:08:40 | gmaan | functional I am not sure as i think that is not yet ready for threading mode | |
| 19:08:53 | gmaan | current functional threading mode job fail | |
| 19:09:09 | gmaan | but yes at least unit tests are all ready | |
| 19:09:22 | sean-k-mooney | oh ok i did not see an exclude file there | |
| 19:09:55 | sean-k-mooney | but ya i know that was in flight i guess we are not quite there yet | |
| 19:10:16 | sean-k-mooney | anyway i think your proposal makes sense so +1 form me | |
| 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. | |