| opendevreview | Merged openstack/nova stable/2024.1: ironic: fix logging of validation errors https://review.opendev.org/c/openstack/nova/+/946802 | 02:36 |
|---|---|---|
| opendevreview | Merged openstack/nova stable/2024.2: Allow hyphen in cinder catalog_info service-type https://review.opendev.org/c/openstack/nova/+/944009 | 02:57 |
| 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 |
| 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 |
| 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 |
| 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 |
| opendevreview | Merged openstack/nova stable/2024.1: Reproduce bug 2085975 in functional https://review.opendev.org/c/openstack/nova/+/934303 | 03:09 |
| opendevreview | Merged openstack/nova stable/2024.1: Route shared storage RPC to evac dest at startup https://review.opendev.org/c/openstack/nova/+/934304 | 03:10 |
| 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:30 |
| 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 | 07:59 |
| ralonsoh | gibi, sean-k-mooney ^ this is for the next pyroute2 release | 08:00 |
| gibi | ralonsoh: I've left feedback inline | 08:18 |
| ralonsoh | gibi, thankks | 08:28 |
| opendevreview | Merged openstack/placement stable/2024.1: Factor out allocation candidate generation strategy https://review.opendev.org/c/openstack/placement/+/938943 | 08:30 |
| 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:30 |
| gibi | ralonsoh: looks good. thanks | 08:35 |
| *** Guest13394 is now known as shoffmann | 08:42 | |
| opendevreview | Merged openstack/placement stable/2024.1: Add round-robin candidate generation strategy https://review.opendev.org/c/openstack/placement/+/938944 | 09:19 |
| sean-k-mooney | ralonsoh: gibi before i merge that i have one reservation | 10:53 |
| 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:53 |
| sean-k-mooney | honestly i think thats fine btu just calling that out | 10:54 |
| sean-k-mooney | gibi: are you busy, i have been thinking about maintance mode a liitle since johnthetubaguy | 10:57 |
| sean-k-mooney | put that seed in my brain | 10:57 |
| ralonsoh | sean-k-mooney, this patch works with both versions | 11:03 |
| sean-k-mooney | oh is create jsut ignored | 11:04 |
| sean-k-mooney | on older verisons | 11:04 |
| sean-k-mooney | ah | 11:05 |
| 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:05 |
| ralonsoh | right | 11:06 |
| 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 |
| sean-k-mooney | which is what happening in this case | 11:06 |
| ralonsoh | the problem, IMO, is how the API of these objects have changed | 11:07 |
| ralonsoh | in a way that should have never happen | 11:07 |
| sean-k-mooney | ya thats unfortunetly out of our contol | 11:07 |
| sean-k-mooney | i approved the fix so it should merge soon thanks for adressing it | 11:07 |
| ralonsoh | cool thanks! | 11:08 |
| sean-k-mooney | since your here | 11:08 |
| sean-k-mooney | ill see if i can find time to rip out the ovn hack properly | 11:08 |
| sean-k-mooney | since it appear not to be needed and i might even backport that to epoxy | 11:08 |
| sean-k-mooney | im hesitnat to bring it back any futher | 11:08 |
| sean-k-mooney | because i dont really know when it was fixed | 11:09 |
| sean-k-mooney | i.e. what version of ovn/neutron | 11:09 |
| ralonsoh | let me check that | 11:09 |
| sean-k-mooney | i appares to not be needed on ubuntu noble with master (which is basicaly epoxy at this point) | 11:09 |
| ralonsoh | the problem is that 2025.1 is still testing in ubuntu 22.04 | 11:11 |
| ralonsoh | and the ovn version provided is 22.03, that doesn't have this support | 11:11 |
| ralonsoh | OVN v23.09.0 - 15 Sep 2023 | 11:13 |
| ralonsoh | - Introduce support for binding remote ports in ovn-northd if the CMS sets | 11:13 |
| ralonsoh | requested-chassis option for a remote logical switch port. | 11:13 |
| sean-k-mooney | ack that what i was afarid of | 11:14 |
| sean-k-mooney | 2025.2 is the first release to be noble+ | 11:14 |
| sean-k-mooney | ok so i can fix this on master but not older branhces | 11:14 |
| sean-k-mooney | ralonsoh: so for the release note ovn 23.09.0 is the first release to have the functionatlity we need | 11:15 |
| ralonsoh | one sec, because the PB additional_chassis support was added before | 11:16 |
| ralonsoh | yeah, in 22.09 | 11:16 |
| ralonsoh | but not provided by ubunty jammy, in any case | 11:16 |
| sean-k-mooney | ack. | 11:17 |
| sean-k-mooney | do you know what in centos 9 stream | 11:18 |
| sean-k-mooney | it wont really make a diffent to the end result since we are not offically supprot that any more | 11:18 |
| sean-k-mooney | but just wonderign where it is | 11:18 |
| 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:45 |
| opendevreview | Merged openstack/placement stable/2023.2: Factor out allocation candidate generation strategy https://review.opendev.org/c/openstack/placement/+/938946 | 11:57 |
| opendevreview | Merged openstack/placement stable/2023.2: Add round-robin candidate generation strategy https://review.opendev.org/c/openstack/placement/+/938947 | 11:57 |
| Uggla | To all sorry for the late notification we will start our latest vPTG day in ~7mn | 12:53 |
| Uggla | Topic: Support for live migration with encrypted volumes (Barbican integration) | 12:53 |
| opendevreview | Merged openstack/os-vif master: Adapt unit tests to pyroute2 0.9.1 https://review.opendev.org/c/openstack/os-vif/+/946977 | 13:04 |
| 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 | 13:25 |
| opendevreview | Andre Aranha proposed openstack/nova master: Replace paramiko with ssh-python https://review.opendev.org/c/openstack/nova/+/946922 | 16:02 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!