Skip to content

Commit 602c102

Browse files
ralonsohvoyageur
authored andcommitted
[OVN] Remove session check in update_network_postcommit
Since [1], when a segment is deleted because the network is before, the segment event handler method ``_handle_segment_change`` does not call ``_notify_mechanism_driver_for_segment_change`` and thus the check performed in ``OVNMechanismDriver.update_network_postcommit`` is not needed anymore. [1]https://review.opendev.org/c/openstack/neutron/+/786373 Conflicts: neutron/plugins/ml2/drivers/ovn/mech_driver/mech_driver.py Closes-Bug: #1739798 Change-Id: I4bb22a0a0a233609a4d23af55a050356049eb214 (cherry picked from commit 3202a5c)
1 parent 820b25a commit 602c102

File tree

1 file changed

+0
-5
lines changed

1 file changed

+0
-5
lines changed

neutron/plugins/ml2/drivers/ovn/mech_driver/mech_driver.py

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -47,7 +47,6 @@
4747
from neutron.common.ovn import exceptions as ovn_exceptions
4848
from neutron.common.ovn import extensions as ovn_extensions
4949
from neutron.common.ovn import utils as ovn_utils
50-
from neutron.common import utils
5150
from neutron.conf.plugins.ml2.drivers.ovn import ovn_conf
5251
from neutron.db import ovn_hash_ring_db
5352
from neutron.db import ovn_revision_numbers_db
@@ -620,10 +619,6 @@ def update_network_postcommit(self, context):
620619
network state. It is up to the mechanism driver to ignore
621620
state or state changes that it does not know or care about.
622621
"""
623-
# FIXME(lucasagomes): We can delete this conditional after
624-
# https://bugs.launchpad.net/neutron/+bug/1739798 is fixed.
625-
if utils.is_session_active(context._plugin_context.session):
626-
return
627622
self._ovn_client.update_network(
628623
context._plugin_context, context.current,
629624
original_network=context.original)

0 commit comments

Comments
 (0)