| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2026-03-16 | |||
| 16:56:54 | opendevreview | Max Harmathy proposed openstack/nova master: Fix crash due to access to uninitialized variable https://review.opendev.org/c/openstack/nova/+/973103 | |
| 17:02:29 | lajoskatona | o/ | |
| 17:25:45 | dansmith | melwitt: gmaan: could one of you +W this trivial fix? | |
| 17:25:46 | dansmith | https://review.opendev.org/c/openstack/nova/+/973103 | |
| 17:27:18 | nicolairuckel | elodilles: I guess your restarting skills are better than mine. Everything passed now. | |
| 17:30:20 | gmaan | dansmith: it seems cheery pick things are copy/paste in cmt msg | |
| 17:31:14 | gmaan | i can +w if author is not online now or wait to fix it? | |
| 17:33:15 | dansmith | Oh you're right, I didn't even notice | |
| 17:33:49 | opendevreview | Dan Smith proposed openstack/nova master: Fix crash due to access to uninitialized variable https://review.opendev.org/c/openstack/nova/+/973103 | |
| 17:34:29 | dansmith | gmaan: fixed ^ | |
| 17:35:56 | gmaan | +w thanks | |
| 17:36:49 | gmaan | Uggla: done, but it seems Flamingo nova version is wrong if you would like to fix that but in separate change https://review.opendev.org/c/openstack/nova/+/980703/comment/376c7418_68beba95/ | |
| 17:46:26 | opendevreview | Merged openstack/nova stable/2025.2: Preserve vTPM state between power off and power on https://review.opendev.org/c/openstack/nova/+/978908 | |
| 17:50:51 | dansmith | gmaan: thanks and thanks for noticing :) | |
| 17:54:17 | opendevreview | Merged openstack/nova-specs master: Move Gazpacho implemented specs https://review.opendev.org/c/openstack/nova-specs/+/979283 | |
| 18:03:44 | opendevreview | Nicolai Ruckel proposed openstack/nova stable/2025.2: Preserve UEFI NVRAM variable store https://review.opendev.org/c/openstack/nova/+/978766 | |
| 18:34:21 | opendevreview | Merged openstack/nova master: fix: device_by_alias should respect config type https://review.opendev.org/c/openstack/nova/+/975651 | |
| 18:41:05 | opendevreview | Merged openstack/nova master: Fix running test with py314 https://review.opendev.org/c/openstack/nova/+/980457 | |
| 21:49:47 | opendevreview | Merged openstack/nova master: docs: Update libvirt version support matrix for 26.1 Gazpacho https://review.opendev.org/c/openstack/nova/+/980703 | |
| #openstack-nova - 2026-03-17 | |||
| 03:48:51 | opendevreview | OpenStack Proposal Bot proposed openstack/nova master: Imported Translations from Zanata https://review.opendev.org/c/openstack/nova/+/980544 | |
| 08:44:35 | opendevreview | Lajos Katona proposed openstack/nova stable/2026.1: docs: Update libvirt version support matrix for 26.1 Gazpacho https://review.opendev.org/c/openstack/nova/+/980858 | |
| 09:07:03 | opendevreview | Takashi Kajinami proposed openstack/placement master: Use consistent program name https://review.opendev.org/c/openstack/placement/+/980775 | |
| 11:06:29 | opendevreview | Lajos Katona proposed openstack/nova master: Use SDK for Neutron Ports https://review.opendev.org/c/openstack/nova/+/969298 | |
| 12:27:32 | opendevreview | Thibaut Démaret proposed openstack/nova master: libvirt: add disk rotation_rate support for local disks https://review.opendev.org/c/openstack/nova/+/979693 | |
| 12:44:51 | opendevreview | Kamil Sambor proposed openstack/nova master: [vncproxy]Handle ssl.wrap_socket removal in py312 https://review.opendev.org/c/openstack/nova/+/955915 | |
| 12:44:52 | opendevreview | Kamil Sambor proposed openstack/nova master: Replace eventlet with standard threading in novncproxy https://review.opendev.org/c/openstack/nova/+/976089 | |
| 12:56:38 | opendevreview | Merged openstack/nova master: Fix crash due to access to uninitialized variable https://review.opendev.org/c/openstack/nova/+/973103 | |
| 14:23:32 | opendevreview | ribaudr proposed openstack/nova master: Fix Flamingo version in qemu matrix. https://review.opendev.org/c/openstack/nova/+/980956 | |
| 15:15:31 | opendevreview | Thibaut Démaret proposed openstack/nova master: libvirt: add disk rotation_rate support for local disks https://review.opendev.org/c/openstack/nova/+/979693 | |
| 17:28:10 | opendevreview | ribaudr proposed openstack/nova master: Reproduce OSPRH-25388: NUMA cell sorting bug for non-PCI VMs https://review.opendev.org/c/openstack/nova/+/978885 | |
| 17:28:11 | 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 | |
| 17:35:46 | Uggla | sean-k-mooney, if you have a chance to review 978885: Reproduce OSPRH-25388: NUMA cell sorting bug for non-PCI VMs | https://review.opendev.org/c/openstack/nova/+/978885 and 978954: Fix NUMA cell sorting for non-PCI VMs with pack strategy | https://review.opendev.org/c/openstack/nova/+/978954 I'll appreciate it. | |
| 17:39:29 | 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 | |
| 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 | |