| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2026-06-12 | |||
| 08:02:22 | opendevreview | Merged openstack/nova master: libvirt: Add conf option for live migration monitor interval https://review.opendev.org/c/openstack/nova/+/937223 | |
| 08:41:53 | sean-k-mooney | sambork: gibi im correct in thinkign we are still defaulting to eventlet mode for the novnc proxy currently right? | |
| 08:42:17 | sean-k-mooney | i know the plan is to update that to threaded but master today is eventlet ya? | |
| 08:42:26 | sambork | yes | |
| 08:42:30 | sean-k-mooney | cool | |
| 08:44:07 | sean-k-mooney | i have ubuntu 26.04 workign in devstack but the 2 failign test are for the novnc proxy | |
| 08:44:09 | sean-k-mooney | https://e5392d8b5c67a9e24e1f-aa881d493e66ac85e397d3a18f294794.ssl.cf5.rackcdn.com/openstack/a154d2bf5825493cb4ae423d6e989638/testr_results.html | |
| 08:44:23 | sean-k-mooney | and its ths stame test failure i got in eventlet mode with uv on python 3.14 | |
| 08:44:39 | sean-k-mooney | so i just need to swap to threaded mode and that will start passing | |
| 08:45:45 | sambork | yeah also there is this patch https://review.opendev.org/c/openstack/nova/+/987815/6 which should fix this issue based on ci run with 3.14 | |
| 08:46:35 | sean-k-mooney | ah the fork one. ya i could add that as a depens on | |
| 08:47:11 | sean-k-mooney | but i think ill just swap to threading mode for now and i can add a followup to drop the override that depends on that | |
| 08:48:56 | sean-k-mooney | ok ill make bot chagnes to my devstack patch and add a followup to use that thanks | |
| 08:49:27 | sambork | cool, np! | |
| 09:11:45 | opendevreview | Joan Gilabert proposed openstack/nova master: Add mtty support to nova-next https://review.opendev.org/c/openstack/nova/+/922140 | |
| 10:05:31 | opendevreview | Balazs Gibizer proposed openstack/nova stable/2025.2: [whitebox]Turn off iothreads feature https://review.opendev.org/c/openstack/nova/+/992966 | |
| 10:08:40 | opendevreview | Balazs Gibizer proposed openstack/nova stable/2025.1: [whitebox]Turn off iothreads feature https://review.opendev.org/c/openstack/nova/+/992967 | |
| 10:16:16 | opendevreview | Karol Klimaszewski proposed openstack/nova-specs master: Spec for SPDK-based nova-provisioned storage backend https://review.opendev.org/c/openstack/nova-specs/+/985676 | |
| 12:02:37 | opendevreview | Lajos Katona proposed openstack/nova master: Use SDK for Neutron networks https://review.opendev.org/c/openstack/nova/+/928022 | |
| 12:12:45 | opendevreview | Lajos Katona proposed openstack/nova master: Use SDK for Neutron networks https://review.opendev.org/c/openstack/nova/+/928022 | |
| 12:12:54 | opendevreview | Lajos Katona proposed openstack/nova master: Use SDK for Neutron subnets https://review.opendev.org/c/openstack/nova/+/962190 | |
| 12:13:14 | opendevreview | Lajos Katona proposed openstack/nova master: Use SDK for Neutron extensions https://review.opendev.org/c/openstack/nova/+/962270 | |
| 12:50:31 | opendevreview | Ashish Gupta proposed openstack/placement master: tests: Add connection parameter to Database and PlacementFixture https://review.opendev.org/c/openstack/placement/+/993106 | |
| 13:05:36 | opendevreview | Balazs Gibizer proposed openstack/nova stable/2025.2: [whitebox]Turn off iothreads feature https://review.opendev.org/c/openstack/nova/+/992966 | |
| 13:05:37 | opendevreview | Balazs Gibizer proposed openstack/nova stable/2025.2: [whitebox]Disable vtpm live migration https://review.opendev.org/c/openstack/nova/+/993109 | |
| 13:09:41 | opendevreview | Lajos Katona proposed openstack/nova master: Use SDK for Neutron floating IPs https://review.opendev.org/c/openstack/nova/+/962604 | |
| 13:12:19 | melwitt | sean-k-mooney: a couple of patches you might be interested, flaky unit test fix https://review.opendev.org/c/openstack/nova/+/993059 and add irrelevant-files for nova-graceful-shutdown job https://review.opendev.org/c/openstack/nova/+/993063 | |
| 13:19:04 | opendevreview | Balazs Gibizer proposed openstack/nova stable/2025.2: [stable-only][whitebox]Disable vtpm live migration https://review.opendev.org/c/openstack/nova/+/993109 | |
| 13:19:05 | opendevreview | Balazs Gibizer proposed openstack/nova stable/2025.2: [stable-only][whitebox]Turn off iothreads feature https://review.opendev.org/c/openstack/nova/+/992966 | |
| 13:19:24 | sean-k-mooney | melwitt: i saw them last night but decied post midnight code revew were not ideal :) | |
| 13:19:31 | sean-k-mooney | melwitt: but sure ill take a look | |
| 13:19:54 | melwitt | heh np | |
| 13:19:55 | opendevreview | Balazs Gibizer proposed openstack/nova stable/2025.1: [stable-only][whitebox]Turn off iothreads feature https://review.opendev.org/c/openstack/nova/+/992967 | |
| 13:20:22 | sean-k-mooney | melwitt: i htink gibi filed a bug for one of the flaky tests last week by the way | |
| 13:20:37 | sean-k-mooney | has that one already been fixed | |
| 13:20:45 | melwitt | oh really, let me check | |
| 13:20:55 | sean-k-mooney | it was related to the nfs cching change | |
| 13:21:33 | melwitt | oh yeah it's the same thing oops. let me mark as duplicate I guess | |
| 13:21:59 | sean-k-mooney | https://bugs.launchpad.net/nova/+bug/2155773 | |
| 13:22:03 | sean-k-mooney | but ya either is fine | |
| 13:22:17 | sean-k-mooney | i just notice it when they mentioned it on irc in my morning | |
| 13:22:39 | sean-k-mooney | but i didnt have a change to look at it beofre you and gmaan had already found it | |
| 13:23:23 | sean-k-mooney | @mock.patch('nova.virt.libvirt.driver.os.open', return_value=99) | |
| 13:23:23 | melwitt | yeah I wonder, will the gerrit Closes-Bug automation still work if I close the one I opened as a duplicate | |
| 13:23:45 | sean-k-mooney | ya so this is oen of the resaon i said mocking open is tricky | |
| 13:24:08 | gibi | melwitt: thanks for fixing that flakyness | |
| 13:24:31 | melwitt | sorry for introducing it /o\ | |
| 13:25:21 | gibi | nah, that was all of us reviewing and accepting it ;) | |
| 13:25:47 | sean-k-mooney | melwitt: both of those look fine to me so +w | |
| 13:26:33 | melwitt | thanks ! | |
| 13:31:18 | melwitt | sorting the gate-failure bug tag by most recent, I'm reminded I had proposed this other fix for a ceph memory allocation error https://review.opendev.org/c/openstack/nova/+/969538 | |
| 13:48:16 | opendevreview | Ashish Gupta proposed openstack/nova master: tests: use file-backed Placement SQLite in functional threading https://review.opendev.org/c/openstack/nova/+/992581 | |
| 13:48:17 | opendevreview | Dan Smith proposed openstack/nova master: Add unpin AZ functionality https://review.opendev.org/c/openstack/nova/+/990682 | |
| 13:48:18 | opendevreview | Dan Smith proposed openstack/nova master: Doc changes for unpin-az https://review.opendev.org/c/openstack/nova/+/992469 | |
| 13:48:48 | jgilaber | melwitt, Hi! I've been discussing with sean-k-mooney about this patch https://review.opendev.org/c/openstack/nova/+/922140 and would like your inpute as well | |
| 13:49:05 | jgilaber | I'm working on getting the whitebox vgpu tests running with mtty | |
| 13:50:22 | sean-k-mooney | jgilaber: melwitt so basiclly my comemnt was since nova-next is getting quite long, and since nova-vtpm is already using whitebox | |
| 13:50:31 | sean-k-mooney | why not rename that to say nova-whitebox | |
| 13:50:42 | sean-k-mooney | and just add the vgpu/mtty testing to that | |
| 13:51:22 | sean-k-mooney | then we have 1 whitebox job that we can selectivly enabel test in like the recent iothread ones gibi is workign on | |
| 13:52:37 | sean-k-mooney | we have a weakly perodic for whitebox which ahs been pretty stabel so i think we could effectily promote it to check and or remove it rom the perodic line too | |
| 13:52:38 | melwitt | yeah I think that sounds reasonable | |
| 13:53:24 | melwitt | nova-vtpm is a good candidate for rename + expand since it's very small currently | |
| 13:53:32 | sean-k-mooney | jgilaber: out of interst did the boot issues go away when you swap form the 64mb flavor to the standard sized ones? | |
| 13:54:11 | sean-k-mooney | melwitt: i have not had time to look but i also wonder what test in nova-next shoudl be in say the integrated comptue jobs or others | |
| 13:54:34 | sean-k-mooney | melwitt: i.e. is there any clean up on the options or thigns that are now default that are coverd elsewhere | |
| 13:55:01 | jgilaber | sean-k-mooney, the ssh seems to work fine after doubling the memory | |
| 13:55:07 | melwitt | sean-k-mooney: yeah, it's likely time for an audit there | |
| 13:55:28 | sean-k-mooney | i noticed it average runtim was over 2.5 hours | |
| 13:55:40 | sean-k-mooney | most of the other are 1-2 | |
| 13:55:57 | sean-k-mooney | so if we could get it below 2 that would be nice | |
| 13:56:25 | sean-k-mooney | given zuul reports only after the slowest job | |
| 14:14:31 | opendevreview | Dan Smith proposed openstack/nova master: Doc changes for unpin-az https://review.opendev.org/c/openstack/nova/+/992469 | |
| 14:14:31 | opendevreview | Dan Smith proposed openstack/nova master: Add unpin AZ functionality https://review.opendev.org/c/openstack/nova/+/990682 | |
| 14:15:03 | dansmith | melwitt: I had opened gmaan's comment when he made it and didn't see yours about the update tests, so this moves that down ^ | |
| 14:15:30 | dansmith | as noted it's a bit annoying that they're named that because I had missed them in my grepping of the file but alas | |
| 14:19:56 | opendevreview | Joan Gilabert proposed openstack/nova master: Add mtty/mdpy support for testing fake mdevs https://review.opendev.org/c/openstack/nova/+/898100 | |
| 14:19:56 | opendevreview | Joan Gilabert proposed openstack/nova master: move compile earlier https://review.opendev.org/c/openstack/nova/+/950516 | |
| 14:19:57 | opendevreview | Joan Gilabert proposed openstack/nova master: Add mtty support to nova-next https://review.opendev.org/c/openstack/nova/+/922140 | |
| 14:21:13 | melwitt | dansmith: oh yeah I didn't notice it at first either and then was like wait a minute ... cool | |
| 14:47:32 | opendevreview | Joan Gilabert proposed openstack/nova master: Add mtty support to nova-next https://review.opendev.org/c/openstack/nova/+/922140 | |
| 14:53:46 | opendevreview | Konrad Gube proposed openstack/nova-specs master: Propose assisted-volume-extend API https://review.opendev.org/c/openstack/nova-specs/+/949504 | |
| 15:17:57 | opendevreview | Joan Gilabert proposed openstack/nova master: Rename vtpm job and add mtty support for vpgu test https://review.opendev.org/c/openstack/nova/+/922140 | |
| 15:24:41 | opendevreview | Lajos Katona proposed openstack/nova master: Use SDK for Neutron Ports https://review.opendev.org/c/openstack/nova/+/969298 | |
| 15:25:02 | opendevreview | Lajos Katona proposed openstack/nova master: Use SDK for Neutron security-groups https://review.opendev.org/c/openstack/nova/+/981141 | |
| 15:44:13 | opendevreview | Joan Gilabert proposed openstack/nova master: Rename vtpm job and add mtty support for vpgu test https://review.opendev.org/c/openstack/nova/+/922140 | |
| 15:49:50 | opendevreview | melanie witt proposed openstack/nova master: Fix intermittent test_refresh_nfs_swap_cache_stale failure https://review.opendev.org/c/openstack/nova/+/993059 | |
| 15:54:11 | sean-k-mooney | sambork: so just enabling threaded mode didnt fix the novnc test on 3.14 with ubuntu 26.04 but https://review.opendev.org/c/openstack/nova/+/987815 did | |
| 15:54:26 | sean-k-mooney | so restoring fork seams to work | |
| 15:55:07 | sean-k-mooney | with threaded mode it fixed 1 of the these but the other got the token expired thing you were seeing on ubuntu 24.04 | |
| 15:55:46 | sean-k-mooney | so for now i think doing multiprocessing.set_start_method('fork', force=True) uncondtionaly regarless fo eventlet or threaidng is the right move | |
| 15:56:05 | sean-k-mooney | until we fiture out how to make novnc without that in general | |
| 16:00:21 | melwitt | sean-k-mooney: had to change approach on ^ due to it still failing bc the scoping of the mock did not actually help bc they apparently all resolve to the same place in the end | |
| 16:00:40 | sean-k-mooney | yes they do | |
| 16:00:41 | melwitt | I'll be rechecking it some times | |
| 16:00:49 | sean-k-mooney | did you swap to useign mock_open | |
| 16:01:43 | melwitt | no, used list.count() to assert the specified fd was called close with once. a claude idea that _seems_ ok afaict | |
| 16:01:49 | sean-k-mooney | yep just saw | |
| 16:01:55 | melwitt | to just ignore the rest. yeah | |
| 16:02:08 | sean-k-mooney | ill pull that patch locally and run it in a loop on one of my systems for like 30 mins | |