| opendevreview | Rodolfo Alonso proposed openstack/neutron stable/2024.2: Use the `devstack-tobiko` job https://review.opendev.org/c/openstack/neutron/+/977050 | 00:05 |
|---|---|---|
| opendevreview | Merged openstack/python-neutronclient master: [fwaas] Don't ignore missing fw related resources https://review.opendev.org/c/openstack/python-neutronclient/+/978057 | 01:59 |
| opendevreview | Merged openstack/neutron master: Added ``ip_allocation`` extension support to ML2/OVN https://review.opendev.org/c/openstack/neutron/+/977961 | 02:05 |
| opendevreview | Merged x/whitebox-neutron-tempest-plugin master: [metadata-rate-limiting] Allow multiple paths https://review.opendev.org/c/x/whitebox-neutron-tempest-plugin/+/977322 | 04:08 |
| opendevreview | Merged openstack/neutron stable/2025.1: Make the ``logging_rpc`` member a class property https://review.opendev.org/c/openstack/neutron/+/977932 | 04:53 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Use ``Idl.cond_change`` to set a table condition https://review.opendev.org/c/openstack/neutron/+/978163 | 07:16 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Since OVN 20.06, config is stored in "Chassis.other_config" (2) https://review.opendev.org/c/openstack/neutron/+/978164 | 07:23 |
| lajoskatona | haleyb: let me check I forgot that I asked myself something.... | 07:26 |
| opendevreview | Rodolfo Alonso proposed x/whitebox-neutron-tempest-plugin master: Restore the Neutron API default configuration https://review.opendev.org/c/x/whitebox-neutron-tempest-plugin/+/977769 | 08:10 |
| opendevreview | Rodolfo Alonso proposed x/whitebox-neutron-tempest-plugin master: Restore the Neutron API default configuration https://review.opendev.org/c/x/whitebox-neutron-tempest-plugin/+/977769 | 08:10 |
| opendevreview | Rodolfo Alonso proposed x/whitebox-neutron-tempest-plugin master: Method to retrieve a configuration parameter from a service node https://review.opendev.org/c/x/whitebox-neutron-tempest-plugin/+/977973 | 08:11 |
| opendevreview | Rodolfo Alonso proposed x/whitebox-neutron-tempest-plugin master: Restore the Neutron API default configuration https://review.opendev.org/c/x/whitebox-neutron-tempest-plugin/+/977769 | 08:12 |
| opendevreview | Lajos Katona proposed openstack/networking-bagpipe master: CI: Fix failing jobs due to nodeset and req bump https://review.opendev.org/c/openstack/networking-bagpipe/+/977634 | 08:22 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: [OVN] Update the MAC_Binding age threshold default value to 300 https://review.opendev.org/c/openstack/neutron/+/977984 | 08:44 |
| opendevreview | Eduardo Olivares proposed x/whitebox-neutron-tempest-plugin master: DNM: just to test podified job runs properly https://review.opendev.org/c/x/whitebox-neutron-tempest-plugin/+/978174 | 09:01 |
| opendevreview | Merged x/whitebox-neutron-tempest-plugin master: Revert "Skip test_only_dropped_traffic_logged until oslo.config issue is fixed" https://review.opendev.org/c/x/whitebox-neutron-tempest-plugin/+/977779 | 10:21 |
| opendevreview | Rodolfo Alonso proposed x/whitebox-neutron-tempest-plugin master: Method to retrieve a configuration parameter from a service node https://review.opendev.org/c/x/whitebox-neutron-tempest-plugin/+/977973 | 10:34 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Improve ``check_global_dhcp_opts`` method https://review.opendev.org/c/openstack/neutron/+/978182 | 10:38 |
| opendevreview | Merged openstack/neutron stable/2024.2: Make the ``logging_rpc`` member a class property https://review.opendev.org/c/openstack/neutron/+/977933 | 10:40 |
| opendevreview | Elvira García Ruiz proposed openstack/neutron-specs master: Add spec for PVLAN semantics model support https://review.opendev.org/c/openstack/neutron-specs/+/975289 | 11:28 |
| opendevreview | Elvira García Ruiz proposed openstack/neutron-specs master: Add spec for PVLAN semantics model support https://review.opendev.org/c/openstack/neutron-specs/+/975289 | 11:31 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Add ``NetworkSegment.count_segments`` method https://review.opendev.org/c/openstack/neutron/+/978187 | 11:36 |
| opendevreview | Elvira García Ruiz proposed openstack/neutron-specs master: Add spec for PVLAN semantics model support https://review.opendev.org/c/openstack/neutron-specs/+/975289 | 11:47 |
| opendevreview | Elvira García Ruiz proposed openstack/neutron-lib master: [WIP] Add PVLAN api extension https://review.opendev.org/c/openstack/neutron-lib/+/975931 | 12:40 |
| opendevreview | Elvira García Ruiz proposed openstack/neutron-lib master: [WIP] Add PVLAN api extension https://review.opendev.org/c/openstack/neutron-lib/+/975931 | 13:24 |
| opendevreview | Slawek Kaplonski proposed openstack/tap-as-a-service master: DNM Just testing ovs job https://review.opendev.org/c/openstack/tap-as-a-service/+/978192 | 13:26 |
| opendevreview | Rodolfo Alonso proposed x/whitebox-neutron-tempest-plugin master: Method to retrieve a configuration parameter from a service node https://review.opendev.org/c/x/whitebox-neutron-tempest-plugin/+/977973 | 13:50 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: [OVN] Update the MAC_Binding age threshold default value to 300 https://review.opendev.org/c/openstack/neutron/+/977984 | 13:54 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Use ``Idl.cond_change`` to set a table condition https://review.opendev.org/c/openstack/neutron/+/978163 | 13:58 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Since OVN 20.06, config is stored in "Chassis.other_config" (2) https://review.opendev.org/c/openstack/neutron/+/978164 | 13:58 |
| haleyb | will start meeting in one second... | 14:00 |
| haleyb | #startmeeting neutron_drivers | 14:01 |
| opendevmeet | Meeting started Fri Feb 27 14:01:10 2026 UTC and is due to finish in 60 minutes. The chair is haleyb. Information about MeetBot at http://wiki.debian.org/MeetBot. | 14:01 |
| opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 14:01 |
| opendevmeet | The meeting name has been set to 'neutron_drivers' | 14:01 |
| haleyb | Ping list: ykarel, mlavalle, mtomaska, slaweq, tobias-urdin, lajoskatona, haleyb, ralonsoh | 14:01 |
| mlavalle | \o | 14:01 |
| ralonsoh | hello | 14:01 |
| haleyb | we had one item on the agenda | 14:02 |
| haleyb | #link https://bugs.launchpad.net/neutron/+bug/2142096 | 14:02 |
| ralonsoh | Thanks, yes | 14:02 |
| ralonsoh | but it seems that we don't have quorum | 14:02 |
| haleyb | right, we can wait a minute | 14:03 |
| slaweq | o/ | 14:03 |
| slaweq | sorry for being late | 14:03 |
| haleyb | i guess that's quorum | 14:04 |
| ralonsoh | Ok, so let me introduce this RFE | 14:05 |
| ralonsoh | We had some users complaining about the HA responses time | 14:05 |
| ralonsoh | when they had a failover, the LRP chassis takes 3 seconds to migrate to the new active chassis | 14:05 |
| ralonsoh | this is because of the default BDF OVN parameters | 14:05 |
| ralonsoh | this feature is providing, via Neutron config file, a way to configure these values | 14:06 |
| ralonsoh | the idea is to provide 3 pre-defined configurations (normal, aggressive, conservative) and a 4th that will allow the admin to configure the 3 parameters | 14:07 |
| ralonsoh | and that's all, nothing else | 14:07 |
| ralonsoh | Patch: | 14:07 |
| ralonsoh | https://review.opendev.org/c/openstack/neutron/+/977209 | 14:07 |
| haleyb | so if i'm understanding these values correctly it impacts leader election, i.e. how many failures it takes to decide to move the leader, correct? | 14:08 |
| ralonsoh | yes, by default is bfd-min-rx * bfd-mult = 3000 ms | 14:08 |
| ralonsoh | so OVN folks recommended these pre-sets | 14:09 |
| ralonsoh | of course, any admin can use "manual" and set any value there | 14:09 |
| haleyb | we had a recent incident were we had to tweak these values, and we actually chose much higher numbers than your patch | 14:09 |
| haleyb | but we might have chosen to make sure there were no elections for a while | 14:09 |
| haleyb | for reference we chose bfd-min-rx = bfd-min-tx = 4000 and bfd-mult = 5 | 14:10 |
| ralonsoh | that is a lot, for sure | 14:10 |
| haleyb | i am going to check with the other group here on what they finally wound up at, just for my own sanity | 14:11 |
| ralonsoh | now the conservative pre-set has a 6 seconds delay, yours is 20 seconds | 14:11 |
| ralonsoh | for sure, I think we can discuss the pre-set values in the patch | 14:11 |
| ralonsoh | but I think the idea of providing that in Neutron is good for admins | 14:11 |
| haleyb | yes, would be something we could expose | 14:12 |
| ralonsoh | would be a spec needed for this? | 14:12 |
| slaweq | I am ok with those config options to be added | 14:12 |
| ralonsoh | (IMO, I don't think is necessary a spec, but I can't vote) | 14:13 |
| slaweq | regarding spec, I think it is not needed, as those are just new config options | 14:13 |
| mlavalle | I agree with slaweq | 14:13 |
| haleyb | i don't think we need a spec either | 14:13 |
| mlavalle | besides, we have a recommendation from the ovn team | 14:13 |
| slaweq | but maybe we can have some documentation written with explanation what those options are, what would be the effect of setting higher/lower values, etc. | 14:13 |
| ralonsoh | yes, this patch needs also a documentation section | 14:14 |
| ralonsoh | I'll add it | 14:14 |
| lajoskatona | o/ | 14:14 |
| haleyb | my only question is why have the strategy part? instead of just exposing the values to set? | 14:15 |
| ralonsoh | that was a recommendation done by OVN folks to avoid weird values. Also for testing: we can ensure that our product works well with the 3 pre-sets | 14:15 |
| ralonsoh | and the manual will require a request exception to be supported | 14:16 |
| mlavalle | that makes sense | 14:16 |
| haleyb | ok, i understand the settings now. so is this changing from the OVN defaults? i'm remembering they were pretty low | 14:18 |
| ralonsoh | the "normal" preset has the default OVN values | 14:18 |
| haleyb | ok, so 1000/100/3 are the defaults, i should just look on my deployment | 14:20 |
| haleyb | i guess they are not set by default, at least in older OVN versions | 14:21 |
| haleyb | either way this is a step forward | 14:21 |
| ralonsoh | these values are hardcoded by default | 14:21 |
| ralonsoh | I'll send you the links | 14:21 |
| lajoskatona | do we need these links somewhere in the neutron docs? | 14:22 |
| ralonsoh | yes, I need to add a documentation section | 14:22 |
| haleyb | oh it's set in the xml | 14:22 |
| lajoskatona | ralonsoh: ack | 14:22 |
| haleyb | the code doesn't set them in the controller | 14:22 |
| ralonsoh | [1]https://github.com/openvswitch/ovs/blob/b43ed9036e6fb38fe09ff4746b603edaef5319da/vswitchd/vswitch.xml#L4147 | 14:23 |
| ralonsoh | [2]https://github.com/openvswitch/ovs/blob/b43ed9036e6fb38fe09ff4746b603edaef5319da/vswitchd/vswitch.xml#L4155 | 14:23 |
| ralonsoh | [3]https://github.com/openvswitch/ovs/blob/b43ed9036e6fb38fe09ff4746b603edaef5319da/vswitchd/vswitch.xml#L4236 | 14:23 |
| ralonsoh | yes, I was searching my Jira ticket | 14:23 |
| ralonsoh | I had it documented there | 14:23 |
| haleyb | ralonsoh: so that is the ovs schema, does ovn have it's own values? i'm confused | 14:26 |
| ralonsoh | that's ovs, yes, who handles the BFD checks | 14:27 |
| ralonsoh | but that impacts the OVN HA failover time | 14:27 |
| haleyb | but you set in OVN NB which is why i was confused, it's early for me to think | 14:30 |
| ralonsoh | we can discuss the location of these config parameters | 14:31 |
| ralonsoh | these are written in the nb-global register | 14:31 |
| ralonsoh | https://review.opendev.org/c/openstack/neutron/+/977209/2/neutron/plugins/ml2/drivers/ovn/mech_driver/ovsdb/maintenance.py#1130 | 14:31 |
| haleyb | ovn-nbctl set NB_Global . options:bfd-min-rx=1000 yup | 14:31 |
| haleyb | i'm looking at the source code and ovs and ovn both define them, but just my confusion | 14:34 |
| haleyb | so should we vote? | 14:34 |
| mlavalle | +1 | 14:35 |
| haleyb | +1 from me | 14:36 |
| lajoskatona | +1 | 14:36 |
| slaweq | +1 | 14:36 |
| ralonsoh | thank you all for your time | 14:37 |
| haleyb | alright, officially approved | 14:37 |
| haleyb | thanks ralonsoh for working on it | 14:37 |
| haleyb | any other on-deman topics? | 14:37 |
| haleyb | alright, thanks everyone and have a nice weekend! | 14:38 |
| haleyb | #endmeeting | 14:38 |
| opendevmeet | Meeting ended Fri Feb 27 14:38:55 2026 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 14:38 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/neutron_drivers/2026/neutron_drivers.2026-02-27-14.01.html | 14:38 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/neutron_drivers/2026/neutron_drivers.2026-02-27-14.01.txt | 14:38 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/neutron_drivers/2026/neutron_drivers.2026-02-27-14.01.log.html | 14:38 |
| ralonsoh | have a nice weekend | 14:39 |
| lajoskatona | o/ | 14:39 |
| slaweq | thx, have a nice weekend | 14:39 |
| slaweq | o/ | 14:39 |
| opendevreview | Mohammed Naser proposed openstack/neutron master: Fix OVN network log cross-project drop event attribution https://review.opendev.org/c/openstack/neutron/+/977349 | 15:03 |
| opendevreview | Jakub Libosvar proposed openstack/neutron master: bgp: Connect provider switch when created https://review.opendev.org/c/openstack/neutron/+/978155 | 15:33 |
| opendevreview | Jakub Libosvar proposed openstack/neutron master: WIP: new job tempest-multinode-with-bgp https://review.opendev.org/c/openstack/neutron/+/962188 | 15:33 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: [OVN] New configuration variables to control the OVN BFD settings https://review.opendev.org/c/openstack/neutron/+/977209 | 15:42 |
| opendevreview | Lajos Katona proposed openstack/networking-bagpipe master: CI: Fix failing jobs due to nodeset and req bump https://review.opendev.org/c/openstack/networking-bagpipe/+/977634 | 15:52 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: [OVN] New configuration variables to control the OVN BFD settings https://review.opendev.org/c/openstack/neutron/+/977209 | 15:57 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Add ``NetworkSegment.count_segments`` method https://review.opendev.org/c/openstack/neutron/+/978187 | 16:12 |
| opendevreview | Rodolfo Alonso proposed x/whitebox-neutron-tempest-plugin master: Method to retrieve a configuration parameter from a service node https://review.opendev.org/c/x/whitebox-neutron-tempest-plugin/+/977973 | 16:27 |
| opendevreview | Mohammed Naser proposed openstack/neutron master: Fix OVN network log cross-project drop event attribution https://review.opendev.org/c/openstack/neutron/+/977349 | 16:46 |
| opendevreview | Merged openstack/neutron-tempest-plugin master: Fix 2024.1 job definitions https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/978027 | 17:42 |
| opendevreview | Mohammed Naser proposed openstack/neutron master: Fix OVN network log cross-project drop event attribution https://review.opendev.org/c/openstack/neutron/+/977349 | 18:09 |
| opendevreview | Mohammed Naser proposed openstack/neutron master: Fix OVN network log cross-project drop event attribution https://review.opendev.org/c/openstack/neutron/+/977349 | 20:08 |
| opendevreview | Jakub Libosvar proposed openstack/neutron master: bgp: Fix UpdateLocalOVSEvent match https://review.opendev.org/c/openstack/neutron/+/978005 | 22:14 |
| opendevreview | Miguel Lavalle proposed openstack/neutron master: [WIP][OVN] Implement multiple segments per host for routed networks https://review.opendev.org/c/openstack/neutron/+/975111 | 23:17 |
| opendevreview | Merged openstack/neutron master: [ML2/OVN] Check for external_ids before touching routes https://review.opendev.org/c/openstack/neutron/+/975323 | 23:59 |
Generated by irclog2html.py 4.1.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!