| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2026-08-27 | |||
| 08:16:21 | gibi | a bunch of stabilization bug fix landed. I now took https://bugs.launchpad.net/nova/+bug/2165150 as the next to work on | |
| 08:16:33 | opendevreview | Takashi Kajinami proposed openstack/nova master: libvirt: Add functional tests for TDX capability report https://review.opendev.org/c/openstack/nova/+/998525 | |
| 08:30:47 | antia | gibi: Amazing! Thanks for all your effort on this | |
| 08:32:01 | gibi | now that both TDX and SNP is in the tree I will buy a slice of cake after lunch. It was a nice cooperation across timezones, companies, and even vendors. I would like to see more such feature development in the future in openstack | |
| 08:33:17 | gibi | thanks tkajinam antia sean-k-mooney gmaan bauzas and others in the background who are not on IRC to make it happen | |
| 09:48:38 | opendevreview | Takashi Kajinami proposed openstack/nova master: libvirt: Add functional tests for TDX capability report https://review.opendev.org/c/openstack/nova/+/998525 | |
| 12:16:27 | gmaan | \o/ | |
| 12:24:18 | gibi | I think I found a race that is not test only but a real race in the compute(s) https://bugs.launchpad.net/nova/+bug/2165318 | |
| 12:35:46 | opendevreview | Ghanshyam Maan proposed openstack/nova master: Add event.set cleanup in test_graceful_shutdown https://review.opendev.org/c/openstack/nova/+/1002638 | |
| 12:38:35 | gmaan | gibi: ^^ followup for self.addCleanup event.set | |
| 12:40:05 | gibi | thanks | |
| 13:25:33 | opendevreview | Lajos Katona proposed openstack/nova master: libvirt: Preserve queue_size during live migration https://review.opendev.org/c/openstack/nova/+/995841 | |
| 13:52:26 | opendevreview | Merged openstack/nova master: Remove workaround for ServiceAuthTokenWrapper get_endpoint_data() https://review.opendev.org/c/openstack/nova/+/1002540 | |
| 14:29:09 | stephenfin | melwitt: I'd appreciate your eyes on https://review.opendev.org/c/openstack/nova/+/1002359 while you still have ksa know-how fresh in your head 🙏 | |
| 14:29:43 | melwitt | stephenfin: ah yes thanks for reminding. writing it down this time | |
| 14:49:37 | opendevreview | Stephen Finucane proposed openstack/nova master: conf: Deprecated http_retries options https://review.opendev.org/c/openstack/nova/+/1002657 | |
| 14:49:38 | opendevreview | Stephen Finucane proposed openstack/nova master: WIP: network: Simplify client creation https://review.opendev.org/c/openstack/nova/+/1002658 | |
| 15:10:20 | melwitt | gmaan: can I get a sanity check regarding a stable backport proposed? it would change a HTTP 403 to a HTTP 400. I believe this should be ok for stable bc it is remaining a 4xx status still, but wanted to check with you https://review.opendev.org/c/openstack/nova/+/996878 | |
| 15:11:23 | gmaan | melwitt: sure, will check after internal meeting. | |
| 15:11:51 | gmaan | melwitt: thanks for review on gate fixes, this one also ready https://review.opendev.org/c/openstack/nova/+/1002223 | |
| 15:12:06 | melwitt | cool thank you | |
| 15:12:31 | gmaan | melwitt: stephenfin and this one you had reviewed earlier, i need to rebase it due to code refactor on this https://review.opendev.org/c/openstack/nova/+/986477 | |
| 15:12:36 | melwitt | gmaan: thanks for the fixes! I'm very happy for the improvements. thanks I will review that one too | |
| 15:12:42 | gmaan | thanks | |
| 15:21:29 | melwitt | gmaan: regarding this review comment https://review.opendev.org/c/openstack/nova/+/1002223/comment/9997bc1c_41882d81/ doesn't that mean that the _wait_for_service_parameter will not actually wait at all? if the service reports "up" before the first heartbeat, wouldn't this _wait_for_service_parameter just return immediately seeing "up" and not actually wait? | |
| 15:23:01 | opendevreview | Stephen Finucane proposed openstack/nova master: conf: Deprecate http_retries options https://review.opendev.org/c/openstack/nova/+/1002657 | |
| 15:23:07 | opendevreview | Stephen Finucane proposed openstack/nova master: WIP: network: Simplify client creation https://review.opendev.org/c/openstack/nova/+/1002658 | |
| 15:23:33 | gmaan | melwitt: yes, it will immediately return as service is up. one issue i found where it was waiting was when we restart the service. in restart test failed as service did not came as 'up' just after restart | |
| 15:24:07 | gmaan | but yes, after service start, it will be up till first heartbeat is there or service_down_time elapsed | |
| 15:25:27 | melwitt | thanks | |
| 15:25:51 | melwitt | I wondered if it could somehow be a problem for the tests if it doesn't actually wait but I guess not | |
| 15:30:03 | stephenfin | melwitt: I agree that it would be good to document this. As noted on the review, my understanding is that it's okay to change an error response status code but not to add a new code | |
| 15:30:26 | stephenfin | hopefully I have that right | |
| 15:31:05 | melwitt | I think you are right | |
| 15:32:16 | melwitt | I had been thinking of something else where I wonder if it's not cool to go across code "levels" in certain directions. like it's cool to go from 500 => 400 but not to go from 400 => 500 | |
| 15:33:30 | melwitt | and going from a 4xx to a different 4xx stays within a level (in addition to being an error code that is already currently being returned) | |
| 15:58:40 | opendevreview | Clif Houck proposed openstack/nova master: perf(ironic): eliminate O(N²) ProviderTree deepcopy at startup https://review.opendev.org/c/openstack/nova/+/980676 | |
| 15:58:40 | opendevreview | Clif Houck proposed openstack/nova master: Parallelize per-node resource updates https://review.opendev.org/c/openstack/nova/+/980679 | |
| 16:13:47 | sean-k-mooney | we are alwasy allowed ot change a 500 to a 400 if it was previosly broken | |
| 16:14:10 | sean-k-mooney | 400->500 is just a bug | |
| 16:14:19 | sean-k-mooney | that we shoudl fix before intentully intoducing :) | |
| 16:14:57 | sean-k-mooney | https://docs.openstack.org/nova/latest/contributor/microversions.html#id2:~:text=Footnotes | |
| 16:15:34 | sean-k-mooney | ``` | |
| 16:15:36 | sean-k-mooney | When fixing 500 errors that previously caused stack traces, try to map the new error into the existing set of errors that API call could previously return (400 if nothing else is appropriate). Changing the set of allowed status codes from a request is changing the contract, and should be part of a microversion ``` | |
| 16:16:11 | sean-k-mooney | so 500 to any exisitng responce code is fine and 400, 403, 404 and 415 are alwasy allowed per footnote 2 | |
| 16:33:49 | gmaan | melwitt: on 403->400, yes it is all good to change it to existing code. if it is changing to same level but a new code then it is not backward compatible but it that changing code is existing one then it is ok. | |
| 16:42:06 | opendevreview | Merged openstack/nova master: ci: Remove GLANCE_STANDALONE https://review.opendev.org/c/openstack/nova/+/996322 | |
| 16:44:54 | melwitt | gmaan: awesome thanks | |
| 16:48:09 | melwitt | sean-k-mooney: yes I remember we can change from 5xx to any other error. but I was thinking about other "level" changes but there aren't really any good examples ... the example of 4xx => 5xx is obviously not ok | |
| 16:51:31 | sean-k-mooney | not without a microversion correct | |
| 16:51:47 | sean-k-mooney | but we shoudl not really return a 5xx in general | |
| 16:52:20 | melwitt | yeah, I think 5xx is never desired. that's if all other error handling falls through | |
| 16:52:22 | sean-k-mooney | at least not intentionally unless we are signaling a rate limiting issue or propagaging a 5xx we recived form a different service | |
| 16:52:43 | sean-k-mooney | well 500 is what you get when you hit an edge case we didng accoutn for right | |
| 16:53:49 | melwitt | generally I think so | |
| 17:28:02 | opendevreview | Merged openstack/nova master: libvirt: Drop redundant cast to str in python 3 https://review.opendev.org/c/openstack/nova/+/997810 | |
| 17:49:03 | opendevreview | Merged openstack/nova master: libvirt: Fail when file-backed memory is used with reservation https://review.opendev.org/c/openstack/nova/+/906161 | |
| 17:55:28 | opendevreview | Merged openstack/nova master: Fix race between RPC method completion and task tracking updates https://review.opendev.org/c/openstack/nova/+/1002432 | |
| 17:55:42 | opendevreview | Merged openstack/nova master: Add event.set cleanup in test_graceful_shutdown https://review.opendev.org/c/openstack/nova/+/1002638 | |
| 19:09:30 | gmaan | melwitt: replied to your question https://review.opendev.org/c/openstack/nova/+/1002223/comment/c5a84643_8a1c0fa7/ | |
| 19:09:55 | gmaan | sorry for being lazy to explain on why we need wait after start_compute. though it is only needed in _restart_compute and not in setUp | |
| 19:10:13 | gmaan | let me know if you prefer to remove it from setUp, i can respin it quickly | |
| 19:12:42 | gmaan | basically there is difference in service status until first heartbeat arrives for start fresh service vs restart existing service. where in former case, service initial status is 'up' and latter one is 'down'. once first heartbeat arrives it is as per the heartbeat and service_down_time | |
| 19:14:35 | melwitt | gmaan: haha thanks. yes that makes sense that it's necessary for the restart. I was confused by your last comment about tests failing without the wait because we were commenting at the setUp location in the code :) | |
| 19:15:20 | gmaan | yeah, i commented that in wrong place. let me remove it from setUp to avoid confusion in future | |
| 19:15:31 | melwitt | I kept thinking about it like, that doesn't make sense at this location. I understand now, you meant at the restart location | |
| 19:25:42 | opendevreview | Ghanshyam Maan proposed openstack/nova master: Fix graceful shutdown functional unstable tests https://review.opendev.org/c/openstack/nova/+/1002223 | |
| 19:26:29 | gmaan | melwitt: ^^ fixed it and also added notes | |
| 19:26:59 | melwitt | cool thanks! | |
| 21:33:58 | opendevreview | Merged openstack/nova master: perf(ironic): eliminate O(N²) ProviderTree deepcopy at startup https://review.opendev.org/c/openstack/nova/+/980676 | |
| 21:43:03 | opendevreview | Merged openstack/nova master: Add shebang to mark bash files https://review.opendev.org/c/openstack/nova/+/995213 | |
| #openstack-nova - 2026-08-28 | |||
| 00:23:08 | opendevreview | Merged openstack/nova master: Add functional test for cleanup_host cancelling queued live migrations https://review.opendev.org/c/openstack/nova/+/986477 | |
| 01:24:00 | opendevreview | Merged openstack/nova master: Replace OVN Metadata agent with OVN agent https://review.opendev.org/c/openstack/nova/+/987042 | |
| 06:20:22 | opendevreview | Merged openstack/nova master: Fix graceful shutdown functional unstable tests https://review.opendev.org/c/openstack/nova/+/1002223 | |
| 09:24:04 | opendevreview | Merged openstack/nova master: Remove tempest-pg-full https://review.opendev.org/c/openstack/nova/+/996887 | |
| 12:57:02 | tkajinam | bauzas, hi. could you review this test addition for tdx ? I had to update it according to some changes made in the tdx support implementation https://review.opendev.org/c/openstack/nova/+/998525 | |
| 13:23:11 | opendevreview | Daniel Schulz proposed openstack/nova master: Prevent orphaned attachments by preemptive delete https://review.opendev.org/c/openstack/nova/+/1002439 | |
| 13:30:23 | opendevreview | Ivan Anfimov proposed openstack/nova master: Replace outdated usage of egrep by grep https://review.opendev.org/c/openstack/nova/+/1002754 | |
| 13:31:43 | opendevreview | Ivan Anfimov proposed openstack/nova master: Replace outdated usage of egrep by grep https://review.opendev.org/c/openstack/nova/+/1002754 | |
| 15:25:28 | opendevreview | Lajos Katona proposed openstack/nova master: Use SDK for Neutron networks https://review.opendev.org/c/openstack/nova/+/928022 | |
| 15:28:15 | opendevreview | Lajos Katona proposed openstack/nova master: Use SDK for Neutron networks https://review.opendev.org/c/openstack/nova/+/928022 | |
| 15:49:03 | opendevreview | Daniel Schulz proposed openstack/nova master: Prevent orphaned attachments by preemptive delete https://review.opendev.org/c/openstack/nova/+/1002439 | |
| 18:15:31 | opendevreview | Merged openstack/nova master: Shutdown thread pool executors during graceful shutdown https://review.opendev.org/c/openstack/nova/+/1000211 | |
| 18:15:43 | opendevreview | Merged 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-29 | |||
| 10:38:12 | opendevreview | Stephen Finucane proposed openstack/nova master: tests: Remove errant CONF mocks https://review.opendev.org/c/openstack/nova/+/1002852 | |
| 11:30:45 | opendevreview | Merged openstack/nova master: libvirt: Add functional tests for TDX capability report https://review.opendev.org/c/openstack/nova/+/998525 | |
| 16:47:03 | opendevreview | Udayendu Kar proposed openstack/python-novaclient master: Fixed max-servers handling in host-evacuate-live https://review.opendev.org/c/openstack/python-novaclient/+/1002874 | |
| #openstack-nova - 2026-08-30 | |||
| 00:12:39 | opendevreview | Khoi proposed openstack/nova-specs master: Add availability-zone-scoped quota spec https://review.opendev.org/c/openstack/nova-specs/+/1002901 | |
| 04:59:23 | opendevreview | Takashi Kajinami proposed openstack/nova master: ovo: Drop deprecated item_clas argument https://review.opendev.org/c/openstack/nova/+/1002937 | |
| 05:01:43 | opendevreview | Takashi Kajinami proposed openstack/nova master: ovo: Drop deprecated item_cls argument https://review.opendev.org/c/openstack/nova/+/1002937 | |
| 05:20:23 | opendevreview | Takashi Kajinami proposed openstack/nova master: ovo: Drop deprecated item_cls argument https://review.opendev.org/c/openstack/nova/+/1002937 | |
| 06:00:02 | opendevreview | Takashi Kajinami proposed openstack/nova master: ovo: Drop deprecated item_clas argument https://review.opendev.org/c/openstack/nova/+/1002937 | |
| 15:15:40 | opendevreview | Takashi Kajinami proposed openstack/nova master: ovo: Drop deprecated item_clas argument https://review.opendev.org/c/openstack/nova/+/1002937 | |
| #openstack-nova - 2026-08-31 | |||
| 08:28:25 | gibi | looking at the functional threding job results we are definitely in a better shape now. There is one thing to monitor. I see TIMED_OUT results still after our stabilization fixes landed. And the timeout happnes during the functional test execution. It seems like we have a potentially hanging test case somewhere | |
| 08:29:07 | gibi | I'm not sure why the normal test case timeout does not hit and kill the test case | |
| 08:29:35 | gibi | but given that native threads are not killable from outside I'm not that surprised | |
| 08:29:35 | bauzas | excellent news, thanks gibi | |
| 08:30:55 | stephenfin | sean-k-mooney: Uggla: Would I be able to get some eyes on https://review.opendev.org/c/openstack/nova/+/1002852 today? It's blocking the u-c bump of ksa. gibi has already taken a look (thanks 🙏) | |
| 08:34:25 | sean-k-mooney | done | |
| 08:35:16 | sean-k-mooney | gibi: ya i rechecekd a patch twice over the weekend and it timed out in the functional tests both times | |