| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2026-08-20 | |||
| 18:08:35 | gmaan | https://bugs.launchpad.net/nova/+bug/2164677 | |
| 18:20:11 | melwitt | gmaan: I thought a schema validation failure would be 100% fail rate, so I'm a bit confused for this. the example linked in https://bugs.launchpad.net/nova/+bug/2164677 patch is also not touching any API area so it's not that either | |
| 18:22:44 | gmaan | melwitt: indeed. it is a clear error in tempest schema but not sure why it was/is not failing 100% there is 'instance_uuid' in required field instead of 'server_uuid' field https://github.com/openstack/tempest/blob/master/tempest/lib/api_schema/response/compute/v2_1/servers.py#L555 | |
| 18:26:09 | melwitt | gmaan: hm, it looks like it's /servers/{server_id}/migrations API has the 'server_uuid' | |
| 18:27:15 | melwitt | https://github.com/openstack/nova/blob/ec47550235877e82f529f6a8c99f32dd9ea9c1a8/nova/api/openstack/compute/schemas/server_migrations.py#L85 | |
| 18:27:33 | gmaan | melwitt: yeah, i think i got it. this schema is for list server in-progress live migration and most of time it may be empty so passing but this time it catch the in-progress live migration and hit the schema bug | |
| 18:27:45 | melwitt | ahhh | |
| 18:28:09 | melwitt | makes sense | |
| 18:28:16 | gmaan | here https://github.com/openstack/tempest/blob/master/tempest/api/compute/admin/test_live_migration.py#L477C9-L477C31 | |
| 18:28:46 | gmaan | will fix it but happy that this test is catching some in-progress live migration :) | |
| 18:29:26 | melwitt | yeah agreed | |
| 18:29:59 | melwitt | so yeah looks like in that case the tempest schema is wrong ... it should be server_uuid for the migrations API | |
| 18:32:27 | gmaan | yeah fixing https://review.opendev.org/c/openstack/tempest/+/1001722 | |
| 19:12:24 | gmaan | btw added these gate failures in this etherpad https://etherpad.opendev.org/p/nova-ci-failures-minimal | |
| 19:18:51 | melwitt | ++ | |
| 19:19:06 | melwitt | (aside: looks like i need to clean up some really old stuff on there) | |
| 20:16:56 | opendevreview | Brian Haley proposed openstack/nova master: Change neutron API calls to use project_id https://review.opendev.org/c/openstack/nova/+/994321 | |
| 20:32:18 | gmaan | Gmaan@openrh28 | |
| 20:33:09 | gmaan | oops :) | |
| 21:11:13 | opendevreview | Merged openstack/nova master: Unify the thread pool executors into a central factory https://review.opendev.org/c/openstack/nova/+/998571 | |
| 21:13:10 | opendevreview | Ghanshyam Maan proposed openstack/nova master: Add doc and release notes for the graceful shutdown task tracking https://review.opendev.org/c/openstack/nova/+/997141 | |
| #openstack-nova - 2026-08-21 | |||
| 02:35:39 | opendevreview | Brian Haley proposed openstack/nova master: Change neutron API calls to use project_id https://review.opendev.org/c/openstack/nova/+/994321 | |
| 05:37:48 | opendevreview | Goutham Pacha Ravi proposed openstack/nova master: Extract share management from ComputeManager https://review.opendev.org/c/openstack/nova/+/999349 | |
| 05:37:48 | opendevreview | Goutham Pacha Ravi proposed openstack/nova master: Support cold migration with virtiofs shares https://review.opendev.org/c/openstack/nova/+/989633 | |
| 07:38:58 | antia | gmaan: Thanks for reviewing the TDX series. I replied to some of the comments and will make changes to address the rest. | |
| 13:13:13 | sean-k-mooney | antia: are you expecting to push new version of the patches today. i had hoped you would have prepared them on wednesday orginally? | |
| 13:14:26 | antia | sean-k-mooney: yes, just finalizing now | |
| 13:28:58 | sean-k-mooney | ack ill try an loop back to them so before i drop fo rthe weekend | |
| 13:41:16 | opendevreview | Anton Iacobaeus proposed openstack/nova master: libvirt: Add Intel TDX host capability detection https://review.opendev.org/c/openstack/nova/+/994421 | |
| 13:41:17 | opendevreview | Anton Iacobaeus proposed openstack/nova master: libvirt: Generalize memory encryption config https://review.opendev.org/c/openstack/nova/+/998466 | |
| 13:41:17 | opendevreview | Anton Iacobaeus proposed openstack/nova master: libvirt: Add memory encryption config for TDX https://review.opendev.org/c/openstack/nova/+/998608 | |
| 13:41:18 | opendevreview | Anton Iacobaeus proposed openstack/nova master: libvirt: Launch security Intel TDX https://review.opendev.org/c/openstack/nova/+/999249 | |
| 13:41:18 | opendevreview | Anton Iacobaeus proposed openstack/nova master: doc: Add documentation for Intel TDX https://review.opendev.org/c/openstack/nova/+/1000081 | |
| 13:41:19 | opendevreview | Anton Iacobaeus proposed openstack/nova master: libvirt: reject_mem_enc_instances https://review.opendev.org/c/openstack/nova/+/1000082 | |
| 13:41:19 | opendevreview | Anton Iacobaeus proposed openstack/nova master: libvirt: Add os_firmware_secure https://review.opendev.org/c/openstack/nova/+/1000472 | |
| 13:41:20 | opendevreview | Anton Iacobaeus proposed openstack/nova master: libvirt: Check only x86_64 arch for Intel TDX https://review.opendev.org/c/openstack/nova/+/1000805 | |
| 13:41:49 | antia | sean-k-mooney: submitted a new version of the patches now, but this is without moving the os_firmware_secure patch | |
| 13:42:06 | sean-k-mooney | ack | |
| 13:42:16 | sean-k-mooney | that order is ok was well | |
| 13:42:36 | sean-k-mooney | its a littel cleaner the other way but it shoudl be fien as is | |
| 13:42:49 | sean-k-mooney | the how woudl be to merge the full series before FF in anycase | |
| 13:44:12 | sean-k-mooney | ill need to to do one or two things before i have tiem to re reivew but ill try an do a pass over them in an hour or so | |
| 13:47:00 | antia | thanks! | |
| 13:51:31 | opendevreview | Clif Houck proposed openstack/nova master: perf(ironic): eliminate O(N²) ProviderTree deepcopy at startup https://review.opendev.org/c/openstack/nova/+/980676 | |
| 13:51:32 | opendevreview | Clif Houck proposed openstack/nova master: Parallelize per-node resource updates https://review.opendev.org/c/openstack/nova/+/980679 | |
| 14:01:41 | sean-k-mooney | antia: one thing to keep in mind, in openstack we try to ensure that every commit is indepently correct | |
| 14:01:52 | sean-k-mooney | because we want master to be constaly deployable into production | |
| 14:02:16 | sean-k-mooney | so we allow follow up patches but only when they are not requried for correctness | |
| 14:02:30 | antia | that makes sense | |
| 14:02:52 | sean-k-mooney | that why even if its fixed in a later patch if its a correctness issue we as for it to be fixed in teh relevent patch that intoduced the issue just as wider context | |
| 14:03:25 | sean-k-mooney | that partly because we use gerrit but more because we do not belive in squash merging series | |
| 14:16:27 | dansmith | gmaan: looks like we're in good shape on the graceful patches.. I'll try to hit the last one today | |
| 14:56:42 | opendevreview | Anton Iacobaeus proposed openstack/nova master: libvirt: Add Intel TDX host capability detection https://review.opendev.org/c/openstack/nova/+/994421 | |
| 14:56:42 | opendevreview | Anton Iacobaeus proposed openstack/nova master: libvirt: Generalize memory encryption config https://review.opendev.org/c/openstack/nova/+/998466 | |
| 14:56:43 | opendevreview | Anton Iacobaeus proposed openstack/nova master: libvirt: Add memory encryption config for TDX https://review.opendev.org/c/openstack/nova/+/998608 | |
| 14:56:43 | opendevreview | Anton Iacobaeus proposed openstack/nova master: libvirt: Add os_firmware_secure https://review.opendev.org/c/openstack/nova/+/1000472 | |
| 14:56:44 | opendevreview | Anton Iacobaeus proposed openstack/nova master: libvirt: Launch security Intel TDX https://review.opendev.org/c/openstack/nova/+/999249 | |
| 14:56:44 | opendevreview | Anton Iacobaeus proposed openstack/nova master: doc: Add documentation for Intel TDX https://review.opendev.org/c/openstack/nova/+/1000081 | |
| 14:56:45 | opendevreview | Anton Iacobaeus proposed openstack/nova master: libvirt: reject_mem_enc_instances https://review.opendev.org/c/openstack/nova/+/1000082 | |
| 14:56:45 | opendevreview | Anton Iacobaeus proposed openstack/nova master: libvirt: Check only x86_64 arch for Intel TDX https://review.opendev.org/c/openstack/nova/+/1000805 | |
| 15:03:16 | gmaan | antia: thanks, will check today. i finished till doc yesterday but that is most of the key thing and last two were good maybe, | |
| 15:03:23 | gmaan | dansmith: ack, thanks | |
| 15:03:59 | gmaan | dansmith: and this doc/releasenotes one is last https://review.opendev.org/c/openstack/nova/+/997141/10 | |
| 15:09:35 | dansmith | gmaan: yep, I know | |
| 15:10:37 | gmaan | dansmith: k. I will fix this if i re-spin it otherwise sometime later https://review.opendev.org/c/openstack/nova/+/1000211/comment/734dd672_9a340618/ | |
| 15:10:50 | gmaan | or you want to push me a followup in thtis series? | |
| 15:12:18 | dansmith | not critical, see what other people think | |
| 15:15:01 | gmaan | ack | |
| 15:17:33 | dansmith | ugh, I can't put my finger on it right now.. Uggla when is FF? | |
| 15:18:09 | Uggla | Thursday next week | |
| 15:18:18 | gmaan | Aug 28, end of next week | |
| 15:18:19 | dansmith | duh, linked from the top of the H page | |
| 15:18:35 | sean-k-mooney | the non client lib freze shoudl have been yesterday i think as well | |
| 15:18:53 | sean-k-mooney | i was about to check if the os-tirat release we are waitign on happend? | |
| 15:18:57 | dansmith | ack, so I think we have plenty of time for gouthamr to at least try for a follow-up patch for reconciling missed share access rules | |
| 15:18:57 | sean-k-mooney | Uggla: do you know? | |
| 15:19:20 | dansmith | Uggla: I assume you would be up for a FFE if necessary to get that over the line if the rest of it is landed? | |
| 15:19:42 | gmaan | antia: sean-k-mooney on TDX one, before you both goes offline. one thing to discuss. i have to check the updated series but i cannot see VM create is rejected if os_secure_boot=required and TDX requested, https://review.opendev.org/c/openstack/nova/+/999249/5/nova/virt/libvirt/driver.py#7967 | |
| 15:19:56 | Uggla | dansmith yes if we have a chance to do it. | |
| 15:19:58 | gmaan | is update/re-aranging series done or shoud i wait | |
| 15:20:49 | sean-k-mooney | Uggla: looks like the last release was 2 months ago so im going to go request a new release now | |
| 15:20:51 | sean-k-mooney | https://github.com/openstack/releases/blob/master/deliverables/_independent/os-traits.yaml#L115-L118 | |
| 15:21:27 | Uggla | sean-k-mooney, I don't know. But things are probably lagging a bit because Elod is on pto | |
| 15:22:09 | dansmith | Uggla: AFAIK we're now just waiting for you to circle back on the exception handling comment here: https://review.opendev.org/c/openstack/nova/+/999349 | |
| 15:22:23 | sean-k-mooney | well sylvain is the release liason so they would have been teh one to propeose it normally but they are also on pto | |
| 15:22:28 | sean-k-mooney | rather then elod | |
| 15:22:30 | dansmith | otherwise I think I'm +2 on both assuming gouthamr will propose a followup for a periodic resync of the share access rules | |
| 15:24:50 | gmaan | antia: this one https://review.opendev.org/c/openstack/nova/+/999249/3..5/nova/virt/libvirt/driver.py#b7976 | |
| 15:25:25 | gmaan | i was expecting we reject the request when we find os_secure_boot is required and tdx-requested otherwise it still ignore it. no? | |
| 15:26:04 | Uggla | dansmith, thanks I'm gonna answer. Did you test with nfs or cephfs ? | |
| 15:26:14 | dansmith | nfs | |
| 15:27:39 | dansmith | Uggla: I put notes in the comments about the things I tested (although notably left out which driver I was using) | |
| 15:27:39 | antia | gmaan: There is a follow up patch, which I now moved to be in front, which solves the secure boot interaction with TDX | |
| 15:27:43 | Uggla | ok I have just setup a cephfs devstack for testing it. That should not fundamentally change. But it will be another check. | |
| 15:28:17 | dansmith | very good | |
| 15:28:20 | antia | https://review.opendev.org/c/openstack/nova/+/1000472 with this it shouldn't be required to reject on os_secure_boot=required for TDX | |
| 15:29:32 | gmaan | antia: ok, let me go through the whole series again, will do after sometime. btw which TZ you are in? | |
| 15:29:33 | sean-k-mooney | did gibi or you have time to test that secure boot actully worked in that version | |
| 15:30:06 | antia | gmaan: UTC+2, so just about to head off for the weekend :) | |
| 15:30:09 | Uggla | dansmith, I would have progressed more without yesterday escalation. | |