| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2026-07-16 | |||
| 12:48:38 | gibi | unfortunately we are short staffed so not all patches get reviewed and the ones that gets reviewed might take a long time | |
| 12:53:02 | gibi | gmaan: when you are up, what does this schema stacking means? https://github.com/openstack/nova/blob/42fd9b6c8c5da09c92f5fb7efac1f5da52ca7df4/nova/api/openstack/compute/servers.py#L700-L702 Why do we have v20 top of an unversioned schema there? | |
| 12:53:15 | gibi | this is related to the fix from artpej in https://review.opendev.org/c/openstack/nova/+/995073 | |
| 12:59:08 | gibi | it almost like that the 2.0 schema is better than the 2.0-2.18 schema. That feel backwards | |
| 12:59:19 | gibi | ~2.1-2.18 | |
| 12:59:37 | gibi | artpej: at least you found something strange :) | |
| 13:01:45 | artpej | Yes, my users are weird people who put spaces at the beginning of security group names :) | |
| 13:05:51 | gibi | that is weird too :) | |
| 13:06:11 | gibi | but the schema handling in nova for that call is weird as well | |
| 13:06:18 | gibi | hence my ping to gmaan | |
| 13:07:27 | gibi | we should not have an API where 2.0 accepts " sg " but 2.1-2.18 does not. But 2.0 was a bit special I assume compared to 2.1. Still I feel like we might be able to say lets fix this in 2.1-2.18. | |
| 13:08:29 | gibi | hm | |
| 13:08:34 | gibi | even more weird | |
| 13:08:49 | gibi | it might be that only 2.0 allows leading spaces there and nothing after 2.1 | |
| 13:09:13 | gibi | still we have a strong case, if neutron allows leading spaces then we should not stop it | |
| 13:35:17 | opendevreview | Merged openstack/nova master: Do not try to return empty attachment_id from API https://review.opendev.org/c/openstack/nova/+/996757 | |
| 13:41:22 | opendevreview | Merged openstack/nova master: compute: Validate SEV/SEV-ES support at start up https://review.opendev.org/c/openstack/nova/+/994342 | |
| 13:41:49 | opendevreview | Merged openstack/nova master: libvirt: Ignore SEV-ES when SEV-SNP is detected https://review.opendev.org/c/openstack/nova/+/994343 | |
| 14:17:39 | tkajinam | sean-k-mooney, could you also check https://review.opendev.org/c/openstack/nova-specs/+/996394 and https://review.opendev.org/c/openstack/nova-specs/+/996395 when you have time ? | |
| 14:18:01 | tkajinam | these two update the sev-snp spec according to the latest implementation | |
| 14:28:06 | 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 | |
| 14:55:44 | sean-k-mooney | tkajinam: bothg of those look good to me as well | |
| 14:58:13 | tkajinam | sean-k-mooney, thanks ! | |
| 14:59:04 | opendevreview | Kamil Sambor proposed openstack/nova master: Replace StaticallyDelayingWrapper with futurist DelayingExecutor https://review.opendev.org/c/openstack/nova/+/997410 | |
| 15:11:30 | opendevreview | Merged openstack/nova-specs master: Fix SEV-SNP feature detection in start up https://review.opendev.org/c/openstack/nova-specs/+/996394 | |
| 15:13:34 | opendevreview | Merged openstack/nova-specs master: Use domain capability to detect SEV-SNP support https://review.opendev.org/c/openstack/nova-specs/+/996395 | |
| 16:00:38 | gmaan | gibi: artpej: that seems strange as v2.0 is less strict than >=v2.1 and not other way. anyways I will check it today | |
| 16:13:09 | gibi | gmaan: thanks | |
| 16:35:50 | opendevreview | Balazs Gibizer proposed openstack/placement master: Check overall capacity before generating candidates https://review.opendev.org/c/openstack/placement/+/997617 | |
| 16:36:24 | gibi | sean-k-mooney: it is not terrible to check the flattened capacity in python ^^ | |
| 16:36:49 | sean-k-mooney | cool | |
| 16:36:59 | sean-k-mooney | i mean if we get it workign we can make it faster later | |
| 16:39:33 | gibi | it works and it is not super ugly as I had all the data available in the right place, it is just a bit of data juggling to summarize and compare | |
| 16:39:56 | sean-k-mooney | ack, on a call but ill look after | |
| 16:40:05 | gibi | thanks | |
| 16:40:25 | sean-k-mooney | if its "good enough" then we can start there and defer doign it in sql till later | |
| 16:41:32 | sean-k-mooney | oh your system how long these it take i assume you updated the coments in the tests? | |
| 16:42:50 | gibi | it is so fast that I bumped N to 64 and removed the comments as it runs in sub seconds | |
| 16:43:13 | gibi | it fully eliminates the hot path | |
| 16:43:36 | gibi | I also added a test that shows the the extra check is not too costly for the non edge case | |
| 16:44:05 | gibi | {3} placement.tests.functional.test_allocation_candidates.TestAlmostFittingRoot.test_one_less_device_than_requested_no_wide_tree_optimization [0.231623s] ... ok | |
| 16:44:24 | gibi | this is with 64 RP 63 request | |
| 16:52:24 | gibi | I had enough for today :) See you folks tomorrow | |
| 17:19:32 | sean-k-mooney | gibi: cool enjoy your evenigng | |
| 17:20:07 | sean-k-mooney | just finished the calls so ill take a look shortly | |
| 18:40:06 | opendevreview | Takashi Kajinami proposed openstack/nova master: Refactor CPU traits functional tests for AMD SEV support https://review.opendev.org/c/openstack/nova/+/997635 | |
| 19:10:07 | opendevreview | Takashi Kajinami proposed openstack/nova master: Refactor CPU traits functional tests for AMD SEV support https://review.opendev.org/c/openstack/nova/+/997635 | |
| 19:12:30 | gmaan | artpej: gibi RE SG name with leading/trailing whitespace, yes it is changed/disallowed with v2.1 strict validation on resource name. But agree that it does not make sense for Nova to block it when neuston allow. I added comment in bug as well as gerrit. | |
| 19:13:01 | gmaan | only things is we need to fix this for higher microversion and should not change older microvesion behavior even this is regression. | |
| 19:14:25 | opendevreview | Takashi Kajinami proposed openstack/nova master: Refactor CPU traits functional tests for AMD SEV support https://review.opendev.org/c/openstack/nova/+/997635 | |
| 19:33:07 | opendevreview | Takashi Kajinami proposed openstack/nova master: Refactor CPU traits functional tests for AMD SEV support https://review.opendev.org/c/openstack/nova/+/997635 | |
| 19:34:29 | tkajinam | -889 +234 ... | |
| 19:57:04 | opendevreview | Sylvain Desgrais proposed openstack/nova master: Allow security group names with leading/trailing spaces https://review.opendev.org/c/openstack/nova/+/995073 | |
| #openstack-nova - 2026-07-17 | |||
| 00:52:58 | opendevreview | Takashi Kajinami proposed openstack/nova master: Refactor CPU traits functional tests for AMD SEV support https://review.opendev.org/c/openstack/nova/+/997635 | |
| 01:06:31 | opendevreview | Takashi Kajinami proposed openstack/nova master: Extend tests for _is_supported_mem_encryption_model helper https://review.opendev.org/c/openstack/nova/+/997663 | |
| 02:22:17 | opendevreview | Takashi Kajinami proposed openstack/nova master: Refactor CPU traits functional tests for AMD SEV support https://review.opendev.org/c/openstack/nova/+/997635 | |
| 02:52:27 | opendevreview | Takashi Kajinami proposed openstack/nova master: doco: Drop remaining reference to [oslo_policy] enforce_scope https://review.opendev.org/c/openstack/nova/+/997667 | |
| 02:53:00 | tkajinam | so new oslo.policy release just reached global uc and it breaks doc build and then blocks sev-snp series again :-( | |
| 03:00:46 | tkajinam | gmaan, ^^^ | |
| 03:04:37 | tkajinam | we probably have to update or even remove the whole document, though | |
| 03:21:24 | opendevreview | Takashi Kajinami proposed openstack/nova stable/2026.1: Fix wrong mock for properties https://review.opendev.org/c/openstack/nova/+/997670 | |
| 03:21:59 | opendevreview | Takashi Kajinami proposed openstack/nova stable/2026.1: Drop dependency on [libvirt] num_memory_encrypted_guests https://review.opendev.org/c/openstack/nova/+/997671 | |
| 03:24:48 | opendevreview | Takashi Kajinami proposed openstack/nova stable/2026.1: Accept hw_firmware_stateless=True for BIOS firmware type https://review.opendev.org/c/openstack/nova/+/997672 | |
| 04:28:29 | opendevreview | Takashi Kajinami proposed openstack/nova master: Refactor CPU traits functional tests for AMD SEV support https://review.opendev.org/c/openstack/nova/+/997635 | |
| 08:09:19 | opendevreview | Balazs Gibizer proposed openstack/nova master: Make functional-py313-threading voting https://review.opendev.org/c/openstack/nova/+/997216 | |
| 08:10:59 | gibi | bauzas: some easy wins for friday morning coffee https://review.opendev.org/c/openstack/nova/+/997503 and https://review.opendev.org/c/openstack/nova/+/997503 | |
| 08:11:05 | gibi | I mean https://review.opendev.org/c/openstack/nova/+/997222 | |
| 08:16:29 | gibi | and if you warmed up this is a bit more involved https://review.opendev.org/c/openstack/nova/+/992862 | |
| 08:19:34 | bauzas | gibi: ack, will look | |
| 08:19:38 | gibi | thanks a lo | |
| 08:19:39 | gibi | t | |
| 08:26:58 | gibi | bauzas: aaactually. This is resolving a full gate block due to docs gen https://review.opendev.org/c/openstack/nova/+/997667 | |
| 08:39:21 | bauzas | gibi: what ? a doc patch blocked the gate ? | |
| 08:40:30 | bauzas | anyway, all done | |
| 08:44:25 | gibi | bauzas: yes, there was a oslo.policy bump that break our doc generation | |
| 08:44:41 | gibi | as they removed the enforce_scope config option | |
| 08:44:45 | gibi | our doc still refers to | |
| 08:48:46 | bauzas | ah I see so the doc job fails | |
| 08:56:23 | opendevreview | Merged openstack/nova master: doco: Drop remaining reference to [oslo_policy] enforce_scope https://review.opendev.org/c/openstack/nova/+/997667 | |
| 09:03:31 | opendevreview | Kamil Sambor proposed openstack/nova master: Replace StaticallyDelayingWrapper with futurist DelayingExecutor https://review.opendev.org/c/openstack/nova/+/997410 | |
| 09:58:43 | kleini | Question to the AARCH64 users: There is no SMBIOS sysinfo entry for aarch64 VMs in qemu instance XML file stating "OpenStack Nova" in the product entry. ds-identify does not notice to run on a cloud and cloud-init is disabled. A config drive is required to enable cloud-init. Is there a good reason for that? | |
| 10:30:27 | opendevreview | Marcus Klein proposed openstack/nova master: libvirt: Provide SMBIOS sysinfo for AArch64 guests https://review.opendev.org/c/openstack/nova/+/997708 | |
| 10:33:50 | opendevreview | Marcus Klein proposed openstack/nova master: libvirt: Provide SMBIOS sysinfo for AArch64 guests https://review.opendev.org/c/openstack/nova/+/997708 | |
| 10:40:55 | opendevreview | Merged openstack/nova master: Stabilize test_create_images_negative_invalid_state https://review.opendev.org/c/openstack/nova/+/997222 | |
| 10:41:08 | opendevreview | Merged openstack/nova master: [cover]Disable slow test_schemas test case https://review.opendev.org/c/openstack/nova/+/997503 | |
| 10:43:47 | opendevreview | Marcus Klein proposed openstack/nova master: libvirt: Provide SMBIOS sysinfo for AArch64 guests https://review.opendev.org/c/openstack/nova/+/997708 | |
| 10:54:20 | opendevreview | Balazs Gibizer proposed openstack/nova master: Make functional-py313-threading voting https://review.opendev.org/c/openstack/nova/+/997216 | |
| 11:08:56 | tkajinam | sean-k-mooney, I've proposed https://review.opendev.org/c/openstack/nova/+/997635 and https://review.opendev.org/c/openstack/nova/+/997663 to address your comments in unit tests | |
| 11:09:37 | tkajinam | * fixed | |
| 11:09:50 | sean-k-mooney | -869 +227 :) | |
| 11:10:37 | sean-k-mooney | ill read it properly but assuming the test coverage is equiveltn that is a masive win thank you | |
| 11:13:04 | gibi | it is on my list too | |
| 11:28:54 | opendevreview | Thibaut Démaret proposed openstack/nova master: libvirt: add disk rotation_rate support for local disks https://review.opendev.org/c/openstack/nova/+/979693 | |
| 11:38:26 | opendevreview | Balazs Gibizer proposed openstack/nova master: Check service_ref before destroy https://review.opendev.org/c/openstack/nova/+/996793 | |
| 11:38:34 | opendevreview | Balazs Gibizer proposed openstack/nova master: Make functional-py313-threading voting https://review.opendev.org/c/openstack/nova/+/997216 | |
| 12:15:08 | opendevreview | Balazs Gibizer proposed openstack/placement master: Check overall capacity before generating candidates https://review.opendev.org/c/openstack/placement/+/997617 | |
| 12:17:16 | opendevreview | Marcus Klein proposed openstack/nova master: libvirt: Provide SMBIOS sysinfo for AArch64 guests https://review.opendev.org/c/openstack/nova/+/997708 | |
| 12:53:39 | opendevreview | Balazs Gibizer proposed openstack/nova master: (2/2)Stabilize test_create_images_negative_invalid_state https://review.opendev.org/c/openstack/nova/+/997719 | |
| 14:20:54 | sean-k-mooney | gibi: https://review.opendev.org/c/openstack/nova/+/996793 is very much an unlucky canary for random unrelated failures | |
| 14:23:28 | gibi | sean-k-mooney: it is | |