| opendevreview | Takashi Kajinami proposed openstack/nova master: Refactor CPU traits functional tests for AMD SEV support https://review.opendev.org/c/openstack/nova/+/997635 | 00:52 |
|---|---|---|
| opendevreview | Takashi Kajinami proposed openstack/nova master: Extend tests for _is_supported_mem_encryption_model helper https://review.opendev.org/c/openstack/nova/+/997663 | 01: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 | 02:22 |
| 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:52 |
| tkajinam | so new oslo.policy release just reached global uc and it breaks doc build and then blocks sev-snp series again :-( | 02:53 |
| * tkajinam sighs | 02:53 | |
| tkajinam | gmaan, ^^^ | 03:00 |
| tkajinam | we probably have to update or even remove the whole document, though | 03:04 |
| opendevreview | Takashi Kajinami proposed openstack/nova stable/2026.1: Fix wrong mock for properties https://review.opendev.org/c/openstack/nova/+/997670 | 03:21 |
| 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:21 |
| 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 | 03:24 |
| opendevreview | Takashi Kajinami proposed openstack/nova master: Refactor CPU traits functional tests for AMD SEV support https://review.opendev.org/c/openstack/nova/+/997635 | 04:28 |
| *** Jeff is now known as JeffYang | 05:31 | |
| opendevreview | Balazs Gibizer proposed openstack/nova master: Make functional-py313-threading voting https://review.opendev.org/c/openstack/nova/+/997216 | 08:09 |
| 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:10 |
| gibi | I mean https://review.opendev.org/c/openstack/nova/+/997222 | 08:11 |
| gibi | and if you warmed up this is a bit more involved https://review.opendev.org/c/openstack/nova/+/992862 | 08:16 |
| bauzas | gibi: ack, will look | 08:19 |
| gibi | thanks a lo | 08:19 |
| gibi | t | 08:19 |
| gibi | bauzas: aaactually. This is resolving a full gate block due to docs gen https://review.opendev.org/c/openstack/nova/+/997667 | 08:26 |
| bauzas | gibi: what ? a doc patch blocked the gate ? | 08:39 |
| bauzas | anyway, all done | 08:40 |
| * bauzas needs to look again at the AMD changes btw. | 08:40 | |
| gibi | bauzas: yes, there was a oslo.policy bump that break our doc generation | 08:44 |
| gibi | as they removed the enforce_scope config option | 08:44 |
| gibi | our doc still refers to | 08:44 |
| bauzas | ah I see so the doc job fails | 08:48 |
| opendevreview | Merged openstack/nova master: doco: Drop remaining reference to [oslo_policy] enforce_scope https://review.opendev.org/c/openstack/nova/+/997667 | 08:56 |
| opendevreview | Kamil Sambor proposed openstack/nova master: Replace StaticallyDelayingWrapper with futurist DelayingExecutor https://review.opendev.org/c/openstack/nova/+/997410 | 09:03 |
| 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? | 09:58 |
| *** jgilaber_ is now known as jgilaber | 10:02 | |
| opendevreview | Marcus Klein proposed openstack/nova master: libvirt: Provide SMBIOS sysinfo for AArch64 guests https://review.opendev.org/c/openstack/nova/+/997708 | 10:30 |
| opendevreview | Marcus Klein proposed openstack/nova master: libvirt: Provide SMBIOS sysinfo for AArch64 guests https://review.opendev.org/c/openstack/nova/+/997708 | 10:33 |
| opendevreview | Merged openstack/nova master: Stabilize test_create_images_negative_invalid_state https://review.opendev.org/c/openstack/nova/+/997222 | 10:40 |
| opendevreview | Merged openstack/nova master: [cover]Disable slow test_schemas test case https://review.opendev.org/c/openstack/nova/+/997503 | 10:41 |
| opendevreview | Marcus Klein proposed openstack/nova master: libvirt: Provide SMBIOS sysinfo for AArch64 guests https://review.opendev.org/c/openstack/nova/+/997708 | 10:43 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: Make functional-py313-threading voting https://review.opendev.org/c/openstack/nova/+/997216 | 10:54 |
| 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:08 |
| * tkajinam triggers recheck because doc job is now fix | 11:09 | |
| tkajinam | * fixed | 11:09 |
| sean-k-mooney | -869 +227 :) | 11:09 |
| sean-k-mooney | ill read it properly but assuming the test coverage is equiveltn that is a masive win thank you | 11:10 |
| gibi | it is on my list too | 11:13 |
| 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:28 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: Check service_ref before destroy https://review.opendev.org/c/openstack/nova/+/996793 | 11:38 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: Make functional-py313-threading voting https://review.opendev.org/c/openstack/nova/+/997216 | 11:38 |
| opendevreview | Balazs Gibizer proposed openstack/placement master: Check overall capacity before generating candidates https://review.opendev.org/c/openstack/placement/+/997617 | 12:15 |
| opendevreview | Marcus Klein proposed openstack/nova master: libvirt: Provide SMBIOS sysinfo for AArch64 guests https://review.opendev.org/c/openstack/nova/+/997708 | 12:17 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: (2/2)Stabilize test_create_images_negative_invalid_state https://review.opendev.org/c/openstack/nova/+/997719 | 12:53 |
| sean-k-mooney | gibi: https://review.opendev.org/c/openstack/nova/+/996793 is very much an unlucky canary for random unrelated failures | 14:20 |
| gibi | sean-k-mooney: it is | 14:23 |
| tkajinam | and sev-snp chain is, too :-( | 14:37 |
| tkajinam | I don't have actual number but feel like nova-multi-cell and nova-next are more flaky these days | 14:37 |
| tkajinam | though the failure sometimes looks related to infra rather than actual service logics | 14:38 |
| gibi | tkajinam: funnily i haven't seen the same error happening twice in my patch. the not nice situation to die due to 1000 cuts | 14:40 |
| sean-k-mooney | that was the same when i was testing the alternitve cirrors image | 14:40 |
| sean-k-mooney | i hit everfy random failure the gate has but never twice | 14:41 |
| gibi | can we just turn off CI? and replace it with an LLM and a prompt? :D | 14:41 |
| gibi | (very bad joke, sorry) | 14:41 |
| sean-k-mooney | na that too much effort just push to master it will be grand | 14:42 |
| sean-k-mooney | cannel that vc sartup energy, what could possibel go wrong :) | 14:42 |
| tkajinam | gibi, maybe zuul doesn't like me throwing too many patches :-P | 14:44 |
| tkajinam | the most annoying thing so far is that the base change quite frequently hits the error only in gate, which causes a lot of +2 in the chain ... | 14:45 |
| tkajinam | anyway I'll monitor CI results and attempt to push these to main... so that we can more easily work on tdx part, which may be rebased on top of sev-snp | 14:46 |
| gibi | we should try to bribe zuul, with more, jucier CPU cores | 14:46 |
| tkajinam | lol | 14:46 |
| dansmith | gibi: man, you are in rare form today :) | 15:02 |
| gibi | then I stop on the top of my career (at least for this week). I bow to the crowd and let myself out :) | 15:16 |
| gibi | see you all next week | 15:16 |
| dansmith | jeez, all the way to the door.. have a good weekend :) | 15:17 |
| gmaan | tkajinam: yeah, oslo.policy changes were up but it needed to wait for requirement change to be in. or i could have seperate them from requirements.txt change https://review.opendev.org/c/openstack/nova/+/994950 | 16:36 |
| opendevreview | Takashi Kajinami proposed openstack/nova master: Revert "disable ovn based testing of test_live_migration_with_trunk" https://review.opendev.org/c/openstack/nova/+/997768 | 16:38 |
| opendevreview | Ghanshyam Maan proposed openstack/nova master: Remove disabling the scope enforcement https://review.opendev.org/c/openstack/nova/+/994950 | 16:41 |
| gmaan | rebased it, I think it will pass the requirements job now | 16:41 |
| sean-k-mooney | kleini: yes the data we expose on x86 leakes the version of openstack that is in use allowign you to target it for unpatch vlunermableity and the cloud init comutintiy decied to break the core contract of there utility by makeign it depend on an internal implemntatoin detail that is not intedned to be used like that | 16:53 |
| sean-k-mooney | kleini: basiclly that smbiors/dmiinfo is only exposed by the libvirt driver on x86 | 16:54 |
| sean-k-mooney | it is not possibel to do that for ironic and as far as im aware it was not done in vmware or hyperv | 16:54 |
| sean-k-mooney | so cloud init shoudl never have used that to detect if its on openstack | 16:55 |
| sean-k-mooney | they broke the openstack supprot to "fix" a security report that was arguable invlaide as it required a network where there was the ablity to spoof the non routable 169.... adresses via arp poising or similar | 16:57 |
| sean-k-mooney | kleini: as far as im aware you can renabel the ec2/openstack supprot explcity in your cloud images | 16:58 |
| sean-k-mooney | that woudl generally be my recomendation, you can also force enabel config drive in nova | 16:59 |
| opendevreview | Ghanshyam Maan proposed openstack/nova master: Remove disabling the scope enforcement https://review.opendev.org/c/openstack/nova/+/994950 | 18:21 |
| gmaan | tkajinam: ^^ updated it as per your review comment. | 18:21 |
| opendevreview | Merged openstack/nova master: Check service_ref before destroy https://review.opendev.org/c/openstack/nova/+/996793 | 18:24 |
| opendevreview | Merged openstack/nova master: libvirt: Detect AMD SEV-SNP support https://review.opendev.org/c/openstack/nova/+/994764 | 18:27 |
| opendevreview | Merged openstack/nova master: libvirt: Support launchSecurity element for SEV-SNP https://review.opendev.org/c/openstack/nova/+/994920 | 19:10 |
| opendevreview | Takashi Kajinami proposed openstack/nova master: libvirt: Remove redundant version check for remote managed ports https://review.opendev.org/c/openstack/nova/+/995160 | 21:42 |
| opendevreview | Merged openstack/nova master: libvirt: Omit optional fields for SEV/SEV-ES https://review.opendev.org/c/openstack/nova/+/994921 | 21:58 |
| gmaan | melwitt gibi sean-k-mooney these release notes for enforce_scoope are ready now https://review.opendev.org/q/hashtag:%22remove-enforce-scope-flag%22+hashtag:%22nova%22status:open | 22:00 |
| opendevreview | Merged openstack/nova master: libvirt: Drop sev capability check in launch instance flow https://review.opendev.org/c/openstack/nova/+/994929 | 22:26 |
| opendevreview | Merged openstack/nova master: AMD SEV: omit iommu='on' for virtio devices https://review.opendev.org/c/openstack/nova/+/909635 | 22:27 |
Generated by irclog2html.py 4.1.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!