| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2025-08-28 | |||
| 11:59:26 | bauzas | I looked at all the BPs on Monday and I didn't found any implementation for it | |
| 11:59:35 | sean-k-mooney | yes, an i know its very late to propose it it just short and easy to reveiw | |
| 11:59:55 | sean-k-mooney | if it goes to next cycle that also ok | |
| 11:59:57 | bauzas | I can try then but this is also providing an API microversion, right ? | |
| 12:00:04 | sean-k-mooney | i just wanted to mention it before i go back to the sev one | |
| 12:00:17 | sean-k-mooney | bauzas: yes it is 101 | |
| 12:00:25 | bauzas | if so, I need to make sure we won't have any issue as we couldn't revert a change if we find an issue | |
| 12:00:32 | sean-k-mooney | thre are no other api changes im aware of competing for a micoverison at this point | |
| 12:00:40 | bauzas | sean-k-mooney: thanks for the SEV-ES series review btw. | |
| 12:00:50 | bauzas | sean-k-mooney: yup, I haven't found any | |
| 12:00:52 | sean-k-mooney | the microversion is they 4th patch the first 3 are just clean up | |
| 12:01:11 | bauzas | okay, then I'll look at it today | |
| 12:01:26 | sean-k-mooney | by cleanup i mean code cleanup no externally visable changes | |
| 12:40:47 | gibi | sean-k-mooney: I can take a quick look | |
| 13:08:13 | Uggla | gibi, sorry was distracted and I have just noticed your msg about the patches run. Thanks for letting me know. | |
| 13:08:22 | gibi | sure no worries | |
| 14:22:07 | dansmith | anyone else seeing python3.9 dep resolving failures? | |
| 14:22:24 | dansmith | I'm wondering if placement should drop that job | |
| 14:34:30 | gibi | dansmith: yapp we should drop it F has no 3.9 requirements https://governance.openstack.org/tc/reference/runtimes/2025.2.html | |
| 14:35:17 | gibi | https://review.opendev.org/c/openstack/placement/+/953367 we need this | |
| 14:35:28 | gibi | just approved | |
| 14:35:40 | dansmith | ack | |
| 15:58:31 | bauzas | sean-k-mooney: can you please review again the SEV-ES series ? https://review.opendev.org/c/openstack/nova/+/925685/ and upper | |
| 16:01:52 | sean-k-mooney | yep ill be around for the next 3 hours maybe a little longer so ill get to it once i push one watcher patch | |
| 16:02:00 | sean-k-mooney | its next on my todo list | |
| 16:14:05 | gmaan | sean-k-mooney: ack, i can check it today | |
| 16:21:20 | sean-k-mooney | no rush gibi notcied a reduction in test coverge becuase stephn delete a test instead of moving it in the first patch | |
| 16:21:33 | sean-k-mooney | gmaan: and im not expectign a respin | |
| 16:21:42 | gmaan | k | |
| 16:21:48 | sean-k-mooney | so we can pick that back up early next cycle | |
| 16:22:48 | gmaan | so we are not targeting that for this? | |
| 16:25:28 | gmaan | and this is still WIP, which is easy but big change https://review.opendev.org/c/openstack/nova/+/958750/1 | |
| 16:25:34 | sean-k-mooney | we were | |
| 16:25:57 | sean-k-mooney | but stephenfin is feeling unwell today and realsiticly its not going to land | |
| 16:26:12 | gmaan | ohk. got it. | |
| 16:26:38 | sean-k-mooney | so we can do it all early next cycle and still have 1 microversion for the feature and api cleanup | |
| 16:26:49 | gmaan | ++ | |
| 16:27:20 | gmaan | dansmith: can you +w this devstack change (quick one) for service role https://review.opendev.org/c/openstack/devstack/+/958612 | |
| 16:27:40 | dansmith | gmaan: got it | |
| 16:27:51 | gmaan | thanks | |
| 17:02:05 | gmaan | sean-k-mooney: whoami-rajat_ FYI, service API things on cinder/glance working now - https://zuul.opendev.org/t/openstack/build/03e62efae4054b7997f8fc6a4ca3db7c | |
| 17:02:25 | gmaan | consider configuring and devstack setting glance user in cinder properly https://zuul.opendev.org/t/openstack/build/03e62efae4054b7997f8fc6a4ca3db7c/log/controller/logs/etc/cinder/cinder_conf.txt#74 | |
| 17:02:46 | gmaan | there are unit test etc failure in fixes, I will fix those and ping in cinder. glance channel | |
| 17:03:10 | gmaan | this is testing change on top of fixes in glance, cinder, devstack https://review.opendev.org/c/openstack/cinder/+/958719 | |
| 17:14:31 | sean-k-mooney | ack i can join those if needed | |
| 17:14:38 | opendevreview | Takashi Kajinami proposed openstack/nova master: libvirt: Remove tpm supoort detection for libvirt < 8.0.0 https://review.opendev.org/c/openstack/nova/+/952308 | |
| 17:41:19 | whoami-rajat_ | gmaan, ack, looks good, thanks | |
| 17:42:29 | sean-k-mooney | tkajinam: this is my main comment on the second sev patch https://review.opendev.org/c/openstack/nova/+/925685/comment/6a462652_db266f95/ if you have time can you respond whil i continue to review the rest | |
| 17:44:09 | sean-k-mooney | tkajinam: if you agree that we shoudl accomidate SEV_ES hosts with no SEV slot then we can adress that in a patch at the end of the seriese | |
| 17:47:45 | tkajinam | sean-k-mooney, thanks and checking it now | |
| 17:47:56 | sean-k-mooney | i dont think there si any hardware today that exclsivly supprot SEV-ES and not vanila SEV so i doing think its an edgecase we will actully hit | |
| 17:48:23 | sean-k-mooney | but i dont know if that will change going forward so just calling it out | |
| 17:52:47 | opendevreview | Merged openstack/nova master: Add service role in Nova policy https://review.opendev.org/c/openstack/nova/+/957578 | |
| 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 | |