| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2026-03-19 | |||
| 17:22:16 | sean-k-mooney | so that there are enouch cpus free btu the numa node with the pci device is full | |
| 17:22:48 | sean-k-mooney | https://review.opendev.org/c/openstack/nova/+/978954/4/nova/tests/functional/libvirt/test_pci_sriov_servers.py#4654 | |
| 17:22:59 | sean-k-mooney | its still not correct in v4 | |
| 17:23:11 | Uggla | sean-k-mooney, it is what I did, boot 2 vm, delete 1, spawn a new one that must fail. | |
| 17:23:31 | Uggla | unless I miss something. | |
| 17:23:42 | sean-k-mooney | Uggla: i dont see the vm delete you ddidnt delete server 1 | |
| 17:23:50 | sean-k-mooney | did you do that locally? | |
| 17:24:49 | dansmith | sean-k-mooney: L5870 | |
| 17:25:09 | sean-k-mooney | thats a diffent test | |
| 17:25:09 | dansmith | and if you see my previous review feedback it'll point it out | |
| 17:26:12 | sean-k-mooney | test_create_server_with_pci_dev_and_numa_fails vs test_create_server_with_pci_dev_and_numa | |
| 17:26:23 | sean-k-mooney | the fails one is the one that is incorrect | |
| 17:27:01 | sean-k-mooney | the fails one now would fail even without the pci device becasue there are not enough dedicated cpus | |
| 17:28:41 | dansmith | in my previous feedback, I noted that that test was the same as the fails one above, and commented that we should cover the case where he deleted one to make sure it was failing for the right reason | |
| 17:28:56 | dansmith | seems like both cases are covered now, I'm not sure why it matters which one does it | |
| 17:30:20 | sean-k-mooney | there are only 8 pCPUs on the host | |
| 17:30:34 | sean-k-mooney | if you boot 2 vms as palce hoslder and dont delete 1 of them | |
| 17:30:48 | sean-k-mooney | (that both request 4 cpus) | |
| 17:31:01 | sean-k-mooney | the third vm will get a no valid host regardless fo the pci request | |
| 17:33:54 | Uggla | sean-k-mooney, please make a comment I'll try to fix it tomorrow morning. | |
| 17:34:12 | sean-k-mooney | yep i have it written im jsut reviewing the rest so i can leave them all in one go | |
| 17:35:45 | Uggla | sean-k-mooney, also if you have not seen 946945: Add reproducer test for bug 2105896 | https://review.opendev.org/c/openstack/nova/+/946945 I rebase and stack the fix on top. Coz I would like to close this bug too. | |
| 17:36:57 | sean-k-mooney | Uggla: thanks for rebasing it. no i didnt see that you did that | |
| 17:37:33 | sean-k-mooney | and yes it would be nice to fix that im fine with stacking them or if you want to take that over feel free | |
| 17:37:51 | Uggla | I had to signoff the patch, so that's the reason why I set me as co author. | |
| 17:38:01 | sean-k-mooney | ya that ok | |
| 17:38:17 | sean-k-mooney | i just created the repoducer to help the person who was developign the bug fix | |
| 17:38:18 | Uggla | in any case keeping you as the original author. | |
| 17:39:22 | sean-k-mooney | well not quite i guess i created trhe repoducer ealier but never got around to fixing it | |
| 17:39:40 | Uggla | you can have a look at the fix too. Basically I have just added the reproducer part in the fix that was missing (so setting me as co author on the fix too). | |
| 17:40:46 | Uggla | So if you can confirm the direction is ok, then I'll look for cores to merge them. | |
| 17:49:35 | sean-k-mooney | sure proably not today but i can take a look tomorrow | |
| 17:49:51 | sean-k-mooney | ill see if i can review it today when i finish with your chagne | |
| 18:00:34 | sean-k-mooney | Uggla: comments pushed https://review.opendev.org/c/openstack/nova/+/978954/4/nova/tests/functional/libvirt/test_pci_sriov_servers.py there are a few minor testign regressions but the over all patch looks good | |
| 18:01:51 | Uggla | sean-k-mooney, thx, I'll try to fix that tomorrow morning with a fredh mind. | |
| 18:02:18 | sean-k-mooney | no worries. the changes are subtle so best not to do it while tired | |
| 19:33:12 | opendevreview | Lajos Katona proposed openstack/nova master: WIP: Use SDK for Neutron security-groups https://review.opendev.org/c/openstack/nova/+/981141 | |
| 19:34:08 | sean-k-mooney | Uggla: oh i see did orgianly take over https://review.opendev.org/c/openstack/nova/+/946079/6 and update it for the first person tha ttried to fix it but they abandoned the patch and https://review.opendev.org/c/openstack/nova/+/970337 is a new attempt to fix it | |
| 19:35:47 | sean-k-mooney | so it not usign the orginal fix i was working at least not in the same way | |
| 20:51:24 | sean-k-mooney | Uggla: if you want to chat more about https://review.opendev.org/c/openstack/nova/+/970337/comment/4abdf064_10cf13ce/ tommorow let me know i think https://review.opendev.org/c/openstack/nova/+/946079 was more compelte fix and it shoudl not have been abandoned | |
| 21:52:57 | opendevreview | Dmitriy Chubinidze proposed openstack/nova master: docs: mention q35 machine type for UEFI guests https://review.opendev.org/c/openstack/nova/+/981413 | |
| 21:59:53 | opendevreview | Dmitriy Chubinidze proposed openstack/nova master: docs: mention q35 machine type for UEFI guests https://review.opendev.org/c/openstack/nova/+/981413 | |
| #openstack-nova - 2026-03-20 | |||
| 10:25:35 | opendevreview | ribaudr proposed openstack/nova master: Fix NUMA cell sorting for non-PCI VMs with pack strategy https://review.opendev.org/c/openstack/nova/+/978954 | |
| 10:25:35 | opendevreview | ribaudr proposed openstack/nova master: Reproduce bug #2144660: NUMA cell sorting bug for non-PCI VMs https://review.opendev.org/c/openstack/nova/+/978885 | |
| 11:05:05 | Uggla | sean-k-mooney, dansmith, I have answered comments ^ so if you can have a new review run. It will be great. | |
| 11:28:01 | opendevreview | Dmitriy Chubinidze proposed openstack/nova master: docs: mention q35 machine type for UEFI guests https://review.opendev.org/c/openstack/nova/+/981413 | |
| 12:28:07 | sean-k-mooney | Uggla: im +2 on both of those now. if you need to respine a release not would be nice but its not required | |
| 13:20:40 | Uggla | sean-k-mooney, yes I will write a release as discussed yesterday. | |
| 13:21:20 | Uggla | sean-k-mooney I'll keep you posted when it is done. | |
| 13:21:49 | opendevreview | ribaudr proposed openstack/nova master: Fix error when multiple security group in a project with same name https://review.opendev.org/c/openstack/nova/+/970337 | |
| 13:22:45 | Uggla | sean-k-mooney I have just pushed an update of https://review.opendev.org/c/openstack/nova/+/970337. Tell me if I understand well what you wanted to achieve. | |
| 13:31:24 | sean-k-mooney | ack ill take a look after im off a call | |
| 13:31:54 | opendevreview | ribaudr proposed openstack/nova master: Fix NUMA cell sorting for non-PCI VMs with pack strategy https://review.opendev.org/c/openstack/nova/+/978954 | |
| 13:36:51 | Uggla | sean-k-mooney release notes added in ^ | |
| 13:53:52 | sean-k-mooney | +2 on the sorting series ill re review https://review.opendev.org/c/openstack/nova/+/970337/4 in a little more detail | |
| 13:54:31 | sean-k-mooney | i think you made the changes i asked for but it woudl also be nice to have a very short release note for the security group bug too | |
| 13:55:42 | sean-k-mooney | nova.tests.unit.api.openstack.compute.test_schemas.SchemaTest.test_schemas | |
| 13:56:05 | sean-k-mooney | seems to be hitting the per test timeout on slower nodes in the coverage job | |
| 13:56:20 | sean-k-mooney | i wonder if we shoudl tweak that in the tox.ini | |
| 13:59:28 | Uggla | sean-k-mooney ok for the release note. "seems to be hitting the per test timeout on slower nodes in the coverage job" how do you see that ? | |
| 14:13:44 | sean-k-mooney | i looked at the coverage job on the repoducer | |
| 14:14:05 | sean-k-mooney | the schema validation test timed out but i have seen that on one or two other patches i think | |
| 14:14:13 | sean-k-mooney | so https://zuul.opendev.org/t/openstack/build/b2e50d5600ce479ca7287a2ebae35e79 | |
| 14:14:41 | sean-k-mooney | File "/home/zuul/src/opendev.org/openstack/nova/.tox/cover/lib/python3.12/site-packages/fixtures/_fixtures/timeout.py", line 57, in signal_handler | |
| 14:14:43 | sean-k-mooney | raise TimeoutException() | |
| 14:14:45 | sean-k-mooney | fixtures._fixtures.timeout.TimeoutException | |
| 14:15:09 | sean-k-mooney | that isnet a job timeout its a test timeout all the other test pass and it passed in the non coverage jobs | |
| 14:15:31 | sean-k-mooney | the corevate profileing slows thigns down a bit so i think on a slow node it just take longer then the defautl test timeout | |
| 14:16:24 | sean-k-mooney | im not 100% sure that is the cause as i have not looked at it in detail | |
| 14:16:35 | sean-k-mooney | Uggla: its unrelated to your change | |
| 14:17:00 | sean-k-mooney | i just wanted to mention it before recheckign incase we are sing this in ohter patches as it could become a semi gate blocker if that is unstable | |
| 14:18:10 | Uggla | sean-k-mooney, oh ok. I guess we can keep an eye on it, and discuss in next upstream meeting. | |
| 14:20:27 | obre | Is there any chance that this bugfix gets merged before the G-release? https://review.opendev.org/c/openstack/nova/+/916409 (A bug making rebuild of a BFV-instance impossible) | |
| 14:21:44 | sean-k-mooney | obre: unless the bug was intoduced in this cycle no | |
| 14:22:02 | sean-k-mooney | we are past rc1 so rc2+ only incldue bugs that were intoduced in this release | |
| 14:22:17 | sean-k-mooney | that does not mean it cant be backproted and release reight after | |
| 14:25:39 | Uggla | obre I have included it in the bug review list | |
| 14:26:47 | opendevreview | ribaudr proposed openstack/nova master: Fix error when multiple security group in a project with same name https://review.opendev.org/c/openstack/nova/+/970337 | |
| 14:27:22 | sean-k-mooney | obre: its in merge conflict and the orginal autor is not working on it | |
| 14:27:38 | sean-k-mooney | so someone will have to fix that again i might get to it next week | |
| 14:28:03 | sean-k-mooney | but it need review form peopel other then me to prgoress | |
| 14:28:37 | sean-k-mooney | if i find time ill push an update later today | |
| 14:28:50 | Uggla | sean-k-mooney, what is your opinion on that one: https://bugs.launchpad.net/nova/+bug/2145010 to me it is more a feature request than bug. Do you share the same feeling ? | |
| 14:29:29 | sean-k-mooney | so the are raising a commont issue | |
| 14:29:46 | sean-k-mooney | they are correct that day 1 it shoudl have used both | |
| 14:29:49 | sean-k-mooney | os_distro="debian" | |
| 14:29:51 | sean-k-mooney | os_version="13" | |
| 14:30:07 | sean-k-mooney | but the spec defied that os_distro="debian13" woudl be required | |
| 14:30:14 | sean-k-mooney | i.e that only os_distro woudl be used | |
| 14:30:23 | sean-k-mooney | even though os_version exsited | |
| 14:30:32 | sean-k-mooney | so that is a breakign api change in my view | |
| 14:30:45 | sean-k-mooney | as it will chenat the behvior for exsitng images | |
| 14:31:18 | sean-k-mooney | so i woudl close it as invalid but im biased again that feature as i personlly want to remove the libosinfo functionality entrily | |
| 14:31:26 | sean-k-mooney | i belive it was a mistake to ever include it | |
| 14:31:53 | sean-k-mooney | becuase it does nto provdie stable bevhor across version as they assuem the domian will not be deleted and recreated and rely on that for stablity | |
| 14:32:24 | sean-k-mooney | nova deletes and recreates the domain all the time so if you sue the libosinfo fucntionaltiy you can get unexpected bhavior in nova when you do a package update | |
| 14:33:28 | sean-k-mooney | i partly mitigated that in the past i think but it still makes me uneasey so i woudl mark this opion and whishlist | |
| 14:33:56 | sean-k-mooney | i dont think it woudl be safe to turn this on by defautl (looking at both disto and version) without a flag to opt in | |
| 14:34:34 | sean-k-mooney | they are totally right to expect it to look at both its just not how it was implmeented unfortunetly | |
| 14:35:01 | sean-k-mooney | https://specs.openstack.org/openstack/nova-specs/specs/liberty/approved/libvirt-hardware-policy-from-libosinfo.html | |