| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2025-06-27 | |||
| 07:11:13 | gibi | sean-k-mooney: when you are up, dansmith is +2 on https://review.opendev.org/c/openstack/nova/+/952966 can I get a +A? | |
| 07:11:21 | gibi | this is the py3.13 fix | |
| 07:19:41 | opendevreview | Kamil Sambor proposed openstack/nova master: Replace eventlet.event.Event with threading.Event https://review.opendev.org/c/openstack/nova/+/949754 | |
| 07:25:42 | gibi | is it just me or CI is mosty shit due to etcd fails to start with 'listen tcp 65.17.193.12:2379: bind: cannot assign requested address' | |
| 07:26:41 | gibi | https://bugs.launchpad.net/devstack/+bug/2115338 | |
| 07:29:09 | gibi | maybe it is already fixed ^^: | |
| 07:32:35 | gibi | sean-k-mooney: also if you can look back to https://review.opendev.org/c/openstack/nova/+/953121 Dan is +2 as well. This is the build_and_run_instance error handling patch | |
| 09:08:20 | priteau | Hello. One of our customer experienced this bug: https://bugs.launchpad.net/nova/+bug/2074219. They are on Caracal. It made me realise the fix had not been backported from Epoxy. I submitted backports: https://review.opendev.org/c/openstack/nova/+/953468 and https://review.opendev.org/c/openstack/nova/+/953474 | |
| 09:08:37 | priteau | Backports for 2024.2 first, I will submit 2024.1 once approved. | |
| 09:17:47 | gibi | priteau: thanks for the backports I'm +2. Could you change the gerrit topic to bug/2074219 for better tracking? | |
| 09:18:06 | gibi | elodilles: ^^ some easy backport reviews :) | |
| 09:21:54 | priteau | Thanks, done | |
| 09:42:03 | elodilles | gibi: ACK, will review it today when I get there o/ | |
| 09:54:20 | opendevreview | Balazs Gibizer proposed openstack/nova master: Run unit test with threading mode https://review.opendev.org/c/openstack/nova/+/953475 | |
| 11:42:41 | sean-k-mooney | gibi: done for the 3.13 patch ill look at the other one next | |
| 11:43:22 | gibi | sean-k-mooney: thanks! | |
| 11:43:46 | sean-k-mooney | gibi: i should know this but its been a while. in my devstack env what is the simplest way to run tempest? | |
| 11:44:13 | sean-k-mooney | do you happen to knwo of the top of your head. i knwo that it preconfigures it for you and you can use tox | |
| 11:44:27 | sean-k-mooney | is that the best way or just using the tempest command in one of the venvs? | |
| 11:44:43 | sean-k-mooney | i know i have done both before | |
| 11:45:11 | gibi | cd /opt/stack/tempest/ | |
| 11:45:23 | gibi | source .tox/tempest/bin/activate | |
| 11:45:24 | gibi | tempest run | |
| 11:45:45 | sean-k-mooney | ack thanks i was not sure if that was still thge correct thing or not | |
| 11:45:52 | gibi | that works for me | |
| 11:46:24 | sean-k-mooney | ya i know that there used to be some script for it before they moved things to tox envs for the jobs and i knwo it has the idea of "workspaces" or whatever | |
| 11:46:39 | gibi | I never dig into workspaces | |
| 11:46:39 | sean-k-mooney | so i was not sure if i need to cd to a specific dir or something beyond the root of the repo | |
| 11:47:02 | sean-k-mooney | cool ill give that a go and if i have issue ill just ask gmann later | |
| 11:47:13 | gibi | btw the majority of the unit tests now run with threading in zuul https://review.opendev.org/c/openstack/nova/+/953475 I will work on the exclude list now | |
| 11:47:22 | sean-k-mooney | +2w on the rpc decorator change too | |
| 11:47:22 | gibi | dansmith: ^^ fyu | |
| 11:47:27 | gibi | thanks | |
| 11:48:17 | sean-k-mooney | oh cool sodo you need to set NOVA_RAISE_ON_GREENLET_LEAK=True again explcitly | |
| 11:48:32 | sean-k-mooney | im fine with doing that but shoul that not be inherited form {[testenv]setenv} | |
| 11:48:36 | gibi | I'm not sure about that any more I will check it | |
| 11:48:46 | sean-k-mooney | oh | |
| 11:48:49 | gibi | I'm not sure the env list is merged | |
| 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 | |