| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2026-07-03 | |||
| 13:02:33 | gibi | too new | |
| 13:03:35 | gibi | hm, I have it in sid, but that would be even harder to put on this lab machine | |
| 13:03:36 | sean-k-mooney | tkajinam: i would ask that we also put this in the release not for the feature if that ok | |
| 13:03:57 | sean-k-mooney | gibi: you can enable the virt preview repo | |
| 13:04:01 | sean-k-mooney | are you on c10s | |
| 13:04:15 | sean-k-mooney | https://copr.fedorainfracloud.org/coprs/g/virtmaint-sig/virt-preview/ | |
| 13:04:41 | sean-k-mooney | htat has qemu 2:11.0.0-11 | |
| 13:05:29 | sean-k-mooney | you just need to add https://copr.fedorainfracloud.org/coprs/g/virtmaint-sig/virt-preview/repo/centos-stream-10/group_virtmaint-sig-virt-preview-centos-stream-10.repo | |
| 13:05:33 | sean-k-mooney | to your repo config | |
| 13:06:03 | gibi | ooh OK, I can try that. The lab machine is on centos10 yes | |
| 13:06:13 | sean-k-mooney | with that said i woudl finish any ohter testing you want to do first before doing that | |
| 13:06:24 | sean-k-mooney | it normally does not breakthing but it can | |
| 13:06:29 | gibi | yeah I'm mostly done with the exploratory testing | |
| 13:08:07 | sean-k-mooney | there was a patch probaly litrally decades ago to add a flag to enabel the virt preview repo in devstack but i dotn think that ever landed | |
| 13:08:32 | sean-k-mooney | i remember "dicusssing" that wehn trying to get dpdk suprpot into openstack | |
| 13:30:11 | opendevreview | Takashi Kajinami proposed openstack/nova master: Return 400 for invalid properties for locked memory https://review.opendev.org/c/openstack/nova/+/995813 | |
| 13:33:40 | sean-k-mooney | tkajinam: locked memory is not ment ot requried hugepages | |
| 13:33:41 | tdemaret | Hi core team, I need help to understant, what I must doing to validate my spec and validate my code : my pr is here, and I think, I don t understand something : https://review.opendev.org/c/openstack/nova-specs/+/979696 - https://review.opendev.org/c/openstack/nova/+/979693 ( I see the merge conflict, but it is a 3 times, I think, I need to validate my spec before I fix it ? ) | |
| 13:34:59 | sean-k-mooney | tkajinam: unless im conflatin gthat with realtime | |
| 13:35:16 | sean-k-mooney | tkajinam: i know i added that so that vdpa would work | |
| 13:35:19 | opendevreview | Takashi Kajinami proposed openstack/nova master: Return 400 for invalid properties for locked memory https://review.opendev.org/c/openstack/nova/+/995813 | |
| 13:35:21 | sean-k-mooney | without needing realtime | |
| 13:35:40 | sean-k-mooney | but i tough it was inteded to work with file backed memory and small pages as well | |
| 13:35:50 | sean-k-mooney | i woudl have to dig back into the history | |
| 13:38:03 | sean-k-mooney | tkajinam: actully no your right https://github.com/openstack/nova-specs/blob/b855fe1b5d192aefa238cd9465fc0ee60568b2d1/specs/zed/implemented/libvirt-viommu-device.rst?plain=1#L61 | |
| 13:38:36 | sean-k-mooney | i intneed it to be set only when hugepages to avodi needign to mark the vm as realtime and foce cpu pinning | |
| 13:39:30 | tkajinam | sean-k-mooney, ah, ok. honestly I found that requirement a bit strange first (because locked memory should work without huge pages afaik) but now it's clearer | |
| 13:39:47 | sean-k-mooney | we could make it work withough hugepages | |
| 13:39:52 | tkajinam | yeah | |
| 13:39:54 | sean-k-mooney | but it was to work around a bug in qemu | |
| 13:40:34 | sean-k-mooney | well actully maybe libvirt, qemu or libvirt did not allcoate addtion locked memroy for pci passhtough or vdpa automaticly | |
| 13:40:48 | sean-k-mooney | so we had to mark the entire mroy as locked as a workaround | |
| 13:41:26 | sean-k-mooney | later version fo libvirt do allocate addtioanl locked memeory for passthough devices but we still have the option | |
| 13:43:02 | tkajinam | ok | |
| 13:43:36 | sean-k-mooney | its still usefull in general but lower level then we woudl have prefered to expose | |
| 13:43:56 | sean-k-mooney | i.e. as an operator you really should not need to think about that when creating flavors | |
| 13:44:14 | tkajinam | I think we could consider lifting the limitation after some quick testings, but IMO we should fix the status code separately in a short term (because 403 doesn't match the actual cause imho) | |
| 13:44:15 | tkajinam | yeah | |
| 13:44:37 | sean-k-mooney | oh i was about to ask if you got a 409 | |
| 13:44:43 | sean-k-mooney | ya 403 is wrong | |
| 13:44:49 | sean-k-mooney | 409 or 400 are fine | |
| 13:44:51 | tkajinam | I'm changing it to 400 | |
| 13:45:39 | sean-k-mooney | well it should be the ame as the other flavor iamge conflicts | |
| 13:46:07 | tkajinam | other flavor image conflicts return 400 now so it's consistent with that behavior | |
| 13:46:09 | tkajinam | tdemaret, I'm unsure what your expect by "validate" but in general we expect that spec is reviewed and approved (you can push actual change for reference) and then actual implementations are reviewed according to the spec | |
| 13:47:11 | tkajinam | tafkamax, I saw several comments still open and unanswered so the first thing you need may be to respond to these. | |
| 13:47:21 | 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 | |
| 13:47:30 | tkajinam | tafkamax, oops. sorry that's a wrong mention | |
| 13:47:50 | sean-k-mooney | tdemaret: yes unforctuetly you spec did not get much visablity before the deadlien beacus you had abandoned it | |
| 13:48:12 | sean-k-mooney | so it not approved for the current release and as such the code cannot proceed | |
| 13:48:49 | sean-k-mooney | tdemaret: if you want to advance that for teh 2027.1 cycle the first stpe would be to socialise the freature and repoorpse the spec for the next release | |
| 13:49:17 | opendevreview | Takashi Kajinami proposed openstack/nova master: Validate image properties for stateless firmware at API layer https://review.opendev.org/c/openstack/nova/+/995835 | |
| 13:49:24 | sean-k-mooney | tdemaret: by socailise the feature i mean bring it up in teh nova irc meeint or the ptg | |
| 13:50:07 | sean-k-mooney | overall the idea is a good one and i think it fits in teh scope of nova | |
| 13:50:48 | sean-k-mooney | but procedually its too late in teh cycle since nova had its spec freeze a few weeks ago.the default for other project would have been yesterdady as well | |
| 13:51:37 | sean-k-mooney | the feature design freeze for most teams is milesotne two which was yesterday | |
| 13:52:35 | tdemaret | @sean-k-mooney : Ok it is new for me, I need to socialise to integrate a new spec in the new release. Ok I understand more the workflow. The next PTG is in Oktober ? | |
| 13:52:36 | sean-k-mooney | team are not shifting to completing the already approved work before feature freeze at the end of august, the branches will re open for new feature work for next cycle in mid september | |
| 13:53:06 | sean-k-mooney | tdemaret: yes but you dont need to wait that long | |
| 13:53:29 | sean-k-mooney | we ussually create the spec directory for the next release a few weeks after milestone 2 | |
| 13:54:25 | sean-k-mooney | so you can actully just do that now but i wauld start by bring this up in teh next irc meeting on monday | |
| 13:54:47 | tdemaret | Ok and I must introduce my feature at this time when the directory has been created ? | |
| 13:55:22 | tdemaret | @sean-k-mooney : thanks, I am happy for this feedback | |
| 13:55:49 | sean-k-mooney | tdemaret: so if you want to get visablity it helps, its not stricly requried but unless your already an active contibutor and have reached out to the team its very easy for your spec to go unnotice | |
| 13:56:12 | ykarel | clarkb, melwitt sean-k-mooney i have added summary in https://review.opendev.org/c/zuul/zuul-jobs/+/959393 for the linux bridge/geneve move and current impact and required changes/options to move forward. please do check when you get chance | |
| 13:57:43 | sean-k-mooney | tdemaret: i know the process can seem a bit much for a small change but the realtity is when we add a feature we effectivly need to support that aproximatly forever | |
| 13:58:42 | opendevreview | Takashi Kajinami proposed openstack/nova master: Validate image properties for stateless firmware at API layer https://review.opendev.org/c/openstack/nova/+/995835 | |
| 13:58:45 | tdemaret | sean-k-mooney: thanks, I try to answer on few question I have, and I understand more the process. Thanks for the manual | |
| 14:01:34 | opendevreview | Takashi Kajinami proposed openstack/nova master: compute: Validate SEV/SEV-ES support at start up https://review.opendev.org/c/openstack/nova/+/994342 | |
| 14:05:25 | tdemaret | sean-k-mooney: I have one comment, I don t understand, how I can change something in my spec file. I am a little bit confused/unclear about how to apply it, or what is expected. I have the code example, but I don t have the spec file. I just make a check on the comment and I verify my implementation to be sure, it is ok or I must make something in the spec file. Sorry it is | |
| 14:05:27 | tdemaret | really the newby question... | |
| 14:06:55 | sean-k-mooney | you mena your asking how to update the review for the spec? | |
| 14:08:37 | tdemaret | yes | |
| 14:11:48 | opendevreview | Takashi Kajinami proposed openstack/nova master: compute: Validate SEV/SEV-ES support at start up https://review.opendev.org/c/openstack/nova/+/994342 | |
| 14:14:46 | tdemaret | Really, I don't understand, what I need to change : https://review.opendev.org/c/openstack/nova-specs/+/979696?tab=comments , really I think, I must change the mindset and read more spec file to understand the template and informations are mandatory. | |
| 14:15:03 | opendevreview | Takashi Kajinami proposed openstack/nova master: libvirt: Ignore SEV-ES when SEV-SNP is detected https://review.opendev.org/c/openstack/nova/+/994343 | |
| 14:29:51 | opendevreview | Dan Smith proposed openstack/nova stable/2026.1: Fix mutating global config in websocket proxy https://review.opendev.org/c/openstack/nova/+/995956 | |
| 14:30:48 | opendevreview | Dan Smith proposed openstack/nova stable/2025.2: Fix mutating global config in websocket proxy https://review.opendev.org/c/openstack/nova/+/995957 | |
| 14:31:22 | opendevreview | Dan Smith proposed openstack/nova stable/2025.1: Fix mutating global config in websocket proxy https://review.opendev.org/c/openstack/nova/+/995958 | |
| 14:57:08 | 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 | |
| 15:04:42 | gibi | sean-k-mooney: I tried qemu 11 from virt preview but it is crashing | |
| 15:04:44 | gibi | 2026-07-03T15:00:54.247846Z qemu-system-x86_64: SNP_LAUNCH_UPDATE ret=-5 fw_error=22 'Invalid parameter' | |
| 15:04:47 | gibi | 2026-07-03T15:00:54.247920Z qemu-system-x86_64: SEV-SNP: CPUID validation failed for function 0x80000021, index: 0x0, provided: eax:0x18000065, ebx: 0x00000000, ecx: 0x00000006, edx: 0x00000000, expected: eax:0x18000065, ebx: 0x00000000, ecx: 0x00000000, edx: 0x00000000 | |
| 15:04:51 | gibi | 2026-07-03T15:00:54.247933Z qemu-system-x86_64: SEV-SNP: failed update CPUID page | |
| 15:04:55 | gibi | 2026-07-03 15:00:54.454+0000: shutting down, reason=crashed | |
| 15:13:21 | gibi | OK cpu_model_extra_flags = -tsa-sq-no, -tsa-l1-no | |
| 15:13:23 | gibi | solve sthat | |
| 15:13:28 | gibi | solves that | |
| 15:14:07 | gibi | aand reboot works from within the SNP guest now | |
| 15:15:37 | sean-k-mooney | oh cool | |
| 15:15:56 | gibi | updated the bug | |
| 15:16:03 | gibi | and dropping for the weekend | |
| 15:16:04 | sean-k-mooney | have you tried cpu-mode=host-passthough | |
| 15:16:04 | gibi | o/ | |
| 15:16:10 | gibi | it is with host-passthrough | |
| 15:16:19 | sean-k-mooney | ah thats a sperte bug so | |
| 15:16:24 | sean-k-mooney | gibi: enjoy the weekend | |
| 15:17:08 | gibi | thanks. same to you too | |
| 16:01:52 | opendevreview | Takashi Kajinami proposed openstack/nova master: libvirt: Detect AMD SEV-SNP support https://review.opendev.org/c/openstack/nova/+/994764 | |
| 16:17:28 | opendevreview | Takashi Kajinami proposed openstack/nova master: libvirt: Detect AMD SEV-SNP support https://review.opendev.org/c/openstack/nova/+/994764 | |