| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2025-04-25 | |||
| 14:32:25 | sean-k-mooney | i dont disagree with that sentiemtn | |
| 14:32:32 | dansmith | and if we had no coverage on anything, that's not a good reason to just make untested changes because there's nobody watching | |
| 14:32:44 | sean-k-mooney | woudl you like me to pull it form the queue or defer to a follow up i nthe series | |
| 14:32:53 | dansmith | I just broke the new thing and am running functional so we'll see, hopefully something will notice | |
| 14:33:06 | dansmith | sean-k-mooney: is it in the actual gate? | |
| 14:33:41 | sean-k-mooney | the gate jobs are running | |
| 14:33:43 | gibi | I'm OK to pull it and I can try to add extra coverage. | |
| 14:34:05 | sean-k-mooney | ok ill drop +w | |
| 14:34:31 | dansmith | a bunch of the quota functional tests have run and it didn't fail.. long ways to go still, but... | |
| 14:37:01 | dansmith | {6} nova.tests.functional.db.test_quota.QuotaTestCase.test_server_group_members_count_by_user_2_False [0.224107s] ... FAILED | |
| 14:39:25 | dansmith | so that one is at least running the code, which is good, but I don't see that it has any build requests to check that part | |
| 14:46:33 | sean-k-mooney | dansmith: as an asside do you know if there is a reason we dont include the funtioal test in the coverage report today? i would guess its runtime but i never really tought about it | |
| 14:46:53 | dansmith | yeah I wish we had it too, | |
| 14:46:59 | sean-k-mooney | dansmith: im not sure if that woudl be a good or bad thing | |
| 14:47:24 | dansmith | I'm guessing because it's a separate job, but we could just make the regular functional be the coverage one, if not for the inherited job thing :/ | |
| 14:47:26 | dansmith | I'd like to have it | |
| 14:47:27 | sean-k-mooney | dansmith: well i think we could do it. it might be something i hack on the next time i fell like hacking on those kidn of thing | |
| 14:47:42 | dansmith | I run a lot of coverage reports locally when I suspect someone hasn't actually added a test | |
| 14:48:03 | sean-k-mooney | well tis just calling our tox target so that josb can run anything we want if we update it. | |
| 14:48:51 | sean-k-mooney | what i was mroe wondering is woudl we want a combined coverage reprot (form both testsuites) or seperate unit/functional reports | |
| 14:49:22 | sean-k-mooney | job wise i think combined is nice but ya i just said i woudl ask | |
| 14:49:24 | dansmith | we could, maybe that would be better | |
| 14:50:06 | dansmith | gibi: okay, one test runs that code and doesn't have (AFAICT) any build requests | |
| 14:50:39 | dansmith | gibi: if you wouldn't mind at least mentioning in the commit message that we don't need additional testing, that would be appreciated, | |
| 14:50:55 | dansmith | but adding BR coverage to that existing test seems prudent to me as well, since you're changing that code | |
| 14:51:18 | gibi | dansmith: I will add the missing coverage. No problemo | |
| 14:51:22 | gibi | :) | |
| 14:53:01 | dansmith | gibi: thanks | |
| 15:10:28 | opendevreview | Merged openstack/nova master: FUP improve and add integration tests for PCI SR-IOV servers https://review.opendev.org/c/openstack/nova/+/944106 | |
| 16:11:20 | opendevreview | Thomas Rotsaert proposed openstack/nova master: feat(hypervisors) - Adding new hypervisor property `hypervisor_rack`. https://review.opendev.org/c/openstack/nova/+/948248 | |
| 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 | |