From 6a5201d4c086a98b79ef7473322b9d5e6a4180a2 Mon Sep 17 00:00:00 2001 From: Brian Haley Date: Wed, 8 Oct 2025 20:29:48 -0400 Subject: [PATCH] Fix test_add_external_port_avoid_flapping The condition check for a logical port being down should be "up != True" not "up == False". Closes-bug: #2126685 Change-Id: I1a4c4d6883802825cb8bf6b136581accb784b154 Signed-off-by: Brian Haley --- .../plugins/ml2/drivers/ovn/mech_driver/test_mech_driver.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/neutron/tests/functional/plugins/ml2/drivers/ovn/mech_driver/test_mech_driver.py b/neutron/tests/functional/plugins/ml2/drivers/ovn/mech_driver/test_mech_driver.py index c918a66170f..3c3a46b5663 100644 --- a/neutron/tests/functional/plugins/ml2/drivers/ovn/mech_driver/test_mech_driver.py +++ b/neutron/tests/functional/plugins/ml2/drivers/ovn/mech_driver/test_mech_driver.py @@ -766,7 +766,7 @@ class TestExternalPorts(base.TestOVNFunctionalBase): events = (self.ROW_UPDATE,) super().__init__( events, table, (('up', '=', True),), - old_conditions=(('up', '=', False),)) + old_conditions=(('up', '!=', True),)) def run(self, event, row, old): self.count += 1 @@ -780,7 +780,7 @@ class TestExternalPorts(base.TestOVNFunctionalBase): table = 'Logical_Switch_Port' events = (self.ROW_UPDATE,) super().__init__( - events, table, (('up', '=', False),), + events, table, (('up', '!=', True),), old_conditions=(('up', '=', True),)) def run(self, event, row, old):