| opendevreview | Rodolfo Alonso proposed openstack/neutron-vpnaas master: Remove dead variable TEMPLATE_PATH https://review.opendev.org/c/openstack/neutron-vpnaas/+/999857 | 06:06 |
|---|---|---|
| opendevreview | Rodolfo Alonso proposed openstack/neutron-vpnaas master: Remove dead method ``add_ipsec_connection`` https://review.opendev.org/c/openstack/neutron-vpnaas/+/999858 | 06:06 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron-vpnaas master: Remove empty OVN process subclasses https://review.opendev.org/c/openstack/neutron-vpnaas/+/999859 | 07:01 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron-vpnaas master: vpn: fix misleading ``os.path.join()`` with absolute path https://review.opendev.org/c/openstack/neutron-vpnaas/+/999861 | 07:01 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron-vpnaas master: Remove unused ``debtcollector`` and ``oslo.serialization`` deps https://review.opendev.org/c/openstack/neutron-vpnaas/+/999862 | 07:01 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron-vpnaas master: Register ``libreswan_opts`` in config sample generator https://review.opendev.org/c/openstack/neutron-vpnaas/+/999872 | 07:01 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron-vpnaas master: vpn: initialize ``sb_cfg`` key at OVN VPN agent startup https://review.opendev.org/c/openstack/neutron-vpnaas/+/999986 | 07:01 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron-lib master: Add "writeable FIP router_id" API extension definition https://review.opendev.org/c/openstack/neutron-lib/+/999816 | 07:14 |
| opendevreview | Takashi Kajinami proposed openstack/neutron-lib master: Remove support for hacking < 2.0.0 https://review.opendev.org/c/openstack/neutron-lib/+/999057 | 07:32 |
| LarsErikP | haleyb: that would be nice. thank you! From my POV it's quite important to get neutron >= 28.0.1 out there due to the bugs I mentioned there. At least in our setup, we are 100% stopped from upgrading to 2026.1 because of those ones :/ | 07:35 |
| opendevreview | Elvira García Ruiz proposed openstack/neutron master: Manage PVLAN PGs in OVN after PVLAN plugin is enabled or disabled https://review.opendev.org/c/openstack/neutron/+/998184 | 08:54 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: ovn: Use ``centralize_routing`` on LRP for tunnelled GW networks https://review.opendev.org/c/openstack/neutron/+/1000079 | 10:10 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron-vpnaas master: tests: replace ``load_tests`` with stestr regex filtering https://review.opendev.org/c/openstack/neutron-vpnaas/+/999791 | 11:05 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron-vpnaas master: ovn: Remove backwards compatibility with OVN < v20.09 https://review.opendev.org/c/openstack/neutron-vpnaas/+/1000083 | 11:13 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron-vpnaas master: rpc: defer device driver loading to ``start()`` in OVN VPN agent https://review.opendev.org/c/openstack/neutron-vpnaas/+/1000021 | 11:26 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron-vpnaas master: DNM == Add a stop method for the OVN VPN agent https://review.opendev.org/c/openstack/neutron-vpnaas/+/1000084 | 11:26 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron-vpnaas master: rpc: defer device driver loading to ``start()`` in OVN VPN agent https://review.opendev.org/c/openstack/neutron-vpnaas/+/1000021 | 11:29 |
| kevko | ralonsoh: maybe you can approve backports ? lajoskatona already gave us +2 https://review.opendev.org/c/openstack/neutron-vpnaas-dashboard/+/999948 https://review.opendev.org/c/openstack/neutron-vpnaas-dashboard/+/999949 https://review.opendev.org/c/openstack/neutron-vpnaas-dashboard/+/999950 | 12:07 |
| opendevreview | yatin proposed openstack/neutron stable/2025.1: [CI][2025.1 only] Drop devstack fips CentOS 9-stream jobs https://review.opendev.org/c/openstack/neutron/+/1000093 | 12:50 |
| ralonsoh | let me check | 12:54 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron-vpnaas master: ovn: Remove backwards compatibility with OVN < v20.09 https://review.opendev.org/c/openstack/neutron-vpnaas/+/1000083 | 12:56 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron-vpnaas master: rpc: defer device driver loading to ``start()`` in OVN VPN agent https://review.opendev.org/c/openstack/neutron-vpnaas/+/1000021 | 12:57 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron-vpnaas master: DNM == Add a stop method for the OVN VPN agent https://review.opendev.org/c/openstack/neutron-vpnaas/+/1000084 | 12:57 |
| * haleyb needs to look at drivers agenda and see if there is a topic | 13:02 | |
| kevko | ralonsoh: thanks | 13:02 |
| haleyb | as i don't see anything on the drivers agenda i'll cancel the meeting, if there is something i missed let me know | 13:03 |
| lajoskatona | haleyb: ack, thanks, let's meet on Tuesday than | 13:04 |
| ralonsoh | I think someone wanted to ask something during this time | 13:04 |
| ralonsoh | I don't recall who | 13:04 |
| ralonsoh | but this could be done out of the meeting | 13:04 |
| cardoe | You had answered my question ralonsoh | 13:04 |
| lajoskatona | ralonsoh: it was cardoe ragarding segments work | 13:04 |
| ralonsoh | ah right | 13:04 |
| cardoe | oh well yes there was slaweq's question on my physical vxlan networks | 13:05 |
| ralonsoh | he is OOO now | 13:05 |
| cardoe | wrt https://review.opendev.org/c/openstack/neutron/+/965415 would you rather I pull that into networking-baremetal? | 13:06 |
| cardoe | I'm also trying to engage with tbachman_ about the issues he's seen with HPB | 13:06 |
| ralonsoh | I'm not sure, to be honest | 13:07 |
| ralonsoh | I commented on tbachman_'s patch | 13:07 |
| ralonsoh | the code is ok but that needs more explanation and context for future readers | 13:07 |
| cardoe | Since he's maintaining networking-cisco to do the exact same thing that TheJulia and I are doing with networking-baremetal and networking-generic-switch | 13:07 |
| * tbachman_ waves | 13:07 | |
| cardoe | ralonsoh: I want to put type annotations on those objects in neutron-lib but we're very ambiguous about what some of those fields are. | 13:08 |
| ralonsoh | which ones, for example? | 13:08 |
| cardoe | like tbachman_'s changing from a network object to a networkcontext object but then a number of places in the neutron code also put just a plain dict in that field with some of the fields from a neutron object | 13:08 |
| tbachman_ | ralonsoh: I updated the patch with a comment, and will update both the patch and the bug as per your request. Thanks for looking at it! | 13:08 |
| lajoskatona | I suppose on some those there 10+years of dust so hard to see what's under :-) | 13:09 |
| ralonsoh | perfect, I'll check it later | 13:09 |
| cardoe | ralonsoh: I'd like to annotate the NetworkContext, PortContext, etc... basically all the Context objects. | 13:09 |
| cardoe | Internally for us I've written a detailed ML2 mechanism API document. It's got a lot of references to our ML2 mechanisms. But at some point when I have free time (ha ha) I was going to strip out all the internal bits and try to submit that. | 13:11 |
| ralonsoh | any documentation or annotations improvement is welcome | 13:11 |
| cardoe | I suspect there will be a number of revisions cause there's a bit of ambiguity that I've found and I've just make a guess or a decision that path X is the right way. But that might not be the intention of the overall neutron community. | 13:12 |
| cardoe | The only question I've got is for the underlay / baremetal VXLAN ML2 type spec. | 13:13 |
| cardoe | I'm wondering which way you'd like to see that. | 13:13 |
| cardoe | Cause I use the VXLAN type today just fine. | 13:13 |
| opendevreview | Merged openstack/neutron-vpnaas-dashboard stable/2026.1: Fix KeyError when loading VPN and IPsec connection edit forms https://review.opendev.org/c/openstack/neutron-vpnaas-dashboard/+/999948 | 13:14 |
| opendevreview | Merged openstack/neutron-vpnaas-dashboard stable/2026.1: Send flat update bodies to openstacksdk https://review.opendev.org/c/openstack/neutron-vpnaas-dashboard/+/999949 | 13:14 |
| opendevreview | Merged openstack/neutron-vpnaas-dashboard stable/2026.1: Fix router name and connections expansion in VPN services table https://review.opendev.org/c/openstack/neutron-vpnaas-dashboard/+/999950 | 13:14 |
| cardoe | I've even plugged up geneve and vxlan in KVM to a physical one. | 13:14 |
| ralonsoh | I have zero knowledge in this area | 13:14 |
| cardoe | So the only reason for my spec is that the assumption around the vxlan type is that it's a network that's equally available everywhere. | 13:14 |
| cardoe | But when it comes to the physical world, it's not actually available everywhere. | 13:15 |
| cardoe | tbachman_: I'm not sure if you're stitching multiple fabrics together? | 13:16 |
| cardoe | Someone else in the past 6 or so months posted on the ML about multiple fabrics as well. | 13:16 |
| cardoe | Anyway, I guess if you're not familiar ralonsoh a better plan from me would be to write up a problem statement. | 13:17 |
| ralonsoh | ok | 13:19 |
| cardoe | Another thing we're working on is policy work with neutron. We're going to look at introducing system scope in our deployments. We're also going to play with the CLI behavior to scope you to project. I'm not sure how welcome upstream those changes will be. | 13:19 |
| cardoe | Has anyone heard from OVH about their VPC object in neutron? Since that seems like entirely centered around ML2 mechanism changes. | 13:20 |
| ralonsoh | This decision doesn't belong to the Neutron project only. The policy definitions, in particular the scopes, belong to the TC | 13:20 |
| ralonsoh | this is openstack wide decision | 13:20 |
| tbachman_ | cardoe: multiple fbrics? | 13:21 |
| cardoe | ralonsoh: oh I'm not changing policy definitions. But yes the TC would be involved in any of the major plumbing work. I'm the TC member that's focused on this part. I've recently posted along with gmaan about working on a system reader/audit role working in all the services. | 13:25 |
| ralonsoh | ok | 13:26 |
| cardoe | ralonsoh, tbachman_: the closest thing I've got to a description of multiple fabrics is https://lists.openstack.org/archives/list/openstack-discuss@lists.openstack.org/message/GIDECROTNN2SOF3QXIDKYNJNLDZJRKV6/ | 13:26 |
| cardoe | But I'll work up a better write up. | 13:26 |
| * tbachman_ goes to read (thanks for the link!) | 13:27 | |
| opendevreview | Rodolfo Alonso proposed openstack/neutron-vpnaas master: Add LibreSwan functional tests https://review.opendev.org/c/openstack/neutron-vpnaas/+/999606 | 13:34 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron-vpnaas master: driver: remove OpenSwan driver implementation https://review.opendev.org/c/openstack/neutron-vpnaas/+/999832 | 13:34 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron-vpnaas master: Remove dead variable TEMPLATE_PATH https://review.opendev.org/c/openstack/neutron-vpnaas/+/999857 | 13:34 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron-vpnaas master: Remove dead method ``add_ipsec_connection`` https://review.opendev.org/c/openstack/neutron-vpnaas/+/999858 | 13:34 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron-vpnaas master: Remove empty OVN process subclasses https://review.opendev.org/c/openstack/neutron-vpnaas/+/999859 | 13:34 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron-vpnaas master: vpn: fix misleading ``os.path.join()`` with absolute path https://review.opendev.org/c/openstack/neutron-vpnaas/+/999861 | 13:34 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron-vpnaas master: Remove unused ``debtcollector`` and ``oslo.serialization`` deps https://review.opendev.org/c/openstack/neutron-vpnaas/+/999862 | 13:34 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron-vpnaas master: Register ``libreswan_opts`` in config sample generator https://review.opendev.org/c/openstack/neutron-vpnaas/+/999872 | 13:34 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron-vpnaas master: vpn: initialize ``sb_cfg`` key at OVN VPN agent startup https://review.opendev.org/c/openstack/neutron-vpnaas/+/999986 | 13:34 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron-vpnaas master: ovn: Remove backwards compatibility with OVN < v20.09 https://review.opendev.org/c/openstack/neutron-vpnaas/+/1000083 | 13:34 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron-vpnaas master: rpc: defer device driver loading to ``start()`` in OVN VPN agent https://review.opendev.org/c/openstack/neutron-vpnaas/+/1000021 | 13:35 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron-vpnaas master: DNM == Add a stop method for the OVN VPN agent https://review.opendev.org/c/openstack/neutron-vpnaas/+/1000084 | 13:35 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: ovn: Use ``centralize_routing`` on LRP for tunnelled GW networks https://review.opendev.org/c/openstack/neutron/+/1000079 | 13:43 |
| opendevreview | Merged openstack/neutron-lib master: Remove support for hacking < 2.0.0 https://review.opendev.org/c/openstack/neutron-lib/+/999057 | 16:28 |
| opendevreview | Merged openstack/neutron-lib master: vpn: Add ``is_filter`` to all VPN API definitions https://review.opendev.org/c/openstack/neutron-lib/+/999842 | 16:32 |
| opendevreview | Merged openstack/neutron master: Drop logic already migrated to neutron-lib https://review.opendev.org/c/openstack/neutron/+/999787 | 16:35 |
| opendevreview | Takashi Kajinami proposed openstack/neutron-lib master: Fix DriverBase.is_loaded https://review.opendev.org/c/openstack/neutron-lib/+/1000127 | 17:01 |
| opendevreview | Takashi Kajinami proposed openstack/neutron master: Use PortHardwareOffloadTypeEnumField from neutron-lib https://review.opendev.org/c/openstack/neutron/+/1000128 | 17:04 |
| opendevreview | Takashi Kajinami proposed openstack/neutron master: Use PortHardwareOffloadTypeEnumField from neutron-lib https://review.opendev.org/c/openstack/neutron/+/1000128 | 17:05 |
| -opendevstatus- NOTICE: The Gerrit service on review.opendev.openstack.org will be offline momentarily at 18:00 UTC for a patch upgrade (just under an hour from now), but should return within a few minutes. | 17:11 | |
| -opendevstatus- NOTICE: The Gerrit service on review.opendev.org is going offline momentarily for a patch upgrade, but should return within a few minutes. | 18:03 | |
| opendevreview | Merged openstack/neutron stable/2026.1: Fix accidental removal of router external gateway info https://review.opendev.org/c/openstack/neutron/+/999828 | 18:15 |
| opendevreview | Merged openstack/neutron-lib master: Add "writeable FIP router_id" API extension definition https://review.opendev.org/c/openstack/neutron-lib/+/999816 | 18:51 |
| opendevreview | Doug Goldstein proposed openstack/neutron master: ovn: Add test coverage for empty-HCG guard in link_network_ha_chassis_group https://review.opendev.org/c/openstack/neutron/+/1000150 | 20:17 |
| opendevreview | Doug Goldstein proposed openstack/neutron master: ovn: Add test coverage for empty-HCG guard in link_network_ha_chassis_group https://review.opendev.org/c/openstack/neutron/+/1000150 | 20:19 |
| opendevreview | Brian Haley proposed openstack/neutron master: Fix broad-exception-raised pylint warnings https://review.opendev.org/c/openstack/neutron/+/996576 | 20:47 |
| cardoe | haleyb: that change above brings the tests to master. I also referenced the two bugs that were fixed in master but the test coverage didn’t catch which is what my tests cover now. | 22:10 |
| opendevreview | Miro Tomaska proposed openstack/neutron master: Add upstream EVPN multinode Zuul job https://review.opendev.org/c/openstack/neutron/+/994257 | 23:45 |
Generated by irclog2html.py 4.1.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!