| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2025-06-27 | |||
| 11:48:51 | sean-k-mooney | sorry its false by default | |
| 11:48:59 | sean-k-mooney | so we need to turn it on in teh unit tests | |
| 11:49:17 | gibi | ahh I see | |
| 11:49:18 | sean-k-mooney | that why we also do it her https://review.opendev.org/c/openstack/nova/+/953475/2/tox.ini#64 | |
| 11:49:29 | sean-k-mooney | well its not false exactly its undefiend | |
| 11:49:58 | stephenfin | sean-k-mooney: could you revisit https://review.opendev.org/c/openstack/nova/+/952213 when you have a moment. I respun it to address gmaan's comments | |
| 11:50:28 | sean-k-mooney | same effect. but cool so you hae a exuclded test list and i assume the plan is to burn that down over time | |
| 11:51:15 | sean-k-mooney | sure, i dont have a meeting for anout hour so if there are thing people want me to review between now and then ping away | |
| 11:52:36 | priteau | sean-k-mooney: What about https://review.opendev.org/c/openstack/nova/+/953468 and https://review.opendev.org/c/openstack/nova/+/953474 :) | |
| 11:52:57 | priteau | Although you are the author, unsure about the policy of approving your own patch as backports? | |
| 11:52:58 | gibi | yepp I'm working on those tests. Some will propably need to wait until we transform nova-compute | |
| 11:55:28 | sean-k-mooney | priteau: i opened the tabsl so sure if i have time (i should) ill do those next. although i shoudl kick off that tempest run in the background first | |
| 11:59:07 | sean-k-mooney | stephenfin: i think that is a good enhancement as well so +2w | |
| 11:59:22 | stephenfin | ta | |
| 12:00:49 | sean-k-mooney | i think there are 6 with +2w pending i belive https://review.opendev.org/c/openstack/nova/+/952284/3 is the fristwehre we need to re reivew | |
| 12:00:57 | sean-k-mooney | but i proably wont get to that today | |
| 12:01:24 | sean-k-mooney | ill try and loop back to it on tuesday. | |
| 12:02:28 | sean-k-mooney | priteau: ah that backport i saw the pin in my email but did not get to it yet. | |
| 12:05:10 | priteau | No worries if you don't, elodilles said he will check later | |
| 12:06:24 | sean-k-mooney | gibi: ya i think that is fine, it would be nice to get basic testing running in check and non voting | |
| 12:06:59 | sean-k-mooney | priteau: well im just done so perhaps it will have merged and they can look at the 2024.1 version | |
| 12:07:26 | sean-k-mooney | the orgianal bug was intorduced in 2024.1 so that shoudl be the last backprot requried | |
| 12:08:32 | priteau | yes, that's my plan to submit 2024.1 next. Thanks! | |
| 12:09:15 | sean-k-mooney | would you mind assigned the backprot ones to you https://bugs.launchpad.net/nova/+bug/2074219 | |
| 12:09:44 | sean-k-mooney | technially there is nothigh that needs that i just like to do it | |
| 12:10:23 | priteau | done | |
| 12:11:21 | sean-k-mooney | cool feel free to ping once the patch is up and passed ci and i can take anohter look for 2024.1 | |
| 13:00:07 | sean-k-mooney | is grenade-skip-level-always TIMED_OUT currently a thing that people are seeign or just a one off? | |
| 13:02:41 | gibi | grenade-skip-level-always timeout happens frequently, it times out while dumping the DBs but I found nothing speficic in any logs | |
| 13:39:50 | gibi | sean-k-mooney: dansmith: do you have any suggestions where to look why nova.tests.unit.test_baserpc.BaseAPITestCase.test_ping hangs at the rpc.call (the request never received on the server side) when run with tox, while it passing when run with stestr (same arg list) top of https://review.opendev.org/c/openstack/nova/+/953475 | |
| 13:40:48 | gibi | and when it hands it hangs forever not just for an arbitrary timeout | |
| 13:41:17 | gibi | (OK I never waited forever but it is not stop at the normal testcase timeout) | |
| 13:57:46 | Uggla | gmann, sean-k-mooney can you please have a look at https://review.opendev.org/c/openstack/nova/+/952894 and https://review.opendev.org/c/openstack/nova/+/952895 this is a small bug fix, I'd like to close. | |
| 14:06:20 | gibi | ahh never mind, it hangs with stestr as well. I just forgot to turn the threading mode on in that terminal | |
| 14:11:11 | opendevreview | Tobias Urdin proposed openstack/nova master: Remove gabbi from test-requirements.txt https://review.opendev.org/c/openstack/nova/+/953578 | |
| 14:26:16 | opendevreview | Takashi Kajinami proposed openstack/placement master: Use standard FPGA resource class https://review.opendev.org/c/openstack/placement/+/953580 | |
| 14:36:21 | opendevreview | Takashi Kajinami proposed openstack/placement master: Replace deprecated assertItemsEqual https://review.opendev.org/c/openstack/placement/+/953582 | |
| 14:36:42 | opendevreview | Takashi Kajinami proposed openstack/placement master: Replace deprecated assertItemsEqual https://review.opendev.org/c/openstack/placement/+/953582 | |
| 14:44:36 | opendevreview | Takashi Kajinami proposed openstack/placement master: Replace deprecated assertItemsEqual https://review.opendev.org/c/openstack/placement/+/953582 | |
| 15:15:55 | opendevreview | Merged openstack/nova master: api: Add response body schemas for server diagnostics API https://review.opendev.org/c/openstack/nova/+/940506 | |
| 16:42:00 | opendevreview | Merged openstack/nova master: Fix neutron client dict grabbing https://review.opendev.org/c/openstack/nova/+/952966 | |
| 16:42:16 | opendevreview | Merged openstack/nova master: Note on RPC error decorators around build_and_run_instance https://review.opendev.org/c/openstack/nova/+/953121 | |
| 17:32:19 | opendevreview | Merged openstack/nova master: api: Address issues with remote consoles APIs https://review.opendev.org/c/openstack/nova/+/952213 | |
| 17:32:33 | opendevreview | Merged openstack/nova stable/2024.2: Add repoducer test for bug 2074219 https://review.opendev.org/c/openstack/nova/+/953468 | |
| 17:32:43 | opendevreview | Merged openstack/nova stable/2024.2: Fix detaching devices by alias with mdevs https://review.opendev.org/c/openstack/nova/+/953474 | |
| 18:05:25 | opendevreview | Pierre Riteau proposed openstack/nova stable/2024.1: Add repoducer test for bug 2074219 https://review.opendev.org/c/openstack/nova/+/953656 | |
| 18:11:25 | opendevreview | Pierre Riteau proposed openstack/nova stable/2024.1: Fix detaching devices by alias with mdevs https://review.opendev.org/c/openstack/nova/+/953662 | |
| 20:29:18 | opendevreview | Merged openstack/nova master: api: Address issues with server diagnostics APIs https://review.opendev.org/c/openstack/nova/+/952341 | |
| #openstack-nova - 2025-06-28 | |||
| 07:56:04 | opendevreview | Ghanshyam proposed openstack/nova master: Add new policies for migration APIs https://review.opendev.org/c/openstack/nova/+/953063 | |
| 12:18:23 | opendevreview | Stephen Finucane proposed openstack/nova master: Migrate mypy configuration to pyproject.toml https://review.opendev.org/c/openstack/nova/+/953703 | |
| 12:18:23 | opendevreview | Stephen Finucane proposed openstack/nova master: setup: Remove pbr's wsgi_scripts https://review.opendev.org/c/openstack/nova/+/902688 | |
| 12:18:24 | opendevreview | Stephen Finucane proposed openstack/nova master: Migrate setup configuration to pyproject.toml https://review.opendev.org/c/openstack/nova/+/953705 | |
| 12:18:24 | opendevreview | Stephen Finucane proposed openstack/nova master: Migrate codespell configuration to pyproject.toml https://review.opendev.org/c/openstack/nova/+/953704 | |
| #openstack-nova - 2025-06-29 | |||
| 05:31:14 | opendevreview | Michael Still proposed openstack/nova master: libvirt: Add objects and notifications for sound model. https://review.opendev.org/c/openstack/nova/+/926126 | |
| 05:31:15 | opendevreview | Michael Still proposed openstack/nova master: libvirt: Add objects and notifications for USB controller model. https://review.opendev.org/c/openstack/nova/+/927354 | |
| 05:31:15 | opendevreview | Michael Still proposed openstack/nova master: Implement sound model extra spec for libvirt. https://review.opendev.org/c/openstack/nova/+/940770 | |
| 05:31:16 | opendevreview | Michael Still proposed openstack/nova master: Implement USB controller extra spec for libvirt. https://review.opendev.org/c/openstack/nova/+/950643 | |
| 16:24:40 | opendevreview | Ghanshyam proposed openstack/nova master: Add new policies for migration APIs https://review.opendev.org/c/openstack/nova/+/953063 | |
| 16:25:07 | opendevreview | Ghanshyam proposed openstack/nova master: Add project manager role in Nova API policy rule https://review.opendev.org/c/openstack/nova/+/941347 | |
| 18:31:47 | fungi | it's come up a few times in debian over recent years that other cloud platforms don't rely on ahci driver access for cdrom devices, and so their minimal "cloud" linux kernel config doesn't include support for it, which means users have to manually override to virtio to make configdrive work with debian's official "genericcloud" images. is there a reason not to switch the | |
| 18:31:49 | fungi | default in nova to virtio, as suggested in https://bugs.debian.org/1108403 ? | |
| 19:00:29 | opendevreview | Ghanshyam proposed openstack/nova master: Add project manager role in Nova API policy rule https://review.opendev.org/c/openstack/nova/+/941347 | |
| 19:38:18 | opendevreview | Michael Still proposed openstack/nova master: libvirt: Add objects and notifications for sound model. https://review.opendev.org/c/openstack/nova/+/926126 | |
| 19:38:19 | opendevreview | Michael Still proposed openstack/nova master: libvirt: Add objects and notifications for USB controller model. https://review.opendev.org/c/openstack/nova/+/927354 | |
| 19:38:19 | opendevreview | Michael Still proposed openstack/nova master: Implement sound model extra spec for libvirt. https://review.opendev.org/c/openstack/nova/+/940770 | |
| 19:38:20 | opendevreview | Michael Still proposed openstack/nova master: Implement USB controller extra spec for libvirt. https://review.opendev.org/c/openstack/nova/+/950643 | |
| 20:05:30 | opendevreview | Ghanshyam proposed openstack/nova-specs master: Allow list migrations policy to manager role https://review.opendev.org/c/openstack/nova-specs/+/953722 | |
| 21:24:51 | opendevreview | Merged openstack/os-vif master: Zuul: do not use USE_PYTHON3 https://review.opendev.org/c/openstack/os-vif/+/953510 | |
| #openstack-nova - 2025-06-30 | |||
| 03:24:00 | gmaan | sean-k-mooney: stephenfin: while adding tempest tests for migrations by project manager, I realized that we need one list migrations API (list all migrations not just in-progress live migrations) to allow for manager. I a m amending it in spec, ptal https://review.opendev.org/c/openstack/nova-specs/+/953722 | |
| 08:37:50 | opendevreview | Thomas Goirand proposed openstack/nova master: Add a [libvirt]/force_virtio_cd option https://review.opendev.org/c/openstack/nova/+/953732 | |
| 08:40:18 | opendevreview | Thomas Goirand proposed openstack/nova master: Add a [libvirt]/force_virtio_cd option https://review.opendev.org/c/openstack/nova/+/953732 | |
| 08:41:05 | zigo | Does the above patch makes sense, and does the team understand my motivation as an operator? | |
| 08:41:56 | opendevreview | Thomas Goirand proposed openstack/nova master: Add a [libvirt]/force_virtio_cd option https://review.opendev.org/c/openstack/nova/+/953732 | |
| 09:49:40 | opendevreview | Balazs Gibizer proposed openstack/nova stable/2025.1: Fix neutron client dict grabbing https://review.opendev.org/c/openstack/nova/+/953737 | |
| 10:07:45 | opendevreview | Merged openstack/nova stable/2024.1: Add repoducer test for bug 2074219 https://review.opendev.org/c/openstack/nova/+/953656 | |
| 10:59:09 | sahid | o/ | |
| 10:59:24 | sahid | I'm still stuck finding information regarding live migration that are failing | |
| 10:59:37 | sahid | it's weird that we don't log anything | |
| 11:00:14 | sahid | the ;igration list --server xx show that status is in error but don't explain what the error is | |
| 11:03:10 | sean-k-mooney | you shoudl check the source compute node for logs | |
| 11:03:21 | sean-k-mooney | or conductor in some cases | |
| 11:06:03 | sahid | hum I did not tried on conductor... on source the only error I have is Live Migration failure: 'fa:....' | |
| 11:06:22 | sahid | Migration operation has aborted | |
| 11:06:39 | sean-k-mooney | aborted is not the same as failing. that likely just hit the timeout | |
| 11:09:30 | sahid | sean-k-mooney: I think the Live Migration failure failure is comming frist from libvirt driver | |
| 11:09:39 | sahid | then the exception is propagaded | |
| 11:10:22 | sean-k-mooney | it can if it failed on the migrate step | |
| 11:10:39 | sahid | let me see if I find from where is comming the message Migration operation has aborted | |
| 11:14:19 | sahid | the event is mentionning, compute_rollback_live_migration_at_destination and there is no log at destination | |
| 11:14:33 | sahid | we are missing to log some important details | |
| 11:23:51 | opendevreview | Merged openstack/nova stable/2024.1: Fix detaching devices by alias with mdevs https://review.opendev.org/c/openstack/nova/+/953662 | |
| 11:24:29 | sean-k-mooney | just an fyi i repliekd to mikal email just now, neutron broken debian 12 support with https://github.com/openstack/neutron/commit/8990dd598f84b9da976d72672c6fd6037fc0d125 it seams so the nova hybrid plug job is broken until they fix the coalaiton type | |
| 11:55:09 | opendevreview | Kamil Sambor proposed openstack/nova master: Replace utils.spawn_n with spawn https://review.opendev.org/c/openstack/nova/+/948076 | |
| 11:55:09 | opendevreview | Kamil Sambor proposed openstack/nova master: Use futurist for _get_default_green_pool() https://review.opendev.org/c/openstack/nova/+/948072 | |
| 11:55:10 | opendevreview | Kamil Sambor proposed openstack/nova master: Move ComputeManager to use spawn_on https://review.opendev.org/c/openstack/nova/+/948186 | |
| 11:55:10 | opendevreview | Kamil Sambor proposed openstack/nova master: Add spawn_on https://review.opendev.org/c/openstack/nova/+/948079 | |
| 11:55:11 | opendevreview | Kamil Sambor proposed openstack/nova master: Replace eventlet.event.Event with threading.Event https://review.opendev.org/c/openstack/nova/+/949754 | |
| 12:47:23 | fungi | [repeating myself from yesterday, sorry] it's come up a few times in debian over recent years that other cloud platforms don't rely on ahci driver access for cdrom devices, and so their minimal "cloud" linux kernel config doesn't include support for it, which means users have to manually override to virtio to make configdrive work with debian's official "genericcloud" images. | |
| 12:47:25 | fungi | is there a reason not to switch the default in nova to virtio, as suggested in https://bugs.debian.org/1108403 ? (i see zigo just pushed change 953732 for this too) | |
| 12:48:33 | zigo | I didn't push for this, because I was fearing it would be too much of an aggressive change, but I'd basically agree with fungi that it'd be nice to have virtio by default. | |