| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2026-03-17 | |||
| 17:39:30 | 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 | |
| 18:26:53 | gmaan | bauzas: when you are online tomorrow, please check this (running graceful shutdown job in both mode) https://review.opendev.org/c/openstack/nova/+/978292 | |
| 18:28:29 | gmaan | dansmith: can you please check this trivial one https://review.opendev.org/c/openstack/nova/+/968410 | |
| 18:32:22 | dansmith | gmaan: not _quite_ trivial but yep :) | |
| 18:32:59 | gmaan | :) thanks | |
| 19:14:48 | opendevreview | Merged openstack/nova master: Fix Flamingo version in qemu matrix. https://review.opendev.org/c/openstack/nova/+/980956 | |
| 20:19:55 | opendevreview | Merged openstack/nova master: [Trivial]: Remove unused RBAC default alias https://review.opendev.org/c/openstack/nova/+/968410 | |
| #openstack-nova - 2026-03-18 | |||
| 03:45:39 | opendevreview | OpenStack Proposal Bot proposed openstack/nova master: Imported Translations from Zanata https://review.opendev.org/c/openstack/nova/+/980544 | |
| 08:15:19 | opendevreview | Kamil Sambor proposed openstack/nova master: Replace eventlet with standard threading in novncproxy https://review.opendev.org/c/openstack/nova/+/976089 | |
| 09:05:46 | opendevreview | ribaudr proposed openstack/nova stable/2026.1: Fix Flamingo version in qemu matrix. https://review.opendev.org/c/openstack/nova/+/981085 | |
| 09:09:47 | opendevreview | ribaudr proposed openstack/nova master: Update min support for Hibiscus https://review.opendev.org/c/openstack/nova/+/979301 | |
| 10:55:30 | opendevreview | Gaurav Tomar proposed openstack/nova master: Fix device name duplication error in libvirt during instance resize. https://review.opendev.org/c/openstack/nova/+/981092 | |
| 14:00:01 | opendevreview | Stephen Finucane proposed openstack/nova master: trivial: Clean manager.Manager, service.Service signatures https://review.opendev.org/c/openstack/nova/+/764806 | |
| 15:21:33 | opendevreview | Lajos Katona proposed openstack/nova master: WIP: Use SDK for Neutron security-groups https://review.opendev.org/c/openstack/nova/+/981141 | |
| 16:51:11 | opendevreview | Sylvain Bauza proposed openstack/nova master: DNM: Test resize with 410 computes but one disabled https://review.opendev.org/c/openstack/nova/+/981153 | |
| 17:21:15 | melwitt | elodilles: hey I just noticed that gmaan is not part of nova-stable-maint. my understanding has been that nova-stable-maint should include nova-core and I had thought it was included by way of the gerrit group but it's not and I also don't see in the audit log that it ever was | |
| 19:20:28 | elodilles | melwitt: yes, as far as i remember nova-core was never part of nova-stable-maint o:) but since gmaan is stable core on other projects as well so he knows the stable policy, i'm OK to add him to nova-stable-maint :) | |
| 19:22:03 | melwitt | elodilles: ok, my mistake then. thanks, I am +1 to add gmaan regardless :) | |
| 19:25:11 | elodilles | melwitt: the last change in nova-stable-maint was this as i see: https://lists.openstack.org/archives/list/openstack-discuss@lists.openstack.org/thread/VTXBEKCY3JH7FARUGHYHNCQPLHNJRGTV/#HDH4D62XZCZ24UGKRLNNKC7VW45A3SUN | |
| 19:26:33 | elodilles | Uggla: since you are PTL fyi ^^^ o:) | |
| 19:26:47 | melwitt | ah ok, thanks. I had forgot the email | |
| 20:11:42 | gmaan | yeah, I remember discussion about including nova-core group in nova-stable-maint but do not know the conclusion. | |
| 21:31:50 | opendevreview | Merged openstack/nova stable/2026.1: docs: Update libvirt version support matrix for 26.1 Gazpacho https://review.opendev.org/c/openstack/nova/+/980858 | |
| #openstack-nova - 2026-03-19 | |||
| 08:55:40 | opendevreview | Ahmad Hassan proposed openstack/nova-specs master: Cold VM migration using https https://review.opendev.org/c/openstack/nova-specs/+/964444 | |
| 09:45:30 | opendevreview | ribaudr proposed openstack/nova master: Add reproducer test for bug 2105896 https://review.opendev.org/c/openstack/nova/+/946945 | |
| 09:46:13 | 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 | |
| 10:08:37 | opendevreview | ribaudr proposed openstack/nova stable/2026.1: Fix Flamingo version in qemu matrix. https://review.opendev.org/c/openstack/nova/+/981085 | |
| 11:47:57 | zigo | Feedback from Gazpacho: restarting nova-compute takes now forever, even though it's doing nothing... | |
| 11:48:25 | zigo | I've set systemd's TimeOutStop to 185 as per release notes, but it doesn't seem to work as expected. | |
| 11:52:49 | zigo | even though the image really is a qcow2 and is uploaded as such without mistake. | |
| 11:52:49 | zigo | nova.exception.ImageUnacceptable: Image 1f9640fd-75ff-4bee-87b0-e8f0c5c57ec2 is unacceptable: Image is not raw format | |
| 11:52:49 | zigo | Also, when spawning my first Gazpacho VM, I'm getting: | |
| 11:54:00 | zigo | but that's not helping. | |
| 11:54:00 | zigo | disable_deep_image_inspection=true | |
| 11:54:00 | zigo | I tried: | |
| 11:55:43 | zigo | when really, the image is in Glance, and can be downloaded. | |
| 11:55:43 | zigo | then it fails later on with: 'No image locations are accessible' | |
| 11:55:43 | zigo | if image_meta.get('disk_format') not in ['raw', 'iso']: | |
| 11:55:43 | zigo | If I add 'qcow2' to the line: | |
| 11:56:50 | zigo | Ah no, checksum missmatch when doing "image save" ... | |
| 12:44:33 | zigo | Weird, whatever the image I upload, I always get the same os_hash_value ... | |
| 15:03:20 | 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 | |
| 15:03:21 | 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 | |
| 15:25:53 | dansmith | Uggla: quick question on the second one | |
| 16:27:11 | opendevreview | Stephen Finucane proposed openstack/nova master: trivial: Clean manager.Manager, service.Service signatures https://review.opendev.org/c/openstack/nova/+/764806 | |
| 17:05:57 | dansmith | sean-k-mooney: did you just comment on PS1 of Uggla's patch but -1 the current PS4? | |
| 17:07:36 | sean-k-mooney | dansmith: im review the patch again. i pushed the comment i had form last week | |
| 17:07:44 | sean-k-mooney | now im doing the full review | |
| 17:08:18 | sean-k-mooney | most of them have been adressed althoguht he commit message is slightly incorrect | |
| 17:08:28 | sean-k-mooney | the repoducer tests look mostly fine to me however | |
| 17:12:18 | dansmith | okay | |
| 17:13:46 | sean-k-mooney | i commented on ps1 but left a 0 cr on ps4 by the way i didnt actully leave a -1 since i needed to review the new reviesion . i left a +1 because the ai atributation is not quite corect | |
| 17:14:41 | sean-k-mooney | same issue on the fix patch but it looks mostly correct to me. im just reviewing the tests now | |
| 17:15:34 | dansmith | gerrit was showing your vote as -1 even on PS4 | |
| 17:15:51 | dansmith | I was just thinking maybe you had an old link and didn't realize you were reviewing the old one | |
| 17:16:05 | sean-k-mooney | that not what i see in gerrit | |
| 17:16:31 | sean-k-mooney | but i did say `-1 until this is removed` i jsut didnt click the button | |
| 17:16:51 | sean-k-mooney | i was -1 on the fix patch on patchset 1 | |
| 17:17:32 | sean-k-mooney | anyway Uggla it woudl be nice ot have a sort release note in teh fix patch. im still reviewing it but sofar it looks correct | |
| 17:17:39 | dansmith | not now, since you've +1d.. anyway, was just trying to point it out.. didn't know you were committing old review comments and were planning to circle back | |
| 17:18:15 | sean-k-mooney | Uggla: im double checkign the change you had to make to the other tests | |
| 17:20:06 | Uggla | sean-k-mooney, sure I can add a release note, no problem. | |
| 17:20:37 | sean-k-mooney | im currently thinking through thet logic of test_create_server_with_pci_dev_and_numa_fails | |
| 17:21:36 | sean-k-mooney | in your new code your bootign 2 vms the problem with that is the host shoudl now be rejected dueo to the cpu avialableity i htink | |
| 17:21:59 | sean-k-mooney | we may need to boot 2 vms then delete the first one | |
| 17:22:12 | dansmith | I had that comment and he adjusted the test | |
| 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 | |