| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2026-06-23 | |||
| 10:20:32 | sean-k-mooney | gibi: o/ | |
| 10:20:41 | sean-k-mooney | amy proiably i need to load context | |
| 10:22:26 | sean-k-mooney | gibi: so the issue that is not adressed is you cant change the mtu on live migration and we need to handel that | |
| 10:22:32 | sean-k-mooney | but ya we can likely proceed | |
| 10:23:24 | sean-k-mooney | im trying to see if i had a specific concern in the past beyond that i.e. somethign that woudl be hard to fix later beyond it being a partial fix | |
| 10:25:49 | sean-k-mooney | gibi: so looking at my old comment the main ask i had was to add a functional regression test | |
| 10:26:37 | sean-k-mooney | gibi: so that would still be nice to have but i dont have an objection to the approch to fixign it | |
| 10:28:01 | gibi | OK. Lets see If I get some time to add a functional reproducer. If so I will ping you that patch | |
| 10:28:10 | gibi | thanks for confirming that the direction is OK | |
| 10:28:41 | sean-k-mooney | if you want i can just have ai create one now and i can adress my own request | |
| 10:30:00 | gibi | if you have the capacit I'm OK with that | |
| 10:37:14 | opendevreview | Joan Gilabert proposed openstack/nova master: Report OWNER_NOVA trait on resource providers https://review.opendev.org/c/openstack/nova/+/994299 | |
| 10:55:35 | opendevreview | Anton Iacobaeus proposed openstack/nova master: libvirt: Add Intel TDX host capability detection https://review.opendev.org/c/openstack/nova/+/994421 | |
| 11:14:08 | sean-k-mooney | bauzas: while i think of it im now +2 on https://review.opendev.org/c/openstack/nova/+/988777 if you want to readd yours | |
| 11:18:14 | opendevreview | Joan Gilabert proposed openstack/nova master: move compile earlier https://review.opendev.org/c/openstack/nova/+/950516 | |
| 11:18:15 | opendevreview | Joan Gilabert proposed openstack/nova master: Rename vtpm job and add mtty support for vgpu test https://review.opendev.org/c/openstack/nova/+/922140 | |
| 11:18:15 | opendevreview | Joan Gilabert proposed openstack/nova master: Add mtty support for testing fake mdevs https://review.opendev.org/c/openstack/nova/+/898100 | |
| 11:29:05 | opendevreview | Anton Iacobaeus proposed openstack/nova master: libvirt: Add Intel TDX host capability detection https://review.opendev.org/c/openstack/nova/+/994421 | |
| 13:13:49 | opendevreview | Masanori Ueno proposed openstack/nova master: Add regression test for bug #2153425 https://review.opendev.org/c/openstack/nova/+/988777 | |
| 13:16:33 | opendevreview | Masanori Ueno proposed openstack/nova master: live-migration: Ensure limits are set before _call_livem_checks_on_host https://review.opendev.org/c/openstack/nova/+/990212 | |
| 13:32:25 | opendevreview | sean mooney proposed openstack/nova master: Add regression tests for interface attach MTU bug https://review.opendev.org/c/openstack/nova/+/994437 | |
| 13:32:25 | opendevreview | sean mooney proposed openstack/nova master: Fix network mtu metadata disappears after attaching https://review.opendev.org/c/openstack/nova/+/936093 | |
| 13:33:04 | opendevreview | sean mooney proposed openstack/nova master: Fix network mtu metadata disappears after attaching https://review.opendev.org/c/openstack/nova/+/936093 | |
| 13:33:24 | sean-k-mooney | gibi: that also needed a rebase but ^ should cover all my nits | |
| 13:35:33 | gibi | sean-k-mooney: thanks | |
| 13:35:59 | sean-k-mooney | i didnt quite have tim eto push that before my meeting as i was waiting for tox runs | |
| 13:51:52 | opendevreview | Erlon R. Cruz proposed openstack/nova master: Adding regression test to reproduce bug 1741364 https://review.opendev.org/c/openstack/nova/+/993597 | |
| 13:51:53 | opendevreview | Erlon R. Cruz proposed openstack/nova master: Add RBD XML update functionality for migration process https://review.opendev.org/c/openstack/nova/+/974032 | |
| 14:32:54 | opendevreview | Joan Gilabert proposed openstack/nova master: Report OWNER_NOVA trait on resource providers https://review.opendev.org/c/openstack/nova/+/994299 | |
| 16:09:22 | frickler | nova-ceph-multistore is failing on unmaintained/2024.1, I don't really want to bug people about it much, but maybe if someone knows an easy fix we can apply that instead of just dropping the job https://zuul.opendev.org/t/openstack/build/570a4539d0f24452ab6c5f9d75e3450a | |
| 16:25:37 | melwitt | hm, "oslo_config.cfg.NoSuchOptError: no such option replication in group [volume-feature-enabled]" must be something like, cinder-tempest-plugin is not branched but maybe tempest on unmaintained/2024.1 is pinned cc gmaan | |
| 16:28:30 | gmaan | yes, i think it is pinned | |
| 16:28:56 | opendevreview | Takashi Kajinami proposed openstack/nova master: compute: Validate SEV/SEV-ES support at start up https://review.opendev.org/c/openstack/nova/+/994342 | |
| 16:29:39 | gmaan | yes tempest 45.0.0 is used and it does not have 'replication ' config option https://github.com/openstack/tempest/blob/45.0.0/tempest/config.py | |
| 16:29:55 | gmaan | so test should be skipped in that case | |
| 16:31:08 | gmaan | we need to pin cinder-tempest-plugin also so that it will not pick the latest tests from it | |
| 16:37:09 | melwitt | makes sense | |
| 16:39:11 | gmaan | will propose fix after meeting | |
| 16:39:51 | gmaan | melwitt: sean-k-mooney you might be interested in these. nova side work needed for oslo.policy enforce_scope flag removal https://review.opendev.org/q/hashtag:%22remove-enforce-scope-flag%22+AND+hashtag:%22nova%22+status:open | |
| 16:41:33 | melwitt | gmaan: ack will look | |
| 16:41:53 | gmaan | thanks | |
| 16:55:32 | opendevreview | Takashi Kajinami proposed openstack/nova master: libvirt: Ignore SEV-ES when SEV-SNP is detected https://review.opendev.org/c/openstack/nova/+/994343 | |
| 17:01:51 | sean-k-mooney | gmaan: oh cool | |
| 17:02:09 | sean-k-mooney | looks pretty mechnaical | |
| 17:02:45 | sean-k-mooney | while while somewhat tedious to review is at elast not overly compex | |
| 17:03:01 | opendevreview | Lajos Katona proposed openstack/nova master: Add regression test to repoduce bug 1854844 https://review.opendev.org/c/openstack/nova/+/991772 | |
| 17:04:12 | sean-k-mooney | im slightly suprised the nova change depens on the placment one but i guess that is down to the fixture? | |
| 17:08:27 | gmaan | sean-k-mooney: I think yes, somewhere it was failing and I need to do placement one first | |
| 17:14:39 | sean-k-mooney | i am sumwhat suprised that we need to update teh gabit tests | |
| 17:15:11 | sean-k-mooney | specific for system scoped rp creation | |
| 17:15:41 | sean-k-mooney | given i was not expecting system scoped roels to really be valid based on https://github.com/openstack/placement/blob/master/placement/policies/resource_provider.py | |
| 17:16:37 | sean-k-mooney | given admin or service is "role:admin or role:service" | |
| 17:17:02 | sean-k-mooney | but i guess when scope enforcment is disabeld we are merging system and doamin roels into the set of token roles? | |
| 17:17:44 | sean-k-mooney | gmaan: basiclly im asking did we change the behvior or was gabbi just not emululatign this properly | |
| 17:18:21 | sean-k-mooney | for this change https://review.opendev.org/c/openstack/placement/+/987568/4/placement/tests/functional/gabbits/aggregate-legacy-rbac.yaml#55 | |
| 17:19:00 | gmaan | sean-k-mooney: yes, it is same as nova test change. the update is to the tests which were testing scope disable and system_admin can do the things | |
| 17:20:09 | gmaan | sean-k-mooney: those are needed to create the resource for other tests like update test, for example $ENVIRON[[RP_UUID] | |
| 17:20:25 | gmaan | previously system_admin was allowed in those tests and it created the resouce | |
| 17:20:51 | gmaan | i thought of remove the legacy tests but till we have the legacy policy, we should keep them tested | |
| 17:23:31 | gmaan | Basically, it remove the scope disable tests and update a few that were dependent on the test resource creation by scope disable tests | |
| 17:36:25 | opendevreview | Takashi Kajinami proposed openstack/nova master: libvirt: Ignore SEV-ES when SEV-SNP is detected https://review.opendev.org/c/openstack/nova/+/994343 | |
| 17:36:25 | opendevreview | Takashi Kajinami proposed openstack/nova master: compute: Validate SEV/SEV-ES support at start up https://review.opendev.org/c/openstack/nova/+/994342 | |
| 17:36:26 | opendevreview | Takashi Kajinami proposed openstack/nova master: Drop dependency on [libvirt] num_memory_encrypted_guests https://review.opendev.org/c/openstack/nova/+/994506 | |
| 17:36:26 | opendevreview | Takashi Kajinami proposed openstack/nova master: Fix wrong mock for properties https://review.opendev.org/c/openstack/nova/+/994505 | |
| 18:37:01 | opendevreview | Sahana proposed openstack/nova master: compute: remove redundant exception args from LOG.exception calls https://review.opendev.org/c/openstack/nova/+/994522 | |
| 18:57:19 | opendevreview | melanie witt proposed openstack/nova master: Add non-voting IPv6 ceph live migration job https://review.opendev.org/c/openstack/nova/+/984623 | |
| 19:12:25 | opendevreview | melanie witt proposed openstack/nova master: Add non-voting IPv6 ceph live migration job https://review.opendev.org/c/openstack/nova/+/984623 | |
| 19:31:08 | gmaan | frickler: melwitt this will fix the nova-ceph-multistore job on 2024.1, we cna see devstack-plugin-ceph-tempest-py3 passing there https://review.opendev.org/c/openstack/devstack-plugin-ceph/+/994503 | |
| 19:32:10 | gmaan | but it seems gate is broken for doc job there so I am leaving this until someonw fix the gate. cannot spend more time on those. That is one of the reason i try not to touch unmaintained branches as they need lot of gate fixes before one merge :) | |
| 19:32:56 | gmaan | CC: elodilles just to make you aware about ceph job fix | |
| 19:49:27 | frickler | gmaan: thanks, I can try to look at the docs issue. and yes I can understand your reluctance, thx for checking this anyhow | |
| 19:57:03 | melwitt | gmaan: ah makes sense | |
| 20:01:42 | opendevreview | Merged openstack/nova master: move compile earlier https://review.opendev.org/c/openstack/nova/+/950516 | |
| 20:08:38 | opendevreview | melanie witt proposed openstack/nova master: Add non-voting IPv6 ceph live migration job https://review.opendev.org/c/openstack/nova/+/984623 | |
| 20:12:19 | erlon | sean-k-mooney: hey Sean, can you take a look in the RBD fix we discuss the other day? See if thats how you wanted the regression tests, please let me know if I'm missing anything: https://review.opendev.org/c/openstack/nova/+/974032 | |
| 20:14:22 | sean-k-mooney | erlon: yes i started lookign at that elaire today | |
| 20:14:30 | sean-k-mooney | i have the regression test open i na tab somewhere | |
| 20:16:21 | sean-k-mooney | erlon: lookign at ti brifly yes it looks more or less correct ill try and review it proeprly but probaly tomorrow | |
| 20:16:34 | erlon | nice, thanks | |
| 20:17:48 | sean-k-mooney | i see melwitt updated https://review.opendev.org/c/openstack/nova/+/984623/21 as well so we shoudl get som new reuslt of the ipv6 test in a few hours | |
| 20:19:35 | melwitt | yeah fingers crossed it doesn't run into another issue. this is the first time I try it with depends-on patches proposed to the proper repos. last fail was for database host using ipv4 address. I'll keep iterating on it | |
| 20:25:06 | sean-k-mooney | is there a reason you made the ipv6 connectivity opt in instead of opt out? | |
| 20:25:27 | sean-k-mooney | we often have ipv6 in the jobs | |
| 20:25:40 | sean-k-mooney | even if its not the ipv6 job | |
| 20:25:55 | melwitt | no reason other than to keep existing behavior the same | |
| 20:25:55 | sean-k-mooney | we just dont use those adress for the services by default | |
| 20:26:01 | sean-k-mooney | ack | |
| 20:26:17 | sean-k-mooney | im not sure which is the better default i just said i would ask | |
| 20:26:28 | sean-k-mooney | incasue it was because you hit a bug or something i was not aware of | |
| 20:26:59 | melwitt | oh yeah. not yet anyway haha | |
| 20:27:54 | sean-k-mooney | so if you respin you might want to mention ipv6 in the readme for that roles https://opendev.org/zuul/zuul-jobs/src/commit/6c0c643b8cffa1f61cf7af34f29d8a7e4d1bb114/roles/multi-node-bridge/README.rst | |
| 20:29:26 | sean-k-mooney | melwitt: they do not use molecule in that repo | |
| 20:29:32 | sean-k-mooney | but you may also want to exted https://opendev.org/zuul/zuul-jobs/src/commit/6c0c643b8cffa1f61cf7af34f29d8a7e4d1bb114/test-playbooks/multinode/multi-node-bridge.yaml | |
| 20:29:42 | sean-k-mooney | to test the ipv6 functionlatiy | |
| 20:30:02 | sean-k-mooney | but i dont really review code in that repo oftent so im not sure what the normal expecation are | |
| 20:30:06 | melwitt | good point, I didn't know about those | |
| 20:31:10 | sean-k-mooney | fungi and clarkb are inovled so i knwo there woudl be tests but i am not very familar with how they are setup but i guess this is just usign zuul to run the test playbooks directly | |
| 20:31:57 | sean-k-mooney | looks like tey are using tox and unittest to run them at a quick glance | |
| 20:32:26 | sean-k-mooney | although that may just be for the inline moduels | |
| 20:32:59 | fungi | yeah, clarkb can probably explain it better since he wrote a lot of it, but the idea initially was to have an abstract foundation for multi-node devstack jobs to build on | |