| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2025-08-27 | |||
| 05:26:54 | opendevreview | Merged openstack/nova stable/2025.1: Don't reset port dns_name when shelving instances https://review.opendev.org/c/openstack/nova/+/956216 | |
| 05:27:08 | opendevreview | Merged openstack/nova stable/2025.1: Fix case-sensitivity for metadata keys https://review.opendev.org/c/openstack/nova/+/945842 | |
| 05:30:39 | opendevreview | Merged openstack/nova stable/2025.1: Fix disable memballoon device https://review.opendev.org/c/openstack/nova/+/952959 | |
| 05:30:51 | opendevreview | Merged openstack/nova stable/2025.1: Fix case sensitive comparison https://review.opendev.org/c/openstack/nova/+/945843 | |
| 05:34:24 | opendevreview | Rajesh Tailor proposed openstack/nova master: Update api-ref for server create https://review.opendev.org/c/openstack/nova/+/937534 | |
| 05:35:39 | opendevreview | Merged openstack/nova stable/2025.1: [doc]Clarify where to set pci_in_placement https://review.opendev.org/c/openstack/nova/+/951624 | |
| 05:36:02 | opendevreview | Merged openstack/nova master: api: Separate volume, snapshot and volume attachments https://review.opendev.org/c/openstack/nova/+/952347 | |
| 05:36:23 | opendevreview | Merged openstack/nova master: tests: Use valid UUIDs for cinder resources https://review.opendev.org/c/openstack/nova/+/952935 | |
| 05:49:47 | opendevreview | Rajesh Tailor proposed openstack/nova master: Fix KeyError on assisted snapshot call https://review.opendev.org/c/openstack/nova/+/900783 | |
| 05:59:53 | opendevreview | Rajesh Tailor proposed openstack/nova master: Add upgrade status check for duplicate cell names https://review.opendev.org/c/openstack/nova/+/901810 | |
| 06:04:38 | opendevreview | Merged openstack/nova master: api: Only apply "soft" additionalProperties validation to requests https://review.opendev.org/c/openstack/nova/+/952936 | |
| 06:04:51 | opendevreview | Merged openstack/nova stable/2024.2: wrap wsgi_app.init_application with latch_error_on_raise https://review.opendev.org/c/openstack/nova/+/945440 | |
| 06:43:09 | opendevreview | Max proposed openstack/nova master: performance: reduce calls to libvirt / add caching https://review.opendev.org/c/openstack/nova/+/922855 | |
| 07:32:07 | opendevreview | Florian proposed openstack/nova master: Add check for PCIe devices attach limit for volume and ports https://review.opendev.org/c/openstack/nova/+/955584 | |
| 07:58:17 | ralonsoh | sean-k-mooney, hello! qq regarding this: https://specs.openstack.org/openstack/nova-specs/specs/rocky/implemented/sriov-trusted-vfs.html | |
| 07:59:30 | ralonsoh | so if we have a VF bond inside the VM (with trusted VFs), it is possible to change the port MAC. But that will make the SR-IOV agent to fail. When the MAC-PCI address pair doesn't match, it returns a warning and forces a full resync | |
| 07:59:58 | ralonsoh | it will repeat this every cycle (that means, every 2 seconds, by default) | |
| 08:00:24 | ralonsoh | did we considered somehow this scenario?? from the Neutron point of view | |
| 08:00:47 | opendevreview | Balazs Gibizer proposed openstack/nova stable/2024.2: [doc]Clarify where to set pci_in_placement https://review.opendev.org/c/openstack/nova/+/958623 | |
| 08:27:21 | gibi | sean-k-mooney: dansmith: bauzas: we have both green nova-next and unit test results for the nova-conductor eventlet removal change. See my comment in https://review.opendev.org/c/openstack/nova/+/957088/6#message-5d92177333ca556fc30c6934603d37ab55cd161c | |
| 08:27:25 | gibi | sambork: ^^ | |
| 08:27:41 | bauzas | gibi: that's on my today's duties | |
| 08:27:52 | bauzas | I'm finishing up the AMD SEV-ES series | |
| 08:27:59 | gibi | I will do some local manual testing now and report back on the same patch | |
| 08:30:39 | bauzas | ++ | |
| 08:32:08 | gibi | sambork: as far as I see this is the only remaining ask form yesterday https://review.opendev.org/c/openstack/nova/+/957088/6#message-cd81d9ce4afbc379e730f15d8df8334a5303e42f | |
| 08:32:27 | sambork | gibi, ack, thanks | |
| 08:32:33 | gibi | sambork: hm | |
| 08:32:36 | gibi | sambork: wait a sec | |
| 08:33:45 | gibi | sambork: I have a small thing in the parent of that https://review.opendev.org/c/openstack/nova/+/957424 so if we anyhow want to touch the top then I would like to fix the bottom as well in the same run | |
| 08:34:22 | gibi | sambork: I guess it is easier if I just go and do both changes in the same push | |
| 08:34:46 | sambork | gibi, ok thanks a lot! | |
| 08:34:57 | gibi | sambork: also sean-k-mooney is happy that the doc/reno/zuul patch is separate so I will not squash that in https://review.opendev.org/c/openstack/nova/+/958575/1 | |
| 08:35:28 | gibi | bauzas: ^^ hold you reviews until I re-spin | |
| 08:36:02 | bauzas | ack, anyway, I was aiming to review only by the next 30 mins, no worries | |
| 08:36:39 | opendevreview | Merged openstack/nova stable/2024.1: Reproduce bug/2098496 https://review.opendev.org/c/openstack/nova/+/945100 | |
| 08:42:51 | bauzas | tkajinam: I finished reviewing your series, nothing holding (I mostly +2d) except one breaking change you introduce by requiring a SEV trait | |
| 08:43:55 | bauzas | I overlooked the spec and nothing was mentioned in the upgrade section, but I do feel that if we start requiring a trait, then we need to mention to the operator that he has to fully upgrade his computes then | |
| 08:44:05 | bauzas | or he wouldn't get old computes supporting SEV too | |
| 08:44:45 | bauzas | that's what I called a behavioural change | |
| 08:45:13 | bauzas | sean-k-mooney[m]: when you're up, I'd like your thoughts on the rolling upgrade scenario with SEV nodes | |
| 08:51:52 | opendevreview | Balazs Gibizer proposed openstack/nova master: Switch nova-conductor to use ThreadPoolExecutor https://review.opendev.org/c/openstack/nova/+/957088 | |
| 08:51:52 | opendevreview | Balazs Gibizer proposed openstack/nova master: Ask for pre-prod testing for native threading https://review.opendev.org/c/openstack/nova/+/957424 | |
| 08:51:53 | opendevreview | Balazs Gibizer proposed openstack/nova master: Allow to start unit test without eventlet https://review.opendev.org/c/openstack/nova/+/953436 | |
| 08:51:53 | opendevreview | Balazs Gibizer proposed openstack/nova master: Run nova-conductor in native threading mode https://review.opendev.org/c/openstack/nova/+/958575 | |
| 08:51:54 | opendevreview | Balazs Gibizer proposed openstack/nova master: [test]RPC using threading or eventlet selectively https://review.opendev.org/c/openstack/nova/+/953815 | |
| 08:51:54 | opendevreview | Balazs Gibizer proposed openstack/nova master: Run unit test with threading mode https://review.opendev.org/c/openstack/nova/+/953475 | |
| 08:51:55 | opendevreview | Balazs Gibizer proposed openstack/nova master: [CI]Make nova-tox-py312-threading voting https://review.opendev.org/c/openstack/nova/+/955791 | |
| 08:51:56 | opendevreview | Balazs Gibizer proposed openstack/nova master: Make RBD Tpool usage conditional https://review.opendev.org/c/openstack/nova/+/956089 | |
| 08:51:56 | opendevreview | Balazs Gibizer proposed openstack/nova master: Do not yield in threading mode https://review.opendev.org/c/openstack/nova/+/950994 | |
| 08:51:58 | opendevreview | Balazs Gibizer proposed openstack/nova master: Fix ProviderTree copying with threading Lock https://review.opendev.org/c/openstack/nova/+/956091 | |
| 08:51:58 | opendevreview | Balazs Gibizer proposed openstack/nova master: Make libvirt Tpool proxying conditional https://review.opendev.org/c/openstack/nova/+/956090 | |
| 08:52:00 | opendevreview | Balazs Gibizer proposed openstack/nova master: [vncproxy]Handle ssl.wrap_socket removal in py312 https://review.opendev.org/c/openstack/nova/+/955915 | |
| 08:52:00 | opendevreview | Balazs Gibizer proposed openstack/nova master: [test]Further categorization of disabled unit tests https://review.opendev.org/c/openstack/nova/+/956092 | |
| 08:52:02 | opendevreview | Balazs Gibizer proposed openstack/nova master: Warn on long task wait time for executor https://review.opendev.org/c/openstack/nova/+/952666 | |
| 08:53:03 | gibi | bauzas: done. you can go wild now :) | |
| 09:03:54 | bauzas | gibi: ack | |
| 09:06:11 | tkajinam | bauzas, the HW_CPU_X86_AMD_SEV trait is added even in compute nodes. https://github.com/openstack/nova/blob/master/nova/virt/libvirt/driver.py#L13223 | |
| 09:06:53 | tkajinam | bauzas, the trait hasn't been unused really in scheduling but it has been added to compute nodes with sev enabled. so it does not break scheduling to old compute nodes with SEV | |
| 09:06:54 | bauzas | oh, you're right, and in the reshape, you moved it to the nested RP | |
| 09:06:59 | tkajinam | yes | |
| 09:07:15 | bauzas | my brain forgot that while I was still reviewing it | |
| 09:07:20 | bauzas | sorry | |
| 09:07:23 | tkajinam | no, np | |
| 09:07:30 | bauzas | changing my vote then | |
| 09:07:49 | bauzas | tkajinam: I also have a slight concern, about some deprecation release value you change | |
| 09:07:55 | bauzas | not related to SEV | |
| 09:08:07 | tkajinam | I'll revert that change and push a new version | |
| 09:08:08 | tkajinam | one sec | |
| 09:08:19 | tkajinam | (the others are just rebased so +2 should be kept | |
| 09:09:30 | bauzas | cool | |
| 09:09:41 | opendevreview | Takashi Kajinami proposed openstack/nova master: Detect AMD SEV-ES support https://review.opendev.org/c/openstack/nova/+/925685 | |
| 09:09:42 | opendevreview | Takashi Kajinami proposed openstack/nova master: libvirt: Launch instances with SEV-ES memory encryption https://review.opendev.org/c/openstack/nova/+/926106 | |
| 09:09:42 | opendevreview | Takashi Kajinami proposed openstack/nova master: Add hw_mem_encryption_model image property https://review.opendev.org/c/openstack/nova/+/927706 | |
| 09:09:43 | opendevreview | Takashi Kajinami proposed openstack/nova master: Add functional test scenario for mixed SEV RPs https://review.opendev.org/c/openstack/nova/+/958562 | |
| 09:09:44 | bauzas | and thanks for being present | |
| 09:10:41 | tkajinam | np and thank you for the review ! | |
| 09:17:09 | gibi | sambork: bauzas: sean-k-mooney: dansmith: local testing of the image cache shows that we have a bug somewhere around executor shutdown as only one of the two hosts get the image if the executor size is 1, If I increase the executor size to 2 then both gets it. See logs in https://review.opendev.org/c/openstack/nova/+/957088/7#message-a1b56a536b88c1f4372a2d986364a1f61d5c75af | |
| 09:19:36 | bauzas | hmmm | |
| 09:19:52 | sambork | gibi, ack, checking | |
| 09:20:20 | gibi | If I have to guess then it is a difference in behavior in executor shutdown when there are queued tasks between eventlet and threading executors | |
| 09:20:42 | gibi | but I have to prove that... | |
| 09:24:30 | bauzas | tkajinam: saw my other concern about the nested RP deletion when the config changes ? | |
| 09:25:32 | bauzas | we should mention that in the docs | |
| 09:29:24 | opendevreview | Takashi Kajinami proposed openstack/nova master: Purge nested SEV RPs when SEV is disabled https://review.opendev.org/c/openstack/nova/+/958626 | |
| 09:29:34 | tkajinam | bauzas, yes I saw it... and think we can try ^^^ | |
| 09:30:33 | sambork | gibi, it maybe logic is a little bit different and in case of threadPoolExecutor we are not using work_queue in shutdown logic | |
| 09:30:36 | bauzas | I had that approach first when fixing the GPU SRIOV RPs | |
| 09:30:41 | tkajinam | bauzas, I noticed one difference between vGPU RP and SEV RP. The former is named according to definition in nova.conf while the other is named statically so purging SEV rp should be possible | |
| 09:30:59 | bauzas | tkajinam: yes, you're correct | |
| 09:31:16 | bauzas | we statically define the list of RPs based on what the op provided | |
| 09:31:30 | bauzas | while here, we automatically create the RPs based on what libvirt driver detects | |
| 09:32:11 | tkajinam | yeah | |
| 09:32:22 | bauzas | so yeah, we could do what you wrote (reporting total=0 so ensure_rps() will wipe it) | |
| 09:32:33 | bauzas | nice catch | |
| 09:32:37 | tkajinam | one challenge with vGPU is that it loose the name of RP when a record is completely removed from nova.conf | |
| 09:32:45 | tkajinam | I mean nova loose * | |
| 09:32:50 | bauzas | yes, this is config-driven | |
| 09:33:12 | tkajinam | s/loose/lose/ | |
| 09:33:13 | tkajinam | yeah | |