| *** erlon9 is now known as erlon | 00:04 | |
| opendevreview | Goutham Pacha Ravi proposed openstack/nova master: Support cold migration with virtiofs shares https://review.opendev.org/c/openstack/nova/+/989633 | 01:02 |
|---|---|---|
| opendevreview | Merged openstack/nova master: Stabilize functional test for slow instance.save https://review.opendev.org/c/openstack/nova/+/999121 | 01:59 |
| opendevreview | Goutham Pacha Ravi proposed openstack/nova master: Support cold migration with virtiofs shares https://review.opendev.org/c/openstack/nova/+/989633 | 06:15 |
| *** erlon1 is now known as erlon | 07:34 | |
| gibi | gmaan: I did some digging into the logs of https://bugs.launchpad.net/nova/+bug/2163552 it is strange as everything seems to correct, except that nova-api does not forwards the vif-plugged event to the compute even though the compute already waiting for it. | 09:09 |
| gibi | gmaan: funnily that sounds very similar to the downstream bug report you triaged recently https://redhat.atlassian.net/browse/OSPRH-31354 | 09:14 |
| gibi | and that is reported to Antelope | 09:14 |
| gibi | and we see it on master | 09:14 |
| gibi | crazy | 09:14 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: [DNM]: Troubleshoot race condition in unshelve vif plug https://review.opendev.org/c/openstack/nova/+/1001294 | 09:44 |
| gibi | gmaan: ^^ I added extra logs | 09:44 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: [DNM]: Troubleshoot race condition in unshelve vif plug https://review.opendev.org/c/openstack/nova/+/1001294 | 11:33 |
| opendevreview | Merged openstack/nova master: Make functional-py313-threading voting https://review.opendev.org/c/openstack/nova/+/997216 | 12:45 |
| dansmith | gibi: what is the purpose of the StaticallyDelayingCancellableTaskExecutorWrapper ? | 13:35 |
| dansmith | oh for filtering out stops when a start is right on the heels | 13:36 |
| opendevreview | Merged openstack/nova master: libvirt: Check only x86_64 arch for SEV capability detection https://review.opendev.org/c/openstack/nova/+/998285 | 13:47 |
| gibi | dansmith: yeah that is an old behavior of the libvirt virt driver | 13:51 |
| gibi | just eventlet had a native submit_after facility to handle that | 13:51 |
| dansmith | I remember discussing it now | 13:51 |
| dansmith | it's a very (impressively) long class name | 13:51 |
| gibi | I'm open to suggestions :) (we can still change futurist before the deadline);5;13~ | 13:52 |
| dansmith | are you building that into futurist? | 13:53 |
| gibi | yes | 13:58 |
| gibi | it is in now | 13:58 |
| gibi | waiting for a lib release to use it from there | 13:58 |
| gibi | https://github.com/openstack/futurist/commit/350e39d788a04fdbd38529564a0f44f3e395f80f | 13:58 |
| gibi | https://review.opendev.org/c/openstack/nova/+/997410 this is the patch that will usit if from futurist | 13:59 |
| dansmith | okay those seem like more reasonable names already :) | 13:59 |
| gibi | */use it from/ | 13:59 |
| gibi | thanks to Kamil :) | 13:59 |
| gibi | and thanks to the fact that the delay is not static now | 14:00 |
| gibi | we basically implemented submit_after for native thread pools | 14:00 |
| dansmith | cool | 14:07 |
| *** chandank` is now known as chandankumar | 14:08 | |
| opendevreview | Ashish Gupta proposed openstack/nova master: Fix init_host crash migration test under native threading https://review.opendev.org/c/openstack/nova/+/1001369 | 14:58 |
| gmaan | gibi: ok, i see, it seems similar to that downstream bug. I have some idea to add retry in server external event when it drop the network-vif-plugged but not sure if that will solve the actual root cause or hide. because form downstream bug, it was ~23 sec when resource claim happened (which set the host on instance) and nova-api still did not see the host set and dropped the event | 15:27 |
| gmaan | thanks for the adding more logging, will do multiple recheck if we can hit it | 15:28 |
| gibi | 23 seconds very long | 15:28 |
| gmaan | yeah | 15:28 |
| gibi | the current run of the DNM patch hit the issue but I haven't had the time to check the logs | 15:29 |
| *** M00SE1 is now known as M00SE | 15:30 | |
| gmaan | nice, will check in noon | 15:51 |
| opendevreview | Daniel Marques Caires proposed openstack/nova master: Add functional regression test for bug #2156927 https://review.opendev.org/c/openstack/nova/+/1000548 | 16:17 |
| opendevreview | Daniel Marques Caires proposed openstack/nova master: nova: cast set_host_enabled on disable https://review.opendev.org/c/openstack/nova/+/998464 | 16:17 |
| opendevreview | Daniel Marques Caires proposed openstack/nova master: Add functional regression test for bug #2156927 https://review.opendev.org/c/openstack/nova/+/1000548 | 16:20 |
| opendevreview | Daniel Marques Caires proposed openstack/nova master: nova: cast set_host_enabled on disable https://review.opendev.org/c/openstack/nova/+/998464 | 16:47 |
| opendevreview | Clif Houck proposed openstack/nova master: perf(ironic): eliminate O(N²) ProviderTree deepcopy at startup https://review.opendev.org/c/openstack/nova/+/980676 | 17:21 |
| opendevreview | Clif Houck proposed openstack/nova master: Parallelize per-node resource updates https://review.opendev.org/c/openstack/nova/+/980679 | 17:21 |
| opendevreview | Merged openstack/nova stable/2026.1: Restore 'fork' start method for daemon mode on Python 3.14 https://review.opendev.org/c/openstack/nova/+/999667 | 17:43 |
| gmaan | dansmith: I replied to the unify thread pool comments, I see the value of initializing executors during service start so I am ok to change it in this effort only instead of doing it again. we still have 2 weeks for FF, i hope that is enough time to review. but let me know if you are busy for other things to review before FF then I will add the missing test coverage and refactor the initialization part later. | 19:14 |
| gmaan | melwitt: this might fix the failure you saw for graceful shutdown job fail to stop service https://review.opendev.org/c/openstack/nova/+/1001017 | 19:23 |
| melwitt | ah cool, will look | 19:24 |
| gmaan | melwitt: and on top of it I improved that job tasks name so that it is easy to know which operation tests it is running https://review.opendev.org/c/openstack/nova/+/1001212 | 19:24 |
| gmaan | currently ansible top level block name are not appended in console logs so adding operation name in front of those tasks | 19:25 |
| gmaan | thanks | 19:25 |
| dansmith | gmaan: don't change it now. | 19:45 |
| gmaan | dansmith: ok | 19:48 |
| dansmith | I mean, you can change it now, but don't change it in this commit.. let's not jeopardize landing the functionality :) | 19:49 |
| gmaan | dansmith: got it, let me add the missing test coverage in this change and propose new one independent of graceful shutdown series for refactoring | 19:50 |
| dansmith | yup thanks | 19:51 |
| opendevreview | Ghanshyam Maan proposed openstack/nova master: Unify the thread pool executors into a central factory https://review.opendev.org/c/openstack/nova/+/998571 | 21:34 |
| gmaan | dansmith: ^^ a few jobs are still running in CI but it is ready | 22:38 |
Generated by irclog2html.py 4.1.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!