| opendevreview | Miguel Lavalle proposed openstack/neutron-specs master: Routed networks with multiple segments per host https://review.opendev.org/c/openstack/neutron-specs/+/966334 | 00:48 |
|---|---|---|
| *** Jeff is now known as Guest33901 | 01:05 | |
| *** Jeff is now known as Guest33909 | 03:12 | |
| opendevreview | Oleksandr Kozachenko proposed openstack/neutron master: Fix a race condition in OVN metadata port update https://review.opendev.org/c/openstack/neutron/+/962463 | 04:49 |
| opendevreview | Oleksandr Kozachenko proposed openstack/neutron master: Fix a race condition in OVN metadata port update https://review.opendev.org/c/openstack/neutron/+/962463 | 05:06 |
| *** ykarel_ is now known as ykarel | 07:24 | |
| ralonsoh | hi folks, please check https://review.opendev.org/c/openstack/releases/+/971510 | 07:48 |
| ralonsoh | slaweq, ^ | 07:49 |
| opendevreview | Merged openstack/ovn-octavia-provider master: Skip LB association to provider networks with localnet ports https://review.opendev.org/c/openstack/ovn-octavia-provider/+/967684 | 08:28 |
| opendevreview | Fernando Royo proposed openstack/ovn-octavia-provider stable/2025.2: Skip LB association to provider networks with localnet ports https://review.opendev.org/c/openstack/ovn-octavia-provider/+/971512 | 08:32 |
| opendevreview | Fernando Royo proposed openstack/ovn-octavia-provider stable/2025.1: Skip LB association to provider networks with localnet ports https://review.opendev.org/c/openstack/ovn-octavia-provider/+/971513 | 08:33 |
| opendevreview | Fernando Royo proposed openstack/ovn-octavia-provider stable/2024.2: Skip LB association to provider networks with localnet ports https://review.opendev.org/c/openstack/ovn-octavia-provider/+/971514 | 08:37 |
| opendevreview | Merged openstack/tap-as-a-service master: [OVN] Add BOTH direction to tap mirror https://review.opendev.org/c/openstack/tap-as-a-service/+/969991 | 08:53 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Add a temporary grenade job migrating to OVN agent https://review.opendev.org/c/openstack/neutron/+/955188 | 08:59 |
| opendevreview | Fernando Royo proposed openstack/ovn-octavia-provider master: Fix sync tool to pass lists to member https://review.opendev.org/c/openstack/ovn-octavia-provider/+/971518 | 10:02 |
| noonedeadpunk | Hey folks! I am trying to debug an output from neutron-ovn-db-sync-util and I have some questions about how sane some things are.... | 10:15 |
| noonedeadpunk | specifically I'm wondering about this part: https://opendev.org/openstack/neutron/src/branch/master/neutron/plugins/ml2/drivers/ovn/mech_driver/ovsdb/ovn_db_sync.py#L1558-L1563 | 10:16 |
| noonedeadpunk | the thing is, that `previous_hosts` are taken from `agents` table, which for OVN deployments is supposed to be empty at all I asume? | 10:16 |
| noonedeadpunk | And like - should even a new host be added to agents table even if migration has happened? | 10:17 |
| noonedeadpunk | Also in the script, it seems that any other condition is also covered with warning/action, which feels like - there is no rally a way not to get an alert/action there? | 10:18 |
| noonedeadpunk | as `previous_hosts - current_hosts` - smth wrong, `current_hosts - previous_hosts` - smth wrong, `current_hosts & previous_hosts` - smth wrong | 10:19 |
| noonedeadpunk | I kind of get what the intention is, I guess... to ensure that segmenthostmappings remains populated even after migration.... | 10:20 |
| noonedeadpunk | But then I'd say it's not the correct way to do it? | 10:21 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron stable/2025.2: [Docs] Add doc about Virtual IP addresses in Neutron ML2/OVN https://review.opendev.org/c/openstack/neutron/+/971520 | 10:24 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron stable/2025.1: [Docs] Add doc about Virtual IP addresses in Neutron ML2/OVN https://review.opendev.org/c/openstack/neutron/+/971521 | 10:25 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron stable/2024.2: [Docs] Add doc about Virtual IP addresses in Neutron ML2/OVN https://review.opendev.org/c/openstack/neutron/+/971523 | 10:26 |
| slaweq | ralonsoh lajoskatona ykarel hi, please check those docs backports ^^ when You will have a minute or 2. Thx in advanve | 10:26 |
| ralonsoh | sure | 10:27 |
| ralonsoh | noonedeadpunk, I'm not sure exactly about your point | 10:32 |
| ralonsoh | the previous_hosts variable is taken from the segments, not the agents | 10:33 |
| ralonsoh | actually the OVN agents are not stored in the Neutron DB but in a temporary local cache in the Neutron API | 10:33 |
| noonedeadpunk | ralonsoh: well, not really I guess? | 10:34 |
| noonedeadpunk | https://opendev.org/openstack/neutron/src/branch/master/neutron/services/segments/db.py#L286-L290 | 10:34 |
| noonedeadpunk | As in order to filter based on OVN_CONTROLLER_TYPES - it joins with agents | 10:34 |
| noonedeadpunk | which would be an empty set | 10:35 |
| noonedeadpunk | so in proper ovn setup, previous_hosts is gonna be always empty when include_agent_types is passed | 10:35 |
| noonedeadpunk | unless I miss smth obvious | 10:36 |
| ralonsoh | noonedeadpunk, so yes, if we use the Neutron DB and this join, the result will be always empty | 10:41 |
| ralonsoh | this seems to be a bug | 10:41 |
| noonedeadpunk | ++ ok, thanks for confirming I'm not crazy :) will write up one then | 10:42 |
| noonedeadpunk | I wonder what intended behavior should be though | 10:42 |
| ralonsoh | introduced here https://review.opendev.org/c/openstack/neutron/+/899130/7/neutron/services/segments/db.py | 10:42 |
| ralonsoh | are you opening this bug? | 10:42 |
| ralonsoh | if not, I'll do it | 10:42 |
| noonedeadpunk | I can open it, sure | 10:43 |
| noonedeadpunk | Already wrote half of it | 10:43 |
| ralonsoh | perfect and thanks! | 10:43 |
| noonedeadpunk | https://bugs.launchpad.net/neutron/+bug/2136864 | 10:47 |
| ralonsoh | cool | 10:47 |
| noonedeadpunk | btw I'm not sure this is regression | 10:55 |
| noonedeadpunk | As then include_agent_types won't be accepted... So it was written with this behavior in mind I'd say | 10:55 |
| ralonsoh | this method was implemented and then another patch used it in the db sync code | 10:59 |
| ralonsoh | it is not a regression but a bug since the first implementation | 10:59 |
| ralonsoh | this is the second patch, that made use of the new implementation of `get_hosts_mapped_with_segments` | 11:00 |
| ralonsoh | https://review.opendev.org/c/openstack/neutron/+/899077 | 11:00 |
| noonedeadpunk | yeah, right | 11:01 |
| opendevreview | Fernando Royo proposed openstack/ovn-octavia-provider master: Fix sync tool to pass lists to member CRUD actions https://review.opendev.org/c/openstack/ovn-octavia-provider/+/971518 | 11:11 |
| opendevreview | Fernando Royo proposed openstack/ovn-octavia-provider master: Fix sync tool to pass lists to member CRUD actions https://review.opendev.org/c/openstack/ovn-octavia-provider/+/971518 | 11:11 |
| opendevreview | Fernando Royo proposed openstack/ovn-octavia-provider master: Fix sync tool to pass lists to member CRUD actions https://review.opendev.org/c/openstack/ovn-octavia-provider/+/971518 | 11:12 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/neutron master: [ovn-db-sync] Do not expect Chassis to be in agents https://review.opendev.org/c/openstack/neutron/+/971527 | 11:41 |
| noonedeadpunk | I am not 100% sure this is the right approach, but it looks ok in my sandbox.... | 11:41 |
| opendevreview | Merged openstack/neutron stable/2025.2: Search for active port binding when updating LSP host info https://review.opendev.org/c/openstack/neutron/+/971237 | 12:17 |
| opendevreview | Merged openstack/neutron stable/2025.1: Search for active port binding when updating LSP host info https://review.opendev.org/c/openstack/neutron/+/971238 | 12:45 |
| *** haleyb is now known as haleyb|out | 14:09 | |
| opendevreview | Dmitriy Rabotyagov proposed openstack/neutron master: [ovn-db-sync] Do not expect Chassis to be in agents https://review.opendev.org/c/openstack/neutron/+/971527 | 14:22 |
| opendevreview | Fernando Royo proposed openstack/ovn-octavia-provider stable/2025.2: Fix sync tool to pass lists to member CRUD actions https://review.opendev.org/c/openstack/ovn-octavia-provider/+/971550 | 14:47 |
| opendevreview | Fernando Royo proposed openstack/ovn-octavia-provider stable/2025.1: Fix OVN DB sync when syncing an OVN LB from scratch https://review.opendev.org/c/openstack/ovn-octavia-provider/+/971551 | 14:55 |
| opendevreview | Fernando Royo proposed openstack/ovn-octavia-provider stable/2025.1: Fix sync tool to pass lists to member CRUD actions https://review.opendev.org/c/openstack/ovn-octavia-provider/+/971552 | 14:55 |
| ralonsoh | froyo, https://review.opendev.org/c/openstack/ovn-octavia-provider/+/971550: this patch needs a reference to the original patch | 14:59 |
| froyo | ralonsoh, do you mean "Related"? | 15:00 |
| ralonsoh | froyo, the "(cherry picked from commit...." reference | 15:00 |
| ralonsoh | using "-x" with the cherry-pick command | 15:01 |
| froyo | ouch, right! I change command as generat the backport previous to merge the master one... and forget the -x, also taking advantage of new patchset will change the [1] pointing to the backport to the stable branch | 15:02 |
| froyo | thx!!! good eye! | 15:02 |
| opendevreview | Fernando Royo proposed openstack/ovn-octavia-provider stable/2025.2: Fix sync tool to pass lists to member CRUD actions https://review.opendev.org/c/openstack/ovn-octavia-provider/+/971550 | 15:08 |
| opendevreview | Fernando Royo proposed openstack/ovn-octavia-provider stable/2025.1: Fix sync tool to pass lists to member CRUD actions https://review.opendev.org/c/openstack/ovn-octavia-provider/+/971552 | 15:09 |
| opendevreview | Fernando Royo proposed openstack/ovn-octavia-provider stable/2025.1: Fix OVN DB sync when syncing an OVN LB from scratch https://review.opendev.org/c/openstack/ovn-octavia-provider/+/971551 | 15:10 |
| opendevreview | Merged openstack/ovn-octavia-provider master: Fix sync tool to pass lists to member CRUD actions https://review.opendev.org/c/openstack/ovn-octavia-provider/+/971518 | 15:40 |
| opendevreview | Jakub Libosvar proposed openstack/neutron master: bgp: Improve existing server Commands https://review.opendev.org/c/openstack/neutron/+/971561 | 16:51 |
| opendevreview | Merged openstack/ovn-octavia-provider stable/2025.2: Skip LB association to provider networks with localnet ports https://review.opendev.org/c/openstack/ovn-octavia-provider/+/971512 | 17:06 |
| opendevreview | Merged openstack/ovn-octavia-provider stable/2025.1: Skip LB association to provider networks with localnet ports https://review.opendev.org/c/openstack/ovn-octavia-provider/+/971513 | 17:06 |
| opendevreview | Merged openstack/python-neutronclient stable/2025.2: v2_0: Use 'bindings' when listing port bindings https://review.opendev.org/c/openstack/python-neutronclient/+/971388 | 17:44 |
| opendevreview | Merged openstack/python-neutronclient stable/2025.1: v2_0: Use 'bindings' when listing port bindings https://review.opendev.org/c/openstack/python-neutronclient/+/971455 | 18:20 |
| opendevreview | Merged openstack/neutron stable/2024.2: Search for active port binding when updating LSP host info https://review.opendev.org/c/openstack/neutron/+/971239 | 18:23 |
| opendevreview | Merged openstack/neutron stable/2025.2: [Docs] Add doc about Virtual IP addresses in Neutron ML2/OVN https://review.opendev.org/c/openstack/neutron/+/971520 | 18:23 |
| opendevreview | Merged openstack/neutron stable/2025.1: [Docs] Add doc about Virtual IP addresses in Neutron ML2/OVN https://review.opendev.org/c/openstack/neutron/+/971521 | 18:23 |
| opendevreview | Merged openstack/neutron stable/2024.2: [Docs] Add doc about Virtual IP addresses in Neutron ML2/OVN https://review.opendev.org/c/openstack/neutron/+/971523 | 18:24 |
| opendevreview | Jakub Libosvar proposed openstack/neutron master: bgp: Improve existing server Commands https://review.opendev.org/c/openstack/neutron/+/971561 | 18:52 |
| opendevreview | Merged openstack/ovn-octavia-provider stable/2024.2: Skip LB association to provider networks with localnet ports https://review.opendev.org/c/openstack/ovn-octavia-provider/+/971514 | 19:35 |
| opendevreview | Merged openstack/neutron stable/2025.1: Check if network is external in auto-allocate code https://review.opendev.org/c/openstack/neutron/+/971161 | 22:19 |
Generated by irclog2html.py 4.0.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!