| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2025-06-13 | |||
| 10:53:56 | opendevreview | Balazs Gibizer proposed openstack/nova master: Translate scatter-gather to futurist https://review.opendev.org/c/openstack/nova/+/947966 | |
| 10:53:57 | opendevreview | Balazs Gibizer proposed openstack/nova master: Add spawn_on https://review.opendev.org/c/openstack/nova/+/948079 | |
| 10:53:57 | opendevreview | Balazs Gibizer proposed openstack/nova master: Replace utils.spawn_n with spawn https://review.opendev.org/c/openstack/nova/+/948076 | |
| 10:53:58 | opendevreview | Balazs Gibizer proposed openstack/nova master: Move ConductorManager to use spawn_on https://review.opendev.org/c/openstack/nova/+/948187 | |
| 10:53:58 | opendevreview | Balazs Gibizer proposed openstack/nova master: Move ComputeManager to use spawn_on https://review.opendev.org/c/openstack/nova/+/948186 | |
| 10:53:59 | opendevreview | Balazs Gibizer proposed openstack/nova master: Rename DEFAULT_GREEN_POOL to DEFAULT_EXECUTOR https://review.opendev.org/c/openstack/nova/+/948086 | |
| 10:53:59 | opendevreview | Balazs Gibizer proposed openstack/nova master: Make nova.utils.pass_context private https://review.opendev.org/c/openstack/nova/+/948188 | |
| 10:54:00 | opendevreview | Balazs Gibizer proposed openstack/nova master: Make the default executor configurable https://review.opendev.org/c/openstack/nova/+/948087 | |
| 10:54:01 | opendevreview | Balazs Gibizer proposed openstack/nova master: Document threading mode and tuneables https://review.opendev.org/c/openstack/nova/+/949364 | |
| 10:54:01 | opendevreview | Balazs Gibizer proposed openstack/nova master: Print ThreadPool statistics https://review.opendev.org/c/openstack/nova/+/948340 | |
| 10:54:03 | opendevreview | Balazs Gibizer proposed openstack/nova master: Run nova-next with n-sch in threading mode https://review.opendev.org/c/openstack/nova/+/948450 | |
| 10:54:03 | opendevreview | Balazs Gibizer proposed openstack/nova master: Allow services to start with threading https://review.opendev.org/c/openstack/nova/+/948311 | |
| 10:54:05 | opendevreview | Balazs Gibizer proposed openstack/nova master: Run nova-api and -metadata in threaded mode https://review.opendev.org/c/openstack/nova/+/951957 | |
| 10:54:05 | opendevreview | Balazs Gibizer proposed openstack/nova master: Do not yield in threading mode https://review.opendev.org/c/openstack/nova/+/950994 | |
| 13:06:18 | opendevreview | Balazs Gibizer proposed openstack/nova master: Document threading mode and tuneables https://review.opendev.org/c/openstack/nova/+/949364 | |
| 13:06:18 | opendevreview | Balazs Gibizer proposed openstack/nova master: Print ThreadPool statistics https://review.opendev.org/c/openstack/nova/+/948340 | |
| 13:06:19 | opendevreview | Balazs Gibizer proposed openstack/nova master: Run nova-next with n-sch in threading mode https://review.opendev.org/c/openstack/nova/+/948450 | |
| 13:06:19 | opendevreview | Balazs Gibizer proposed openstack/nova master: Allow services to start with threading https://review.opendev.org/c/openstack/nova/+/948311 | |
| 13:06:20 | opendevreview | Balazs Gibizer proposed openstack/nova master: Run nova-api and -metadata in threaded mode https://review.opendev.org/c/openstack/nova/+/951957 | |
| 13:06:20 | opendevreview | Balazs Gibizer proposed openstack/nova master: Do not yield in threading mode https://review.opendev.org/c/openstack/nova/+/950994 | |
| 13:59:35 | sean-k-mooney | gmaan: just an fyi i have seen no movement on https://review.opendev.org/c/openstack/cyborg/+/949772 so it might be time to bring it up on the tc call. i did not see any responce to my email either. | |
| 14:00:44 | sean-k-mooney | gmaan: if we dont see any movement on this by m2 i will put up a patch to nova to move the job to the perodic-weekly pipelien and experimental instead of check | |
| 14:06:03 | opendevreview | Balazs Gibizer proposed openstack/nova master: Add spawn_on https://review.opendev.org/c/openstack/nova/+/948079 | |
| 14:06:04 | opendevreview | Balazs Gibizer proposed openstack/nova master: Move ConductorManager to use spawn_on https://review.opendev.org/c/openstack/nova/+/948187 | |
| 14:06:04 | opendevreview | Balazs Gibizer proposed openstack/nova master: Move ComputeManager to use spawn_on https://review.opendev.org/c/openstack/nova/+/948186 | |
| 14:06:05 | opendevreview | Balazs Gibizer proposed openstack/nova master: Rename DEFAULT_GREEN_POOL to DEFAULT_EXECUTOR https://review.opendev.org/c/openstack/nova/+/948086 | |
| 14:06:05 | opendevreview | Balazs Gibizer proposed openstack/nova master: Make nova.utils.pass_context private https://review.opendev.org/c/openstack/nova/+/948188 | |
| 14:06:06 | opendevreview | Balazs Gibizer proposed openstack/nova master: Print ThreadPool statistics https://review.opendev.org/c/openstack/nova/+/948340 | |
| 14:06:06 | opendevreview | Balazs Gibizer proposed openstack/nova master: Make the default executor configurable https://review.opendev.org/c/openstack/nova/+/948087 | |
| 14:06:07 | opendevreview | Balazs Gibizer proposed openstack/nova master: Allow services to start with threading https://review.opendev.org/c/openstack/nova/+/948311 | |
| 14:06:07 | opendevreview | Balazs Gibizer proposed openstack/nova master: Document threading mode and tuneables https://review.opendev.org/c/openstack/nova/+/949364 | |
| 14:06:09 | opendevreview | Balazs Gibizer proposed openstack/nova master: Do not yield in threading mode https://review.opendev.org/c/openstack/nova/+/950994 | |
| 14:06:09 | opendevreview | Balazs Gibizer proposed openstack/nova master: Run nova-next with n-sch in threading mode https://review.opendev.org/c/openstack/nova/+/948450 | |
| 14:06:11 | opendevreview | Balazs Gibizer proposed openstack/nova master: Run nova-api and -metadata in threaded mode https://review.opendev.org/c/openstack/nova/+/951957 | |
| 14:18:22 | opendevreview | Balazs Gibizer proposed openstack/nova master: Document threading mode and tuneables https://review.opendev.org/c/openstack/nova/+/949364 | |
| 14:18:23 | opendevreview | Balazs Gibizer proposed openstack/nova master: Run nova-next with n-sch in threading mode https://review.opendev.org/c/openstack/nova/+/948450 | |
| 14:18:23 | opendevreview | Balazs Gibizer proposed openstack/nova master: Allow services to start with threading https://review.opendev.org/c/openstack/nova/+/948311 | |
| 14:18:24 | opendevreview | Balazs Gibizer proposed openstack/nova master: Run nova-api and -metadata in threaded mode https://review.opendev.org/c/openstack/nova/+/951957 | |
| 14:18:24 | opendevreview | Balazs Gibizer proposed openstack/nova master: Do not yield in threading mode https://review.opendev.org/c/openstack/nova/+/950994 | |
| 14:19:18 | gibi | Uggla: I thanks for the reviews on the eventlet series. I think I fixed / answered all your questions and comments now. | |
| 14:19:45 | gibi | I'm still working on some of the executor statistcs improvements we discussed on the sync | |
| 14:21:47 | Uggla | gibi, thanks. I hope it was useful. I'll try to do a new check next week. Regarding the stats ok. Tbh I thought that the outcome was let's start with the current stuffs. We will improve along the way. | |
| 14:22:20 | Uggla | But you may already spot some improvements. | |
| 14:26:01 | gibi | Uggla: yes they were usefull | |
| 14:26:53 | gibi | Uggla: yepp the remaining stats improvements (task waiting time, executor thread sleep time / utiliziation) will be top of the series not inside it. So they are not blocking landing what we have | |
| 15:22:52 | gmaan | sean-k-mooney: yes, I am also monitoring it and brought it in TC meeting last week https://meetings.opendev.org/meetings/tc/2025/tc.2025-06-10-17.00.log.html#l-145 | |
| 15:24:00 | gmaan | sean-k-mooney: I am adding the same in next week meeting agenda where (if no response) TC will take call for next action. As per process TC need to mark project inactive before m-2 so that release team can have final list of deliverables to release that cycle. | |
| 15:24:39 | gmaan | this is current inactive project list and process https://governance.openstack.org/tc/reference/emerging-technology-and-inactive-projects.html#current-inactive-projects | |
| 17:24:17 | sean-k-mooney | gmaan: ack thanks | |
| 21:54:45 | opendevreview | Pierre Riteau proposed openstack/nova master: Remove Unicode characters https://review.opendev.org/c/openstack/nova/+/952621 | |
| #openstack-nova - 2025-06-14 | |||
| 06:08:58 | opendevreview | melanie witt proposed openstack/nova master: Add hw_tpm_secret_security image property https://review.opendev.org/c/openstack/nova/+/940196 | |
| 06:08:59 | opendevreview | melanie witt proposed openstack/nova master: TPM: migrate legacy instances to new security policy https://review.opendev.org/c/openstack/nova/+/942501 | |
| 06:08:59 | opendevreview | melanie witt proposed openstack/nova master: Add hw:tpm_secret_security extra spec validation https://review.opendev.org/c/openstack/nova/+/940197 | |
| 06:09:00 | opendevreview | melanie witt proposed openstack/nova master: TPM: support instances with `host` secret security https://review.opendev.org/c/openstack/nova/+/941795 | |
| 06:09:00 | opendevreview | melanie witt proposed openstack/nova master: TPM: support booting instances with `user` secret security https://review.opendev.org/c/openstack/nova/+/942502 | |
| 06:09:01 | opendevreview | melanie witt proposed openstack/nova master: TPM: support instances with `deployment` secret security https://review.opendev.org/c/openstack/nova/+/942021 | |
| 06:09:01 | opendevreview | melanie witt proposed openstack/nova master: TPM: support live migration of `host` secret security https://review.opendev.org/c/openstack/nova/+/941483 | |
| 06:09:02 | opendevreview | melanie witt proposed openstack/nova master: TPM: support live migration of `deployment` secret security https://review.opendev.org/c/openstack/nova/+/925771 | |
| 06:09:03 | opendevreview | melanie witt proposed openstack/nova master: TPM: test live migration between hosts with different security https://review.opendev.org/c/openstack/nova/+/952629 | |
| 06:09:03 | opendevreview | melanie witt proposed openstack/nova master: Add vtpm_secret_(uuid|value) to LiveMigrateData https://review.opendev.org/c/openstack/nova/+/952628 | |
| 06:09:05 | opendevreview | melanie witt proposed openstack/nova master: TPM: update instance request_spec with secret security https://review.opendev.org/c/openstack/nova/+/952630 | |
| #openstack-nova - 2025-06-16 | |||
| 06:10:29 | opendevreview | Rajesh Tailor proposed openstack/nova-specs master: Show finish_time field in instance action show https://review.opendev.org/c/openstack/nova-specs/+/929780 | |
| 08:57:11 | opendevreview | Lajos Katona proposed openstack/os-vif stable/2025.1: OVS Trunk: Add bridge_name to external_ids https://review.opendev.org/c/openstack/os-vif/+/952567 | |
| 11:11:24 | opendevreview | Merged openstack/nova master: Remove Unicode characters https://review.opendev.org/c/openstack/nova/+/952621 | |
| 11:23:36 | opendevreview | Balazs Gibizer proposed openstack/nova master: Warn on long task wait time for executor https://review.opendev.org/c/openstack/nova/+/952666 | |
| 15:05:22 | opendevreview | Merged openstack/nova master: api: Address issues with instance actions API https://review.opendev.org/c/openstack/nova/+/951941 | |
| 16:12:27 | Uggla | Reminder that tomorrow is spec review day. | |
| 16:39:19 | bauzas | Uggla: good point, thanks for the reminder ! | |
| 16:39:41 | Uggla | bauzas, you are welcome. | |
| 16:39:46 | sean-k-mooney | oh ya... i had forgotten about that | |
| 16:42:29 | Uggla | And we have a lot of spec to review. | |
| 17:33:55 | gibi | dansmith: sean-k-mooney: do I see it correctly that the build_and_run_instance does not ever report error and revert task state at failure via these decorators as the implementation immediately calls spawn but never waits for the result so the error is not raised back? https://github.com/openstack/nova/blob/64ca204c9cf497b0dcfff2d3a24b0dd795a57d1d/nova/compute/manager.py#L2341-L2344 | |
| 17:34:41 | gibi | I need to drop but I can read back tomorrow and try to reproduce it as a bug | |
| 17:35:17 | dansmith | looks like it | |
| 17:35:31 | dansmith | maybe those decorators need to be moved to the inner _locked funciton | |
| 17:36:31 | dansmith | the finally in there will mark the instance is error I think, so it might only be the task_state (not important on a new instance you are going to delete) and the event (which might not be noticed either) | |
| 18:29:15 | sean-k-mooney | ya that looks like what is happening dan is likely correct about moving them although we woudl have to look at git blame to see when/how they were orgianlly added | |
| 18:32:46 | sean-k-mooney | it looks like we used to do some network validation https://opendev.org/openstack/nova/commit/1d8eddb2614de8daaddddd64ad1a8de4c215fe7a | |
| 18:33:47 | sean-k-mooney | well no not validation exactly | |
| 18:34:30 | sean-k-mooney | its possibel althogh unlikely that the objects.NetworkRequest.from_tuple(t) calls coudl have raised. | |
| 18:35:43 | sean-k-mooney | _do_build_and_run_instance seams to have the real logic that we need those decorator for. | |
| 18:37:12 | sean-k-mooney | ah https://opendev.org/openstack/nova/commit/437a1e92c11d51fb828d355091cdcdd4694dfed7 that when they were actully added | |
| 18:38:17 | sean-k-mooney | and orgianlly we did not use spawn_n. it is technically possibel for @utils.synchronized(instance.uuid) to raise as well | |
| 21:32:00 | opendevreview | melanie witt proposed openstack/nova master: Add vtpm_secret_(uuid|value) to LiveMigrateData https://review.opendev.org/c/openstack/nova/+/952628 | |
| 21:32:01 | opendevreview | melanie witt proposed openstack/nova master: TPM: support instances with `deployment` secret security https://review.opendev.org/c/openstack/nova/+/942021 | |
| 21:32:01 | opendevreview | melanie witt proposed openstack/nova master: TPM: support live migration of `host` secret security https://review.opendev.org/c/openstack/nova/+/941483 | |
| 21:32:02 | opendevreview | melanie witt proposed openstack/nova master: TPM: test live migration between hosts with different security https://review.opendev.org/c/openstack/nova/+/952629 | |
| 21:32:02 | opendevreview | melanie witt proposed openstack/nova master: TPM: support live migration of `deployment` secret security https://review.opendev.org/c/openstack/nova/+/925771 | |
| 21:32:03 | opendevreview | melanie witt proposed openstack/nova master: TPM: update instance request_spec with secret security https://review.opendev.org/c/openstack/nova/+/952630 | |
| #openstack-nova - 2025-06-17 | |||
| 09:55:53 | Uggla | Reminder: Today is spec review day. | |
| 09:56:20 | Uggla | Happy reviews. | |
| 12:41:35 | priteau | Hello Nova folks. I think I have hit this bug last week: https://bugs.launchpad.net/nova/+bug/2084238. I shelved an instance with a GPU device in PCI passthrough mode, and now it won't unshelve. Any thoughts on the issue? | |
| 13:29:33 | opendevreview | Merged openstack/nova-specs master: Re-propose "libvirt: AMD SEV-ES support" for 2025.2 https://review.opendev.org/c/openstack/nova-specs/+/950429 | |
| 13:40:52 | dansmith | priteau: is it pre-existing instances (instanced created before the upgrade) that are stuck? | |
| 13:46:10 | priteau | Yes, this instance is a few years old. | |
| 13:50:09 | dansmith | priteau: okay does that mean new instances aren't affected? | |
| 13:50:27 | dansmith | I'd have to look at what's going on here, but sounds like maybe a failed migration | |
| 13:50:33 | dansmith | have you run all your online data migrations post-upgrade? | |
| 14:01:20 | priteau | We've done all upgrades in the same way, with Kolla, which should have run all database migrations | |
| 14:05:27 | dansmith | priteau: not all data migrations are critical, and some are opportunistic | |