| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2025-06-12 | |||
| 17:41:44 | sean-k-mooney | TheJulia: the ironic driver enriches the network metadta for exmaple https://github.com/openstack/nova/blob/master/nova/virt/ironic/driver.py#L1045-L1106 | |
| 17:42:36 | sean-k-mooney | the rest use https://github.com/openstack/nova/blob/master/nova/api/metadata/base.py#L112 | |
| 17:44:22 | sean-k-mooney | so we have an objet in memoy thaat can redner subpaths into json for the metadta api https://github.com/openstack/nova/blob/master/nova/api/metadata/base.py#L317 | |
| 17:45:00 | sean-k-mooney | or produce the content for the config drive https://github.com/openstack/nova/blob/master/nova/api/metadata/base.py#L590 | |
| 17:45:06 | TheJulia | Yeah, for portgroups. we can likely just rip that out and address it locally in ironic at some point | |
| 17:46:00 | sean-k-mooney | so it would not take much to jsut add a method to this obejc tthat wil return the data in a singel json dict | |
| 17:46:07 | sean-k-mooney | filepath -> file content | |
| 17:46:16 | TheJulia | yeah, that would be fairly straight forward | |
| 17:46:38 | sean-k-mooney | that acctully what that generator returns already | |
| 17:47:06 | sean-k-mooney | its just a mateter of passing it to ironic in a way that would work better for ye | |
| 17:47:58 | TheJulia | something for some point down the road | |
| 17:50:23 | sean-k-mooney | yep, we can always imrpove things like that if we know about the pain point and someone has time to od the work | |
| 17:51:24 | sean-k-mooney | TheJulia: speaning of improment https://review.opendev.org/c/openstack/nova-specs/+/471815 | |
| 17:51:55 | sean-k-mooney | i need to get back to that but tha tis aimed at making the data we pass more useful for the vlan trunking case | |
| 17:52:28 | sean-k-mooney | although that not nessisarly ironic speicific it shoudl be useful for any dirver the support neutron trunk ports | |
| 17:52:37 | TheJulia | Hmmmm | |
| 17:52:49 | TheJulia | I'll add that to my list to look at when I don't have a migraine | |
| 17:52:55 | TheJulia | Unfortunately, I woke up with one today :( | |
| 17:53:43 | sean-k-mooney | that sucks, i often get eystrain headaches or if i forget to take my blodd pressure medication but i fortunate to never had to deal with migraines | |
| 17:55:23 | sean-k-mooney | that actully is propsoing inlcudign the mtu info for the vlan sub intereface by the way | |
| 17:56:46 | TheJulia | oh good, becuase that would be a logical requirement. | |
| #openstack-nova - 2025-06-13 | |||
| 07:11:21 | opendevreview | Johannes Beisiegel proposed openstack/nova master: Adds packing_host_pci_numa_cells_allocation_strategy option https://review.opendev.org/c/openstack/nova/+/952451 | |
| 08:55:37 | opendevreview | Balazs Gibizer proposed openstack/nova stable/2024.1: Cache [pci]alias parsing https://review.opendev.org/c/openstack/nova/+/952489 | |
| 10:20:32 | 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 | |
| 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:56 | opendevreview | Balazs Gibizer proposed openstack/nova master: Use futurist for _get_default_green_pool() https://review.opendev.org/c/openstack/nova/+/948072 | |
| 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:57 | opendevreview | Balazs Gibizer proposed openstack/nova master: Add spawn_on https://review.opendev.org/c/openstack/nova/+/948079 | |
| 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:58 | opendevreview | Balazs Gibizer proposed openstack/nova master: Move ConductorManager to use spawn_on https://review.opendev.org/c/openstack/nova/+/948187 | |
| 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: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: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: Print ThreadPool statistics https://review.opendev.org/c/openstack/nova/+/948340 | |
| 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:03 | opendevreview | Balazs Gibizer proposed openstack/nova master: Allow services to start with threading https://review.opendev.org/c/openstack/nova/+/948311 | |
| 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:05 | opendevreview | Balazs Gibizer proposed openstack/nova master: Do not yield in threading mode https://review.opendev.org/c/openstack/nova/+/950994 | |
| 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 | |
| 13:06:18 | opendevreview | Balazs Gibizer proposed openstack/nova master: Print ThreadPool statistics https://review.opendev.org/c/openstack/nova/+/948340 | |
| 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:19 | opendevreview | Balazs Gibizer proposed openstack/nova master: Allow services to start with threading https://review.opendev.org/c/openstack/nova/+/948311 | |
| 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:20 | opendevreview | Balazs Gibizer proposed openstack/nova master: Do not yield in threading mode https://review.opendev.org/c/openstack/nova/+/950994 | |
| 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: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 ComputeManager to use spawn_on https://review.opendev.org/c/openstack/nova/+/948186 | |
| 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:05 | opendevreview | Balazs Gibizer proposed openstack/nova master: Make nova.utils.pass_context private https://review.opendev.org/c/openstack/nova/+/948188 | |
| 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:06 | opendevreview | Balazs Gibizer proposed openstack/nova master: Make the default executor configurable https://review.opendev.org/c/openstack/nova/+/948087 | |
| 14:06:06 | opendevreview | Balazs Gibizer proposed openstack/nova master: Print ThreadPool statistics https://review.opendev.org/c/openstack/nova/+/948340 | |
| 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:07 | opendevreview | Balazs Gibizer proposed openstack/nova master: Allow services to start with threading https://review.opendev.org/c/openstack/nova/+/948311 | |
| 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:09 | opendevreview | Balazs Gibizer proposed openstack/nova master: Do not yield in threading mode https://review.opendev.org/c/openstack/nova/+/950994 | |
| 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: Allow services to start with threading https://review.opendev.org/c/openstack/nova/+/948311 | |
| 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:24 | opendevreview | Balazs Gibizer proposed openstack/nova master: Do not yield in threading mode https://review.opendev.org/c/openstack/nova/+/950994 | |
| 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: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: Add hw:tpm_secret_security extra spec validation https://review.opendev.org/c/openstack/nova/+/940197 | |
| 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: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: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: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: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: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: Add vtpm_secret_(uuid|value) to LiveMigrateData https://review.opendev.org/c/openstack/nova/+/952628 | |
| 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: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 | |