| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2025-04-10 | |||
| 14:47:22 | artom | sean-k-mooney, before I forget, one (I think) important capability is instance resume on host reboot | |
| 14:47:47 | artom | I probably won't be around for the spec review, so I wanted to get that in writing :P | |
| 15:06:53 | gibi_ | artom: added to the etherpad at least :) | |
| 15:07:20 | artom | Oh yeah | |
| 15:07:50 | artom | Oh you did that already, thanks! | |
| 15:11:35 | opendevreview | Andre Aranha proposed openstack/nova master: Replace paramiko with ssh-python https://review.opendev.org/c/openstack/nova/+/946922 | |
| 15:13:13 | Uggla | stephenfin, gtema ping I think we will start the OpenAPI series topic soon. | |
| 15:13:47 | stephenfin | Uggla: Okay. I'm in the Cinder room for the next 30 or so, so if you could have one more topic first that would suit me (cc gtema) | |
| 15:22:14 | Uggla | stephenfin, fyi we move to the power management topic. | |
| 15:22:49 | Uggla | so It should give you enough time to finish your session and join | |
| 15:43:04 | Uggla | stephenfin, we restart with your topic at 15:50 UTC I hope you will be able to join. However gtema is in the call I guess I may speak on your behalf. | |
| 15:50:20 | Uggla | s/I/he/ | |
| 17:15:12 | opendevreview | sean mooney proposed openstack/nova master: Fix creating virtual servers with multiple security groups https://review.opendev.org/c/openstack/nova/+/946079 | |
| 17:15:13 | opendevreview | sean mooney proposed openstack/nova master: add repoducer test for bug 2105896 https://review.opendev.org/c/openstack/nova/+/946945 | |
| 17:17:30 | opendevreview | sean mooney proposed openstack/nova master: Fix creating virtual servers with multiple security groups https://review.opendev.org/c/openstack/nova/+/946079 | |
| 17:18:38 | opendevreview | sean mooney proposed openstack/nova master: Fix creating virtual servers with multiple security groups https://review.opendev.org/c/openstack/nova/+/946079 | |
| 17:38:23 | opendevreview | sean mooney proposed openstack/nova master: [DNM] test if ovn send plug time event when live migrating https://review.opendev.org/c/openstack/nova/+/946950 | |
| 18:04:23 | opendevreview | Masahito Muroi proposed openstack/nova master: Add check_instance_state validation to lock API https://review.opendev.org/c/openstack/nova/+/946223 | |
| 18:35:40 | sean-k-mooney | supamatt: by the way i updated https://review.opendev.org/c/openstack/nova/+/946079 with a repoducer. assuming that passes ci we can likely proceed with that. | |
| 18:36:06 | sean-k-mooney | supamatt: we perhaps should consider adding a release note but i think it would be ok as is too | |
| 18:37:25 | opendevreview | Merged openstack/placement stable/2024.1: Add a global limit on the number of allocation candidates https://review.opendev.org/c/openstack/placement/+/938942 | |
| #openstack-nova - 2025-04-11 | |||
| 02:36:06 | opendevreview | Merged openstack/nova stable/2024.1: ironic: fix logging of validation errors https://review.opendev.org/c/openstack/nova/+/946802 | |
| 02:57:32 | opendevreview | Merged openstack/nova stable/2024.2: Allow hyphen in cinder catalog_info service-type https://review.opendev.org/c/openstack/nova/+/944009 | |
| 03:09:26 | opendevreview | Merged openstack/nova stable/2024.2: Fix parameter order in add_instance_info_to_node https://review.opendev.org/c/openstack/nova/+/943994 | |
| 03:09:41 | opendevreview | Merged openstack/nova stable/2024.2: Revert "Test live migration between hosts with differnet cpu_shared_sets" https://review.opendev.org/c/openstack/nova/+/932379 | |
| 03:09:47 | opendevreview | Merged openstack/nova stable/2024.2: [doc]Add `socket` option to [pci]alias numa_policy https://review.opendev.org/c/openstack/nova/+/934049 | |
| 03:09:49 | opendevreview | Merged openstack/nova stable/2024.2: Revert "[libvirt] Live migration fails when config_drive_format=iso9660" https://review.opendev.org/c/openstack/nova/+/936645 | |
| 03:09:55 | opendevreview | Merged openstack/nova stable/2024.1: Reproduce bug 2085975 in functional https://review.opendev.org/c/openstack/nova/+/934303 | |
| 03:10:01 | opendevreview | Merged openstack/nova stable/2024.1: Route shared storage RPC to evac dest at startup https://review.opendev.org/c/openstack/nova/+/934304 | |
| 07:30:44 | opendevreview | Balazs Gibizer proposed openstack/nova stable/2024.1: [doc]Add `socket` option to [pci]alias numa_policy https://review.opendev.org/c/openstack/nova/+/946974 | |
| 07:59:09 | opendevreview | Rodolfo Alonso proposed openstack/os-vif master: Adapt unit tests to pyroute2 0.9.1 https://review.opendev.org/c/openstack/os-vif/+/946977 | |
| 08:00:20 | ralonsoh | gibi, sean-k-mooney ^ this is for the next pyroute2 release | |
| 08:18:30 | gibi | ralonsoh: I've left feedback inline | |
| 08:28:16 | ralonsoh | gibi, thankks | |
| 08:30:15 | opendevreview | Merged openstack/placement stable/2024.1: Factor out allocation candidate generation strategy https://review.opendev.org/c/openstack/placement/+/938943 | |
| 08:30:29 | opendevreview | Rodolfo Alonso proposed openstack/os-vif master: Adapt unit tests to pyroute2 0.9.1 https://review.opendev.org/c/openstack/os-vif/+/946977 | |
| 08:35:00 | gibi | ralonsoh: looks good. thanks | |
| 09:19:11 | opendevreview | Merged openstack/placement stable/2024.1: Add round-robin candidate generation strategy https://review.opendev.org/c/openstack/placement/+/938944 | |
| 10:53:20 | sean-k-mooney | ralonsoh: gibi before i merge that i have one reservation | |
| 10:53:52 | sean-k-mooney | when we make this change it will make older version fo pyrout2 not work with os-vif in the unit tests | |
| 10:54:03 | sean-k-mooney | honestly i think thats fine btu just calling that out | |
| 10:57:43 | sean-k-mooney | gibi: are you busy, i have been thinking about maintance mode a liitle since johnthetubaguy | |
| 10:57:53 | sean-k-mooney | put that seed in my brain | |
| 11:03:52 | ralonsoh | sean-k-mooney, this patch works with both versions | |
| 11:04:07 | sean-k-mooney | oh is create jsut ignored | |
| 11:04:14 | sean-k-mooney | on older verisons | |
| 11:05:48 | sean-k-mooney | ah | |
| 11:05:50 | sean-k-mooney | If you pass in create=True, and the attribute doesn’t exist, patch will create the attribute for you when the patched function is called, and delete it again after the patched function has exited. | |
| 11:06:14 | ralonsoh | right | |
| 11:06:40 | sean-k-mooney | i guess the impoarnt part is actuly "This is useful for writing tests against attributes that your production code creates at runtime." | |
| 11:06:47 | sean-k-mooney | which is what happening in this case | |
| 11:07:13 | ralonsoh | the problem, IMO, is how the API of these objects have changed | |
| 11:07:29 | ralonsoh | in a way that should have never happen | |
| 11:07:39 | sean-k-mooney | ya thats unfortunetly out of our contol | |
| 11:07:55 | sean-k-mooney | i approved the fix so it should merge soon thanks for adressing it | |
| 11:08:01 | ralonsoh | cool thanks! | |
| 11:08:07 | sean-k-mooney | since your here | |
| 11:08:26 | sean-k-mooney | ill see if i can find time to rip out the ovn hack properly | |
| 11:08:46 | sean-k-mooney | since it appear not to be needed and i might even backport that to epoxy | |
| 11:08:55 | sean-k-mooney | im hesitnat to bring it back any futher | |
| 11:09:02 | sean-k-mooney | because i dont really know when it was fixed | |
| 11:09:20 | sean-k-mooney | i.e. what version of ovn/neutron | |
| 11:09:38 | ralonsoh | let me check that | |
| 11:09:41 | sean-k-mooney | i appares to not be needed on ubuntu noble with master (which is basicaly epoxy at this point) | |
| 11:11:24 | ralonsoh | the problem is that 2025.1 is still testing in ubuntu 22.04 | |
| 11:11:41 | ralonsoh | and the ovn version provided is 22.03, that doesn't have this support | |
| 11:13:14 | ralonsoh | OVN v23.09.0 - 15 Sep 2023 | |
| 11:13:22 | ralonsoh | requested-chassis option for a remote logical switch port. | |
| 11:13:22 | ralonsoh | - Introduce support for binding remote ports in ovn-northd if the CMS sets | |
| 11:14:02 | sean-k-mooney | ack that what i was afarid of | |
| 11:14:19 | sean-k-mooney | 2025.2 is the first release to be noble+ | |
| 11:14:38 | sean-k-mooney | ok so i can fix this on master but not older branhces | |
| 11:15:20 | sean-k-mooney | ralonsoh: so for the release note ovn 23.09.0 is the first release to have the functionatlity we need | |
| 11:16:02 | ralonsoh | one sec, because the PB additional_chassis support was added before | |
| 11:16:18 | ralonsoh | yeah, in 22.09 | |
| 11:16:28 | ralonsoh | but not provided by ubunty jammy, in any case | |
| 11:17:43 | sean-k-mooney | ack. | |
| 11:18:02 | sean-k-mooney | do you know what in centos 9 stream | |
| 11:18:29 | sean-k-mooney | it wont really make a diffent to the end result since we are not offically supprot that any more | |
| 11:18:38 | sean-k-mooney | but just wonderign where it is | |
| 11:45:26 | opendevreview | Merged openstack/placement stable/2023.2: Add a global limit on the number of allocation candidates https://review.opendev.org/c/openstack/placement/+/938945 | |
| 11:57:42 | opendevreview | Merged openstack/placement stable/2023.2: Factor out allocation candidate generation strategy https://review.opendev.org/c/openstack/placement/+/938946 | |
| 11:57:46 | opendevreview | Merged openstack/placement stable/2023.2: Add round-robin candidate generation strategy https://review.opendev.org/c/openstack/placement/+/938947 | |
| 12:53:27 | Uggla | To all sorry for the late notification we will start our latest vPTG day in ~7mn | |
| 12:53:53 | Uggla | Topic: Support for live migration with encrypted volumes (Barbican integration) | |
| 13:04:46 | opendevreview | Merged openstack/os-vif master: Adapt unit tests to pyroute2 0.9.1 https://review.opendev.org/c/openstack/os-vif/+/946977 | |
| 13:25:05 | opendevreview | Maksim Malchuk proposed openstack/nova stable/2024.1: Allow hyphen in cinder catalog_info service-type https://review.opendev.org/c/openstack/nova/+/944010 | |
| 16:02:49 | opendevreview | Andre Aranha proposed openstack/nova master: Replace paramiko with ssh-python https://review.opendev.org/c/openstack/nova/+/946922 | |
| #openstack-nova - 2025-04-14 | |||
| 09:22:24 | opendevreview | Stefan K proposed openstack/nova-specs master: Add Cloud Hypervisor support spec https://review.opendev.org/c/openstack/nova-specs/+/945549 | |
| 09:32:56 | opendevreview | Stefan K proposed openstack/nova-specs master: Add Cloud Hypervisor support spec https://review.opendev.org/c/openstack/nova-specs/+/945549 | |
| 11:00:38 | opendevreview | Merged openstack/nova-specs master: tox: Drop envdir https://review.opendev.org/c/openstack/nova-specs/+/941184 | |
| 11:49:26 | opendevreview | Andre Aranha proposed openstack/nova master: Replace paramiko with ssh-python https://review.opendev.org/c/openstack/nova/+/946922 | |
| 12:50:12 | bbezak | Hi - I'd like to bring to nova team's attention pretty interesting (and somewhat convoluted to troubleshoot) bug - https://bugs.launchpad.net/nova/+bug/2104255. Namely stripping a switchdev capability from active SR-IOV VF on nova-compute service restart. Workaround is to restart nova-compute after VF is detach | |
| 12:53:43 | sean-k-mooney | bbezak: note that nova does not supprot VF-LAG at all :) | |
| 12:53:47 | sean-k-mooney | at least not offically | |
| 12:54:10 | sean-k-mooney | melonox never acutlly did the work to enabel it in neutorn or nova they just figured out a hack to make it work | |
| 12:55:05 | bbezak | indeed. I like that patch though - https://review.opendev.org/c/openstack/nova/+/884439. now we don't need to add switchdev by hand to binding profiles ;) | |
| 12:55:10 | bbezak | and it works very well :) | |
| 12:55:28 | sean-k-mooney | thats a diffent feature | |
| 12:55:41 | sean-k-mooney | so your using hardware offlowaded ovs now generic sriov | |