| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2025-08-29 | |||
| 04:55:44 | tkajinam | bauzas, sean-k-mooney ^^^ | |
| 04:57:34 | opendevreview | Merged openstack/nova master: Allow to start unit test without eventlet https://review.opendev.org/c/openstack/nova/+/953436 | |
| 05:13:43 | opendevreview | Merged openstack/placement master: Remove Python 3.9 support https://review.opendev.org/c/openstack/placement/+/953367 | |
| 07:20:54 | bauzas | tkajinam: sure, will look | |
| 07:40:26 | opendevreview | Rajesh Tailor proposed openstack/nova stable/2025.1: Fix 'nova-manage image_property set' command https://review.opendev.org/c/openstack/nova/+/958834 | |
| 08:01:37 | opendevreview | Tobias Urdin proposed openstack/nova master: api-ref: Add new lines not allowed for server description https://review.opendev.org/c/openstack/nova/+/928758 | |
| 08:02:19 | opendevreview | Tobias Urdin proposed openstack/nova master: libvirt: update description for live_migration_completion_timeout https://review.opendev.org/c/openstack/nova/+/874083 | |
| 08:03:45 | opendevreview | Tobias Urdin proposed openstack/nova master: libvirt: set remaining to 0 when no disk to migrate https://review.opendev.org/c/openstack/nova/+/873846 | |
| 08:05:19 | opendevreview | Tobias Urdin proposed openstack/nova master: Remove libvirt tunnelled migration https://review.opendev.org/c/openstack/nova/+/879021 | |
| 08:06:11 | opendevreview | Tobias Urdin proposed openstack/nova master: Remove libvirt tunnelled migration https://review.opendev.org/c/openstack/nova/+/879021 | |
| 08:08:28 | opendevreview | Tobias Urdin proposed openstack/nova master: Retry more volume API functions on server error https://review.opendev.org/c/openstack/nova/+/942981 | |
| 08:10:58 | opendevreview | Tobias Urdin proposed openstack/nova master: Remove gabbi from test-requirements.txt https://review.opendev.org/c/openstack/nova/+/953578 | |
| 09:17:16 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for volumes APIs https://review.opendev.org/c/openstack/nova/+/952348 | |
| 09:17:16 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for snapshots APIs https://review.opendev.org/c/openstack/nova/+/952349 | |
| 09:17:17 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for volume attachments APIs https://review.opendev.org/c/openstack/nova/+/952350 | |
| 09:17:17 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for floating IP APIs https://review.opendev.org/c/openstack/nova/+/952972 | |
| 09:17:18 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for security group APIs https://review.opendev.org/c/openstack/nova/+/952973 | |
| 09:17:18 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for services APIs https://review.opendev.org/c/openstack/nova/+/953196 | |
| 09:17:19 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for server usage audit log APIs https://review.opendev.org/c/openstack/nova/+/953209 | |
| 09:17:20 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for tenant network APIs https://review.opendev.org/c/openstack/nova/+/956088 | |
| 09:17:20 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for simple tenant usage APIs https://review.opendev.org/c/openstack/nova/+/956096 | |
| 09:17:22 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for limits API https://review.opendev.org/c/openstack/nova/+/956139 | |
| 09:17:22 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for quota class sets API https://review.opendev.org/c/openstack/nova/+/956140 | |
| 09:17:24 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for quota sets API https://review.opendev.org/c/openstack/nova/+/956141 | |
| 09:17:24 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for migrations API https://review.opendev.org/c/openstack/nova/+/956142 | |
| 09:17:26 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for server migrations API https://review.opendev.org/c/openstack/nova/+/956143 | |
| 09:17:26 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for server tags API https://review.opendev.org/c/openstack/nova/+/956144 | |
| 09:17:28 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for server metadata APIs https://review.opendev.org/c/openstack/nova/+/956145 | |
| 09:17:28 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Simplify servers views (1/3) https://review.opendev.org/c/openstack/nova/+/956231 | |
| 09:17:30 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Simplify servers views (2/3) https://review.opendev.org/c/openstack/nova/+/956232 | |
| 09:17:30 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Simplify servers views (3/3) https://review.opendev.org/c/openstack/nova/+/956233 | |
| 09:17:32 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for servers APIs (1/6) https://review.opendev.org/c/openstack/nova/+/956234 | |
| 09:17:32 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for servers APIs (2/6) https://review.opendev.org/c/openstack/nova/+/956236 | |
| 09:17:34 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for servers APIs (3/6) https://review.opendev.org/c/openstack/nova/+/956237 | |
| 09:17:34 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for servers APIs (4/6) https://review.opendev.org/c/openstack/nova/+/956238 | |
| 09:17:36 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for servers APIs (5/6) https://review.opendev.org/c/openstack/nova/+/956239 | |
| 09:17:36 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for servers APIs (6/6) https://review.opendev.org/c/openstack/nova/+/956240 | |
| 09:17:38 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for server shares APIs https://review.opendev.org/c/openstack/nova/+/956266 | |
| 09:17:38 | opendevreview | Stephen Finucane proposed openstack/nova master: tests: Invert validation check https://review.opendev.org/c/openstack/nova/+/956241 | |
| 10:04:47 | opendevreview | Merged openstack/placement master: Reproduce bug 2104040: allocate in over-capacity https://review.opendev.org/c/openstack/placement/+/945464 | |
| 10:06:30 | tkajinam | o/ anyone has a min to review https://review.opendev.org/c/openstack/placement/+/958820 to remove remaining Jammy job ? | |
| 10:08:11 | gibi | tkajinam: +2 bauzas ^^ easy win | |
| 10:08:49 | bauzas | yup | |
| 10:09:00 | bauzas | tkajinam: you deserve some rest, you know | |
| 10:09:15 | tkajinam | bauzas, gibi thanks ! | |
| 10:10:27 | tkajinam | I eventually took PTO today so had a good rest during the day :-) | |
| 10:11:11 | bauzas | well, you actually worked two days in a day yesterday so indeed, take PTO :) | |
| 10:45:29 | opendevreview | Merged openstack/placement master: Fix placement allocate while over-capacity https://review.opendev.org/c/openstack/placement/+/945465 | |
| 10:51:49 | gibi | dansmith: ^^ \o/ | |
| 10:59:23 | stephenfin | gmaan: gibi: bauzas: As sean-k-mooney said I've been out sick with a bug the last two days. Sorry for pushing that series so late: (a) I didn't realise FF was yesterday (I thought I'd another week) and (b) I couldn't push earlier as there would have been horrific conflicts with the (much larger) OpenAPI series. I'm happy to wait until early in G to get that in and appreciate the reviews | |
| 10:59:58 | stephenfin | I'm getting both it and the OpenAPI series tidied up so that we can hopefully get them in asap once we branch | |
| 11:01:41 | gibi | stephenfin: hope you are feeling better now. Thanks for working on these improvements | |
| 11:03:35 | stephenfin | nw | |
| 11:34:45 | gibi | sean-k-mooney: how do you feel about https://review.opendev.org/c/openstack/nova/+/936093 ? I think we saw the issue now downstream as well (see slack). I reviewed the patch and I think this is a good small fix. | |
| 11:35:33 | sean-k-mooney | gibi: ill need to load context as it looks liek i have not reviewd it befreo | |
| 11:36:14 | sean-k-mooney | we do kwno that there is a bug in mtu chagne on live migration but we need to ensure that it never changes across the migrtion | |
| 11:36:16 | gibi | tldr: mtu is not in our network cache and that can cause live migration issue later as libvirt don't allow MTU change during live migration but we wrongly used the nil from the cache | |
| 11:36:30 | sean-k-mooney | right | |
| 11:36:51 | sean-k-mooney | so ill readd the patch but how we palnned to fix that was to make sure we dont modify the xml by saving the orginal mtu if its present | |
| 11:37:35 | sean-k-mooney | we also do not incldue the mtu intetionally in teh netork metadlata if the neutron network has dhcp enabled | |
| 11:37:51 | gibi | OK. So this is more complicated than I anticipated | |
| 11:37:52 | sean-k-mooney | so we need to be careful not to change that | |
| 11:38:28 | sean-k-mooney | gibi: so the issue sometime come up with people going form ml2/ovs to ml2/ovn | |
| 11:38:38 | sean-k-mooney | nova does nto actully supprot changing the mtu on a network | |
| 11:38:56 | sean-k-mooney | there was an optional extention to allow that in neturon | |
| 11:39:09 | sean-k-mooney | btu it was never supproted on the nova side | |
| 11:39:20 | sean-k-mooney | so the mtu should never change on a live migration | |
| 11:39:49 | sean-k-mooney | howver when you do the db hacks to migrate form ml2/ovs to ml2/ovn you cahnge from vxlan to geneve and that reduces the mtu | |
| 11:40:21 | gibi | in this case the source mtu is undefined, that converted to 0 and then of course that does not match the dest mtu that is set | |
| 11:41:01 | sean-k-mooney | well the is an other rinkel | |
| 11:41:20 | sean-k-mooney | there are 2 formats for virtio feature flags | |
| 11:41:26 | sean-k-mooney | 32bit and 64bit | |
| 11:41:43 | sean-k-mooney | if you spcify the mtu in the xml it changes form 32bit to 64bit | |
| 11:41:59 | sean-k-mooney | which is also nto supproted by qemu during a live migration | |
| 11:42:19 | sean-k-mooney | so if the xml does nto have an mtu we cant add it durign a live migration | |
| 11:42:32 | gibi | I see, so this is the case ^^ | |
| 11:42:42 | gibi | nova tries to add it | |
| 11:43:14 | sean-k-mooney | that why we orginally started down the line of grab the mtu form the port and store it then generate the new config and restore the mtu | |
| 11:43:33 | sean-k-mooney | gibi: yep so we fixed this once before after i acindeltely broke vexhost | |
| 11:45:32 | sean-k-mooney | https://review.opendev.org/c/openstack/nova/+/553072 | |
| 11:45:54 | sean-k-mooney | that was the change to set the mtu in the first palace (its also neede for jumbo frames) | |
| 11:46:12 | sean-k-mooney | https://bugs.launchpad.net/nova/+bug/1747496 | |
| 11:47:05 | sean-k-mooney | that shoudl result in ports being attached with an mtu as well | |
| 11:47:39 | sean-k-mooney | gibi: the issue they are reproting in https://bugs.launchpad.net/nova/+bug/2080531 | |
| 11:47:42 | opendevreview | Merged openstack/placement master: Remove the job with Ubuntu 22.04 https://review.opendev.org/c/openstack/placement/+/958820 | |
| 11:47:50 | sean-k-mooney | i think artom had satred to fxi a while ago | |
| 11:47:54 | sean-k-mooney | let me see if i can find that | |
| 11:48:30 | opendevreview | Stephen Finucane proposed openstack/nova master: db: Move regex helpers to utils https://review.opendev.org/c/openstack/nova/+/958745 | |
| 11:48:31 | opendevreview | Stephen Finucane proposed openstack/nova master: tests: Clean up flavors tests https://review.opendev.org/c/openstack/nova/+/958746 | |
| 11:48:31 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Simplify API version check for flavor description https://review.opendev.org/c/openstack/nova/+/958747 | |
| 11:48:32 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add ability to filter flavors by name https://review.opendev.org/c/openstack/nova/+/958748 | |
| 11:48:32 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Remove dead fields from flavors response https://review.opendev.org/c/openstack/nova/+/958749 | |
| 11:48:33 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Restrict additional query string arguments https://review.opendev.org/c/openstack/nova/+/958750 | |
| 11:48:33 | opendevreview | Stephen Finucane proposed openstack/nova master: tests: Fix typo https://review.opendev.org/c/openstack/nova/+/958855 | |
| 11:48:34 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add runtime check for query additionalProperties https://review.opendev.org/c/openstack/nova/+/958856 | |
| 11:48:34 | opendevreview | Stephen Finucane proposed openstack/nova master: tests: Add missing test coverage https://review.opendev.org/c/openstack/nova/+/958857 | |
| 11:48:36 | opendevreview | Stephen Finucane proposed openstack/nova master: WIP: api: Add runtime check for general additionalProperties https://review.opendev.org/c/openstack/nova/+/958858 | |
| 11:48:46 | sean-k-mooney | gibi: https://review.opendev.org/c/openstack/nova/+/791553/1 | |
| 11:49:05 | sean-k-mooney | so we have a regression test https://review.opendev.org/c/openstack/nova/+/791235/3 | |
| 11:49:11 | sean-k-mooney | then then the poc to fix it | |