| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2026-08-20 | |||
| 15:28:18 | gmaan | dansmith: btw, this is ready. CI is flaky over other unrelated failure so zuul -1 https://review.opendev.org/c/openstack/nova/+/1000211/14 | |
| 15:28:46 | dansmith | ack | |
| 15:28:49 | gmaan | thanks | |
| 15:44:28 | dansmith | gouthamr: the log is too big to paste, as is the sql dump of course, so I emailed them to you and you can decide what to do witht hem | |
| 15:48:10 | gouthamr | ah! thank you dansmith, got that email - this is super helpful | |
| 15:49:39 | dansmith | gouthamr: using --unrestrict let me delete the stuck access rule. attempting to start the instance after that re-created the rule and the instance started (for the first time ever) | |
| 15:50:26 | dansmith | so that's good, I was able to get it working again with just admin/api stuff and no surgery.. IIRC, this would all have required surgery to resolve before you added the idempotent give-myself-access-always-on-start behavior | |
| 15:50:32 | dansmith | so a definite win for robustness, IMHO | |
| 15:50:33 | gouthamr | wow 😅 | |
| 16:06:25 | gouthamr | dansmith: ty for pushing for that robustness.. this looks a lot better than i initially intended it | |
| 16:06:37 | gouthamr | will get back to you about the m-shr issues | |
| 17:40:44 | opendevreview | Brian Haley proposed openstack/nova master: Change neutron API calls to use project_id https://review.opendev.org/c/openstack/nova/+/994321 | |
| 18:06:57 | gmaan | another bug (most prob tempest schema) happening in gate, reported https://bugs.launchpad.net/nova/+bug/2164677 | |
| 18:08:01 | gmaan | currently I am seeing these many failure happening in CI, not sure we have etherpad to track them ( Uggla may knows ) but listing here just in case anyone else want it to check it before recheck | |
| 18:08:09 | gmaan | https://bugs.launchpad.net/nova/+bug/2164606 | |
| 18:08:10 | gmaan | https://bugs.launchpad.net/nova/+bug/2163552 | |
| 18:08:16 | gmaan | https://bugs.launchpad.net/nova/+bug/2163448 | |
| 18:08:28 | gmaan | https://bugs.launchpad.net/nova/+bug/2160254 | |
| 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 | |