| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2025-04-25 | |||
| 16:11:58 | opendevreview | Thomas Rotsaert proposed openstack/nova master: feat(hypervisors) - Adding new hypervisor property `hypervisor_rack`. https://review.opendev.org/c/openstack/nova/+/948248 | |
| 16:18:19 | opendevreview | ribaudr proposed openstack/nova stable/2025.1: FUP Remove unnecessary PCI check https://review.opendev.org/c/openstack/nova/+/946625 | |
| 16:23:16 | opendevreview | ribaudr proposed openstack/nova stable/2025.1: FUP Remove unnecessary PCI check https://review.opendev.org/c/openstack/nova/+/948251 | |
| 16:26:13 | opendevreview | ribaudr proposed openstack/nova stable/2025.1: FUP Remove unnecessary PCI check https://review.opendev.org/c/openstack/nova/+/946623 | |
| 16:39:13 | opendevreview | ribaudr proposed openstack/nova stable/2025.1: FUP Update pci-passthrough and virtual-gpu documentation https://review.opendev.org/c/openstack/nova/+/948253 | |
| 16:40:29 | opendevreview | ribaudr proposed openstack/nova stable/2025.1: FUP Update pci-passthrough and virtual-gpu documentation https://review.opendev.org/c/openstack/nova/+/948254 | |
| 16:41:00 | opendevreview | ribaudr proposed openstack/nova stable/2025.1: FUP Update pci-passthrough and virtual-gpu documentation https://review.opendev.org/c/openstack/nova/+/948254 | |
| 17:41:50 | opendevreview | Thomas Rotsaert proposed openstack/nova master: feat(hypervisors) - Adding new hypervisor property `hypervisor_rack`. https://review.opendev.org/c/openstack/nova/+/948258 | |
| 17:43:41 | opendevreview | Thomas Rotsaert proposed openstack/nova master: feat(hypervisors) - Adding new hypervisor property `hypervisor_rack`. https://review.opendev.org/c/openstack/nova/+/948258 | |
| 19:21:35 | opendevreview | Merged openstack/nova master: Remove nova debugger functionality https://review.opendev.org/c/openstack/nova/+/922496 | |
| 19:21:47 | opendevreview | Merged openstack/nova master: split monkey_patching form import https://review.opendev.org/c/openstack/nova/+/922425 | |
| 19:21:58 | opendevreview | Merged openstack/nova master: Remove superfluous monkey patching form func test https://review.opendev.org/c/openstack/nova/+/947237 | |
| 19:41:41 | opendevreview | Merged openstack/nova master: Remove workaround for ovn live migration https://review.opendev.org/c/openstack/nova/+/946950 | |
| 21:16:12 | opendevreview | Thomas Rotsaert proposed openstack/nova master: feat(hypervisors) - Adding new hypervisor property `hypervisor_rack`. https://review.opendev.org/c/openstack/nova/+/948276 | |
| 21:16:48 | opendevreview | Thomas Rotsaert proposed openstack/nova master: feat(hypervisors) - Adding new hypervisor property `hypervisor_rack`. https://review.opendev.org/c/openstack/nova/+/948276 | |
| #openstack-nova - 2025-04-26 | |||
| 11:19:18 | gibi | sean-k-mooney: fyi the latest pyroute2 bump broke os-vif so I proposed a revert https://review.opendev.org/c/openstack/requirements/+/948283 | |
| 11:19:55 | sean-k-mooney[m] | didnt rodolfo fix that | |
| 11:20:20 | sean-k-mooney[m] | although we have not done a release since | |
| 11:21:18 | sean-k-mooney[m] | @gibi https://review.opendev.org/c/openstack/os-vif/+/946977 | |
| 11:21:47 | gibi | sean-k-mooney: that is just unit test adaptation | |
| 11:21:55 | gibi | what I see is interface detach broken in tempest | |
| 11:21:58 | sean-k-mooney[m] | gibi: oh its using asyncio | |
| 11:22:18 | gibi | yeah | |
| 11:22:25 | gibi | we need to remain on 0.8.1 I guess | |
| 11:22:30 | sean-k-mooney[m] | we do do that in tempset | |
| 11:22:47 | sean-k-mooney[m] | but i guess the bump was not done at the time | |
| 11:23:27 | sean-k-mooney[m] | we could try changing the executor for eventlet to asyncio but i think that also does not work properly yet | |
| 11:23:40 | sean-k-mooney[m] | ok well i think a revert for now makes sense | |
| 11:23:54 | sean-k-mooney[m] | and then we need to look deeper | |
| 11:24:22 | gibi | yeah | |
| 11:25:56 | gibi | anyhow I only stopped by for rechecks. I'm moving on :) | |
| 13:35:01 | frickler | I approved the revert, but I'd be interested in learning why this wasn't detected by the tempest job we are running on reqs, does it only affect ovs? maybe we should have a second tempest job in reqs, then? | |
| 14:56:20 | gibi | frickler: thanks. As far as I see it is only a problem if the interface detach times out due to slow node and retried by nova, the os-vif unplug is retried while I guess in pyroute2 something is still running for the first try. So probably the original tempest run does not hit it as it did not need to re-try the detach | |
| 18:15:19 | sean-k-mooney | i have not looked but there is a small chance that once we remvoe the linux bridge plugin we can entrily remove pyroute2 from os-vif | |
| #openstack-nova - 2025-04-28 | |||
| 01:06:42 | opendevreview | DongHun, Kim proposed openstack/nova master: Add support for memory allocation mode='immediate' in Nova https://review.opendev.org/c/openstack/nova/+/948303 | |
| 01:25:25 | opendevreview | DongHun, Kim proposed openstack/nova master: Add support for memory allocation mode='immediate' in Nova https://review.opendev.org/c/openstack/nova/+/948303 | |
| 01:42:44 | opendevreview | DongHun, Kim proposed openstack/nova master: Add support for memory allocation mode='immediate' in Nova https://review.opendev.org/c/openstack/nova/+/948303 | |
| 01:56:42 | opendevreview | DongHun, Kim proposed openstack/nova master: Add support for memory allocation mode='immediate' in Nova https://review.opendev.org/c/openstack/nova/+/948303 | |
| 02:47:31 | opendevreview | Masanori Kuroha proposed openstack/nova master: Copy applied provider.yaml https://review.opendev.org/c/openstack/nova/+/948304 | |
| 03:05:08 | opendevreview | Masanori Kuroha proposed openstack/nova master: Copy applied provider.yaml https://review.opendev.org/c/openstack/nova/+/948304 | |
| 04:28:05 | opendevreview | DongHun, Kim proposed openstack/nova master: Add support for memory allocation mode='immediate' in Nova https://review.opendev.org/c/openstack/nova/+/948303 | |
| 05:00:22 | opendevreview | DongHun, Kim proposed openstack/nova master: Add support for memory allocation mode='immediate' in Nova https://review.opendev.org/c/openstack/nova/+/948303 | |
| 05:41:12 | opendevreview | DongHun, Kim proposed openstack/nova master: Add support for memory allocation mode='immediate' in Nova https://review.opendev.org/c/openstack/nova/+/948303 | |
| 06:04:40 | opendevreview | DongHun, Kim proposed openstack/nova master: Add support for memory allocation mode='immediate' in Nova https://review.opendev.org/c/openstack/nova/+/948303 | |
| 06:08:18 | opendevreview | Balazs Gibizer proposed openstack/nova master: [quota]Refactor group counting to scatter-gather https://review.opendev.org/c/openstack/nova/+/948064 | |
| 06:08:19 | opendevreview | Balazs Gibizer proposed openstack/nova master: Use futurist for _get_default_green_pool() https://review.opendev.org/c/openstack/nova/+/948072 | |
| 06:08:19 | opendevreview | Balazs Gibizer proposed openstack/nova master: WIP:Translate scatter-gather to futurist https://review.opendev.org/c/openstack/nova/+/947966 | |
| 06:08:20 | opendevreview | Balazs Gibizer proposed openstack/nova master: Add spawn_on https://review.opendev.org/c/openstack/nova/+/948079 | |
| 06:08:20 | opendevreview | Balazs Gibizer proposed openstack/nova master: Replace utils.spawn_n with spawn https://review.opendev.org/c/openstack/nova/+/948076 | |
| 06:08:21 | opendevreview | Balazs Gibizer proposed openstack/nova master: Move ConductorManager to use spawn_on https://review.opendev.org/c/openstack/nova/+/948187 | |
| 06:08:21 | opendevreview | Balazs Gibizer proposed openstack/nova master: Move ComputeManager to use spawn_on https://review.opendev.org/c/openstack/nova/+/948186 | |
| 06:08:22 | opendevreview | Balazs Gibizer proposed openstack/nova master: Rename DEFAULT_GREEN_POOL to DEFAULT_EXECUTOR https://review.opendev.org/c/openstack/nova/+/948086 | |
| 06:08:22 | opendevreview | Balazs Gibizer proposed openstack/nova master: Make nova.utils.pass_context private https://review.opendev.org/c/openstack/nova/+/948188 | |
| 06:08:24 | opendevreview | Balazs Gibizer proposed openstack/nova master: WIP: allow service to start with threading https://review.opendev.org/c/openstack/nova/+/948311 | |
| 06:08:24 | opendevreview | Balazs Gibizer proposed openstack/nova master: Make the default executor configurable https://review.opendev.org/c/openstack/nova/+/948087 | |
| 06:31:01 | opendevreview | DongHun, Kim proposed openstack/nova master: Add support for memory allocation mode='immediate' in Nova https://review.opendev.org/c/openstack/nova/+/948303 | |
| 08:21:55 | mkuroha | @sean @gibi Hi, this patch is the initial implementation for the behavior change of additional.traits in provider.yaml that was discussed at the PTG: https://review.opendev.org/c/openstack/nova/+/948304 I would appreciate it if you could review the current code mm | |
| 08:48:04 | bauzas | gibi: so the goal for this week is about testing scatter-gather for the scheduler ? | |
| 08:52:57 | gibi | mkuroha: thanks for proposing that patch. It looks pretty clean and simple, so I like it. How do you feel about this approach? | |
| 08:54:55 | gibi | bauzas: yeah, I want to see the first light locally that nova-scheduler starts without monkey patching and with all the spawn calls actually taking native threads from a threadpool. If it start and not imediately fails that is already success. Then I will try throwing tempest at it and see if it works, does not hang, does not loose threads from the pool, etc. | |
| 08:55:34 | gibi | bauzas: it needs to depend the still open (and not fully working) oslo.sevice patch so this will be mostly local trials and maybe CI runs if I can extend devstack to pass env variable to nova-scheduler at start | |
| 08:56:27 | gibi | I expect that this means less actual reviewable patches proposed, and more WIP patches iterated | |
| 08:56:36 | gibi | this week. | |
| 08:57:01 | gibi | Also this week I'm off Thrusday, Friday | |
| 08:57:10 | gibi | so it is a short week anyhow | |
| 08:58:24 | gibi | so you can review the long series for spotting conceptual errors but that series is mostly WIP and expected to change based on the feedback provided by the local testing of scatter-gather in the scheduler in native threading mode. | |
| 08:58:43 | gibi | I will write up a post about how to replicate the local testing setup I have so others can try | |
| 09:10:27 | bauzas | gibi: cool, sorry wasn't nagged by a notification when you pinged me | |
| 09:10:44 | bauzas | gibi: I'll also be on PTO on Thur and Fri | |
| 09:11:36 | bauzas | gibi: so tbc, that's this gerrit branch that I could look at ? https://review.opendev.org/c/openstack/nova/+/948064/3 | |
| 09:12:20 | gibi | yes that is the one I'm trying to use to test things locally | |
| 09:12:36 | bauzas | noted | |
| 09:12:44 | gibi | there are two independent patches. A small delete https://review.opendev.org/c/openstack/nova/+/947260?usp=search | |
| 09:13:03 | gibi | and the replacement series here https://review.opendev.org/c/openstack/nova/+/947212 | |
| 09:13:29 | gibi | on the latter I expect Kamil to propose more similar patches this week | |
| 09:13:56 | bauzas | I already reviewed the hacking patch | |
| 09:14:10 | gibi | yeah, I just noticed. Thanks | |
| 09:14:41 | bauzas | for the moment, the removals and hacking patches were easy to review and without any concerns | |
| 09:14:45 | gibi | if you look at the recheck history of https://review.opendev.org/c/openstack/nova/+/947212 you see that our gate is not super happy | |
| 09:15:03 | gibi | I filed https://bugs.launchpad.net/glance/+bug/2109428 | |
| 09:15:04 | bauzas | but the futurist branch needs more brain time for me :)= | |
| 09:15:17 | gibi | bauzas: yeah the long barnch is more complicated | |
| 09:15:20 | gibi | and less hashed out | |
| 09:15:50 | bauzas | yeah saw the libvirt failure | |
| 09:15:55 | bauzas | seems a new regression IMHO | |
| 09:16:17 | gibi | also we reverted a pyroute2 bump over the weekend https://bugs.launchpad.net/glance/+bug/2109428 | |
| 09:16:27 | gibi | sorry worng link | |
| 09:16:28 | gibi | https://review.opendev.org/c/openstack/requirements/+/948283 | |
| 09:17:12 | gibi | but the author of pyroute2 already responded in https://bugs.launchpad.net/os-vif/+bug/2109396 so that is nicely moving along | |
| 09:17:42 | bauzas | thanks for the info | |
| 09:18:50 | gibi | now that I'm back to relying on the upstream CI, I will spend more time caring for it and filing bugs j) | |
| 09:18:54 | gibi | :) | |
| 09:31:07 | mkuroha | gibi: Thank you for the confirmation!!:) I have one concern: I'm wondering if there is any concern in the behavior when /etc/nova/provider.yaml is deleted while /applied_provider_config/provider.yaml is present. (Current: the additional.traits of the resource provider and the /applied_provider_config/provider.yaml file remain.). | |
| 09:31:33 | mkuroha | Additionally, in this patch, are there any tests other than the nova's functional tests that should be added? (This is my almost first patch for nova, and I'm not very familiar with it, so I would like to confirm it.) | |
| 09:33:07 | gibi | mkuroha: good point about the edge case of deleting /etc/nova/provider.yaml. I think the expected behavior in this case that all the traits that are listed in the applied file is removed and then the applied file is deleted at the end. | |
| 09:35:25 | gibi | mkuroha: the functional test looks good to me. Adding one for the deleted provider.yaml case would be nice too. If you feel that the merge_provider_config or other changes could benefit from some unit test then that is also OK to add. | |
| 09:36:04 | gibi | but for me the current PoC would be enough to write and approve a spec | |
| 09:37:32 | gibi | so I suggest to write a small spec based on the PoC, so that we can approve that by the rest of the team | |
| 09:40:15 | mkuroha | gibi: I see, thank you for your all comments!! I will start working on fixing the spec side :) | |
| 09:40:25 | gibi | mkuroha: thanks for the effort! | |