| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2026-08-19 | |||
| 20:56:47 | opendevreview | Ghanshyam Maan proposed openstack/nova master: Create services required thread pool executors during service initialization https://review.opendev.org/c/openstack/nova/+/1001572 | |
| 21:01:04 | gmaan | dansmith: gibi: ^^ this is the further refactoring of thread pool executors as discussed in original change | |
| 21:20:16 | opendevreview | Ghanshyam Maan proposed openstack/nova master: Shutdown thread pool executors during graceful shutdown https://review.opendev.org/c/openstack/nova/+/1000211 | |
| #openstack-nova - 2026-08-20 | |||
| 03:03:02 | opendevreview | Ghanshyam Maan proposed openstack/nova master: Shutdown thread pool executors during graceful shutdown https://review.opendev.org/c/openstack/nova/+/1000211 | |
| 03:57:00 | opendevreview | Ghanshyam Maan proposed openstack/nova master: Create services required thread pool executors during service initialization https://review.opendev.org/c/openstack/nova/+/1001572 | |
| 04:00:55 | gmaan | gibi: i saw the same notification test failing again, reported the bug with trackback and link https://bugs.launchpad.net/nova/+bug/2164606 | |
| 04:01:29 | opendevreview | Ghanshyam Maan proposed openstack/nova master: Create services required thread pool executors during service initialization https://review.opendev.org/c/openstack/nova/+/1001572 | |
| 13:38:36 | dansmith | gouthamr: I've restarted m-shr and no reconcile has happened. I've captured the logs for the service during the restart, a dump of the manila database, and a capture of what osc shows me | |
| 13:38:51 | dansmith | gouthamr: anything else you want me to capture (other service logs) before I force fix the share and move on? | |
| 14:21:38 | gouthamr | dansmith: o/ ty, looking | |
| 14:57:04 | opendevreview | Brian Haley proposed openstack/nova master: Change neutron API calls to use project_id https://review.opendev.org/c/openstack/nova/+/994321 | |
| 15:22:17 | gouthamr | dansmith: in meetings and got distracted; did you have the log posted somewhere? i can take a look at it and i'm thinking we have an eventlet bug.. I don't need anyhing else though, so yes.. force-fix the share with "access delete --unrestrict" as admin | |
| 15:22:50 | dansmith | gouthamr: no, I was waiting to see if you wanted me to capture anything else.. I'll post what I have in a sec then | |
| 15:23:01 | gouthamr | dansmith: i reported this https://bugs.launchpad.net/manila/+bug/2164609 and started investigating.. my hunch is this has to do with the eventlet backend (default today) | |
| 15:23:31 | gouthamr | dansmith: tyty | |
| 15:23:51 | dansmith | gouthamr: the logs and things could go in the bug.. do you want the DB dump there as well (or not at all?) | |
| 15:26:17 | dansmith | oh, is that bug for something more specific and thus not quite appropriate? | |
| 15:26:29 | dansmith | FWIW, it does seem like restarting the service takes a long time, like it's waiting for something | |
| 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 | |