| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2026-06-11 | |||
| 19:08:06 | gmaan | dansmith: melwitt commented, you do not need to have all of them if you just run the update server tests. | |
| 19:08:29 | dansmith | gmaan: so why are they there for like 2.98 which also didn't change rebuild? | |
| 19:09:15 | gmaan | dansmith: they did, response of GET server, update, and rebuild is changed all together | |
| 19:09:48 | melwitt | yeah. happens often with responses bc so many response schema are shared among APIs | |
| 19:09:54 | gmaan | dansmith: api sample tests expect all the APi sample file in same microversion dir if tests are running for that microversion even API is changed or not | |
| 19:10:10 | gmaan | yeah, they are shared from view builder | |
| 19:10:14 | melwitt | req schema should be minimal blast radius | |
| 19:10:17 | dansmith | okay so wait | |
| 19:10:41 | dansmith | "expect all the APi sample file in same microversion dir if tests are running for that microversion even API is changed or not" is what I figured, which is that I didn't change server response but still need the sample | |
| 19:10:48 | gmaan | in unpin az, we should only need req and resp file of PUT server only | |
| 19:11:25 | gmaan | dansmith: because you are running test as part of ServersSampleJson2104Test https://review.opendev.org/c/openstack/nova/+/990682/3/nova/tests/functional/api_sample_tests/test_servers.py#749 | |
| 19:11:42 | gmaan | i mean tests for post, get servers | |
| 19:11:43 | dansmith | ohh, ok I see | |
| 19:11:52 | dansmith | I was going to say.. doesn't work without, but I see your point | |
| 19:11:59 | dansmith | if I change the base | |
| 19:12:39 | gmaan | yeah, we do not need to run any other test than update sevrer in this microversion | |
| 19:21:28 | opendevreview | Merged openstack/nova stable/2025.1: Fix resize to unpermitted flavors https://review.opendev.org/c/openstack/nova/+/988154 | |
| 22:26:56 | melwitt | looks like I have another flaky test test_refresh_nfs_swap_cache_stale 😩 | |
| 22:45:07 | dansmith | gmaan: yeah I was just missing that inheriting from that test class is what was making it require them all for me and I figured that since the last few also had all the samples that I needed them | |
| 22:45:09 | dansmith | but makes sense not | |
| 22:45:11 | dansmith | *now | |
| 23:01:55 | opendevreview | melanie witt proposed openstack/nova master: Fix intermittent test_refresh_nfs_swap_cache_stale failure https://review.opendev.org/c/openstack/nova/+/993059 | |
| 23:05:04 | opendevreview | melanie witt proposed openstack/nova master: Fix intermittent test_refresh_nfs_swap_cache_stale failure https://review.opendev.org/c/openstack/nova/+/993059 | |
| 23:19:06 | melwitt | gmaan: ^ another flaky test shame fix above | |
| 23:45:50 | melwitt | huh I noticed nova-graceful-shutdown job is running on that unit test only change, perhaps needs some irrelevant-files in .zuul.yaml? | |
| 23:48:20 | gmaan | melwitt: lgtm, +2 | |
| 23:48:27 | melwitt | thanks gmaan | |
| 23:48:46 | gmaan | melwitt: oh is it? i thought i used the same irrelevant file there like other jobs | |
| 23:49:20 | melwitt | it inherits from devstack-multinode that might be why it doesn't have | |
| 23:50:05 | gmaan | oh, yeah, no irrelevant file in nova side https://github.com/openstack/nova/blob/master/.zuul.yaml#L967 | |
| 23:50:34 | melwitt | that aside, the job is running pretty fast so | |
| 23:50:46 | melwitt | is beating openstack-tox-cover lol | |
| 23:50:57 | melwitt | *it's | |
| #openstack-nova - 2026-06-12 | |||
| 00:45:08 | opendevreview | Ghanshyam Maan proposed openstack/nova master: Add irrelevant-files for the graceful shutdown jobs https://review.opendev.org/c/openstack/nova/+/993063 | |
| 00:50:31 | opendevreview | Ghanshyam Maan proposed openstack/nova master: Add irrelevant-files for the graceful shutdown jobs https://review.opendev.org/c/openstack/nova/+/993063 | |
| 00:57:58 | gmaan | melwitt: ^^ | |
| 03:23:11 | opendevreview | Brian Haley proposed openstack/nova master: Change neutron API calls to use project_id https://review.opendev.org/c/openstack/nova/+/991948 | |
| 04:24:28 | gmaan | not sure why tox cover job takes more than hr to finish | |
| 05:17:38 | opendevreview | Shalini Srivastava proposed openstack/nova stable/2025.2: Filter invalid kwargs in volume_api.create to prevent TypeError https://review.opendev.org/c/openstack/nova/+/992849 | |
| 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 | |