| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2025-08-28 | |||
| 17:58:50 | tkajinam | sean-k-mooney, I've replied to your comment about sev/sev-es support. at this moment kernel requires that sev is enabled when sev-es is enabled and I followed that logic. | |
| 17:59:07 | sean-k-mooney | ack that works for me | |
| 17:59:31 | sean-k-mooney | i just didnt want use to encode somethign that that might not hold true in general | |
| 17:59:33 | tkajinam | it can be changed in the future and we can decouple these checks when kernel actually changes its logic | |
| 17:59:45 | sean-k-mooney | +1 | |
| 18:05:16 | sean-k-mooney | tkajinam: ill loop back to the lower patches but i am +2 on the main ones | |
| 18:06:15 | tkajinam | sean-k-mooney, thanks ! | |
| 18:08:22 | sean-k-mooney | tkajinam: is it late or very early for you | |
| 18:08:56 | sean-k-mooney | tkajinam: i have never internalised what timezone you are in | |
| 18:09:57 | tkajinam | ;-) | |
| 18:11:52 | tkajinam | It's 3 am in UTC+9 so it's "still" late time here | |
| 18:11:58 | tkajinam | some may say it's early though | |
| 18:12:31 | dansmith | 2300 is late, 0300 is definitely early | |
| 18:14:14 | gmaan | I think sun rises there that time or may be an hr later :) | |
| 18:15:21 | sean-k-mooney | sunrise is my favorit time to wake up or fall asleep although i do the latter much more frequently | |
| 18:15:57 | gmaan | :) later for me during weekends | |
| 18:18:18 | tkajinam | ok I'm leaving before sunrise. I'll check the status of these patches and work on follow-ups once I wake up | |
| 18:21:18 | sean-k-mooney | tkajinam: i have +2w'd the series now | |
| 18:22:12 | tkajinam | sean-k-mooney, thank you ! \o/ | |
| 18:22:14 | sean-k-mooney | tkajinam: none of the comments i left rise to the point of needing a respin but if you submit a followup im happy to review | |
| 18:46:57 | sean-k-mooney | Uggla: https://bugs.launchpad.net/nova/+bug/2121617 fun little bug. apprently you cant set the region_name in our config for manilla and for legacy reason barbican and cidner dont use a option called region_name for that like eveythign else | |
| 18:47:39 | sean-k-mooney | Uggla: we shoudl fix the endpoint discovery logic for manilla and backport that but we should also deprecate the non standar names and supprot the standard ones for cidner/barbican | |
| 18:48:05 | sean-k-mooney | i didnt check the rest of the config options for consitency but we proably shoudl when fixign this. | |
| 18:50:23 | sean-k-mooney | you coudl make manilla work in a multi region setup today if you explcitly set the url however we also do not supprot that so your stuck. you cant use the manilla shares feature in a multi region openstack unless the same instance of manilla is used by all regions | |
| 18:51:58 | sean-k-mooney | setting https://docs.openstack.org/nova/latest/configuration/config.html#manila.auth_url might work if you configre it to talk to the local keyston region api but that would assume each region has its own keystoen with federation which is not really how people deploy multi region openstack | |
| 18:54:00 | sean-k-mooney | so ya we shoudl plan to fix and backport some subset of this but its not a regression intoduced this cycle so we can fix it as a normal bug | |
| 20:36:22 | opendevreview | Merged openstack/nova master: Migrate MEM_ENCRYPTION_CONTEXT from root provider https://review.opendev.org/c/openstack/nova/+/921814 | |
| 20:36:39 | opendevreview | Merged openstack/nova master: Detect AMD SEV-ES support https://review.opendev.org/c/openstack/nova/+/925685 | |
| 21:03:29 | opendevreview | Merged openstack/nova master: Add hw_mem_encryption_model image property https://review.opendev.org/c/openstack/nova/+/927706 | |
| 23:24:33 | opendevreview | Merged openstack/nova master: libvirt: Launch instances with SEV-ES memory encryption https://review.opendev.org/c/openstack/nova/+/926106 | |
| 23:25:15 | opendevreview | Merged openstack/nova master: Add functional test scenario for mixed SEV RPs https://review.opendev.org/c/openstack/nova/+/958562 | |
| 23:27:06 | opendevreview | Merged openstack/nova master: Purge nested SEV RPs when SEV is disabled https://review.opendev.org/c/openstack/nova/+/958626 | |
| #openstack-nova - 2025-08-29 | |||
| 01:40:12 | opendevreview | Takashi Kajinami proposed openstack/placement master: Remove the job with Ubuntu 22.04 https://review.opendev.org/c/openstack/placement/+/958820 | |
| 03:02:53 | opendevreview | Takashi Kajinami proposed openstack/placement master: Remove the job with Ubuntu 22.04 https://review.opendev.org/c/openstack/placement/+/958820 | |
| 03:54:54 | opendevreview | Takashi Kajinami proposed openstack/nova master: Follow-up of AMD SEV-ES support https://review.opendev.org/c/openstack/nova/+/958822 | |
| 03:59:23 | opendevreview | Takashi Kajinami proposed openstack/nova master: Follow-up of AMD SEV-ES support https://review.opendev.org/c/openstack/nova/+/958822 | |
| 04:35:26 | opendevreview | Merged openstack/nova master: Ask for pre-prod testing for native threading https://review.opendev.org/c/openstack/nova/+/957424 | |
| 04:54:40 | opendevreview | Takashi Kajinami proposed openstack/nova master: Follow-up of AMD SEV-ES support https://review.opendev.org/c/openstack/nova/+/958822 | |
| 04:55:29 | tkajinam | I've proposed a follow-up of SEV-ES support, which addresses most of the comments https://review.opendev.org/c/openstack/nova/+/958822 . Could you please have a look when you have time ? | |
| 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 snapshots APIs https://review.opendev.org/c/openstack/nova/+/952349 | |
| 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: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: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: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: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: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 simple tenant usage APIs https://review.opendev.org/c/openstack/nova/+/956096 | |
| 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: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: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: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: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: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: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:28 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Simplify servers views (1/3) https://review.opendev.org/c/openstack/nova/+/956231 | |
| 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:30 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Simplify servers views (3/3) https://review.opendev.org/c/openstack/nova/+/956233 | |
| 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: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: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: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: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: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: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:38 | opendevreview | Stephen Finucane proposed openstack/nova master: tests: Invert validation check https://review.opendev.org/c/openstack/nova/+/956241 | |
| 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 | |
| 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 | |