| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2025-04-11 | |||
| 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 | - Introduce support for binding remote ports in ovn-northd if the CMS sets | |
| 11:13:22 | ralonsoh | requested-chassis option for a remote logical switch port. | |
| 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 | |
| 12:55:43 | bbezak | yes | |
| 12:56:39 | sean-k-mooney | bbezak: for what its worth the translation fo capablities to port bindign was ment to be done before hardwar offloaded ovs was merged | |
| 12:57:00 | sean-k-mooney | bbezak: it actully predates the creation of placement and got put on hold for like 6 years | |
| 12:57:15 | sean-k-mooney | bbezak: so it was always inteneded that you would not set switchdev | |
| 12:57:22 | sean-k-mooney | i.e. that nova would | |
| 12:58:47 | sean-k-mooney | bbezak: anyway looking at the bug report | |
| 12:59:10 | bbezak | thx for background info sean-k-mooney | |
| 12:59:30 | sean-k-mooney | it looks like the network capablities are lost when an instnce is "torn down" | |
| 12:59:50 | sean-k-mooney | does that mean the instnace is delete or stopped? | |
| 13:00:23 | bbezak | well. it happens on nova-compute service restart - on actively attached VFs | |
| 13:00:44 | bbezak | then when tearing vm down. the vfs are not usable | |
| 13:00:55 | sean-k-mooney | right but this is because while its attached to a vm | |
| 13:01:09 | sean-k-mooney | we cannot inspect its ethtool feature flags | |
| 13:02:02 | sean-k-mooney | i guess the problem is that we also cache this info on startup | |
| 13:02:18 | sean-k-mooney | so if the agent gets restarted whiel the vms are running | |
| 13:02:31 | bbezak | yes, it is attached to vm, then nova-compute restart is stripping feature in the db (as it is not visible on the host). and then when vm is removed the vf don't have the capability | |
| 13:02:32 | sean-k-mooney | it will no longer have the network capablities info | |
| 13:02:46 | bbezak | exactly | |
| 13:03:11 | sean-k-mooney | which is not wrong per say | |
| 13:03:22 | sean-k-mooney | it just wrong because it will never get update again when its detached | |
| 13:03:35 | bbezak | yeap | |
| 13:04:03 | sean-k-mooney | there are a couple of ways to fix that | |
| 13:04:27 | sean-k-mooney | they all have trade offs | |
| 13:04:41 | sean-k-mooney | we could clear the cache when we detach an interface | |
| 13:05:00 | sean-k-mooney | we could refuse to update the capablities if the device is not in the aviable sate | |
| 13:07:03 | bbezak | I'm wondering if we could look to existed.extra_info for capabilities | |
| 13:07:07 | bbezak | but maybe that is too narrow | |
| 13:07:08 | bbezak | thinking | |
| 13:07:10 | bbezak | :) | |
| 13:07:17 | bbezak | here https://github.com/openstack/nova/blob/1ad11b13884baeaa6ed9f8f5818f4d176f4d3134/nova/pci/manager.py#L271-L289 | |
| 13:07:56 | sean-k-mooney | you mean merge the values form the db with those form libvirt and have teh db ones take precidence while its in claimmed or allocated | |
| 13:11:28 | gibi | Uggla: here is my eventlet removal summary https://gibizer.github.io/posts/Eventlet-Removal-Flamingo-PTG/ feel free to link to it in you PTG summary mail | |
| 13:11:31 | bbezak | sth like that | |
| 13:11:51 | bbezak | I'm not taking account other pci devices that may don | |
| 13:12:09 | bbezak | I'm not taking into account other pci devices that may don't like that | |
| 13:12:13 | bbezak | I guess | |
| 13:27:58 | sean-k-mooney | bbezak: sorry on a call | |
| 13:28:10 | bbezak | no rush! | |
| 13:29:24 | sean-k-mooney | bbezak: we likely need to experiment with what is the correct approch. | |
| 13:45:16 | bbezak | yeah | |
| 14:07:13 | opendevreview | Balazs Gibizer proposed openstack/nova master: split monkey_patching form import https://review.opendev.org/c/openstack/nova/+/922425 | |
| 14:11:08 | opendevreview | sean mooney proposed openstack/nova master: Remove workaround for ovn live migration https://review.opendev.org/c/openstack/nova/+/946950 | |
| 14:28:28 | opendevreview | Pranali Deore proposed openstack/nova master: DNM: Test glance new location api https://review.opendev.org/c/openstack/nova/+/891207 | |
| 15:00:47 | opendevreview | Amit Uniyal proposed openstack/nova stable/2024.2: Libvirt: updates resource provider trait list https://review.opendev.org/c/openstack/nova/+/932522 | |
| 22:59:22 | opendevreview | Merged openstack/nova stable/2024.2: Libvirt: updates resource provider trait list https://review.opendev.org/c/openstack/nova/+/932522 | |