summaryrefslogtreecommitdiff
path: root/neutron/plugins
diff options
context:
space:
mode:
authorRodolfo Alonso Hernandez <ralonsoh@redhat.com>2023-02-15 00:09:32 +0100
committerRodolfo Alonso <ralonsoh@redhat.com>2023-03-09 10:14:20 +0000
commit745497a1124d9ebf98cd29fd0819b7489a70cf6a (patch)
treebd74c4eb570a0537e48ef3fe6a3639a8a349adc2 /neutron/plugins
parent9d5d706f7b149cb8d152b0afd490464ee3d66196 (diff)
downloadneutron-745497a1124d9ebf98cd29fd0819b7489a70cf6a.tar.gz
[OVN] Remove "update_port_qos_with_external_ids_reference"
This method was planned to be removed in Bobcat. Trivial-Fix Change-Id: I87b568d5f9a699b09c00cbf5e1b0eae71ccab876
Diffstat (limited to 'neutron/plugins')
-rw-r--r--neutron/plugins/ml2/drivers/ovn/mech_driver/ovsdb/maintenance.py36
1 files changed, 0 insertions, 36 deletions
diff --git a/neutron/plugins/ml2/drivers/ovn/mech_driver/ovsdb/maintenance.py b/neutron/plugins/ml2/drivers/ovn/mech_driver/ovsdb/maintenance.py
index 7e427e272a..b05e22ab12 100644
--- a/neutron/plugins/ml2/drivers/ovn/mech_driver/ovsdb/maintenance.py
+++ b/neutron/plugins/ml2/drivers/ovn/mech_driver/ovsdb/maintenance.py
@@ -16,7 +16,6 @@
import abc
import copy
import inspect
-import re
import threading
from futurist import periodics
@@ -675,41 +674,6 @@ class DBInconsistenciesPeriodics(SchemaAwarePeriodicsBase):
txn.add(cmd)
raise periodics.NeverAgain()
- # TODO(ralonsoh): Remove this in the Z+2 cycle
- # A static spacing value is used here, but this method will only run
- # once per lock due to the use of periodics.NeverAgain().
- @periodics.periodic(spacing=600, run_immediately=True)
- def update_port_qos_with_external_ids_reference(self):
- """Update all OVN QoS registers with the port ID
-
- This method will only update the OVN QoS registers related to port QoS,
- not FIP QoS. FIP QoS have the corresponding "external_ids" reference.
- """
- if not self.has_lock:
- return
-
- regex = re.compile(
- r'(inport|outport) == \"(?P<port_id>[a-z0-9\-]{36})\"')
- cmds = []
- for ls in self._nb_idl.ls_list().execute(check_error=True):
- for qos in self._nb_idl.qos_list(ls.name).execute(
- check_error=True):
- if qos.external_ids:
- continue
- match = re.match(regex, qos.match)
- if not match:
- continue
- port_id = match.group('port_id')
- external_ids = {ovn_const.OVN_PORT_EXT_ID_KEY: port_id}
- cmds.append(self._nb_idl.db_set(
- 'QoS', qos.uuid, ('external_ids', external_ids)))
-
- if cmds:
- with self._nb_idl.transaction(check_error=True) as txn:
- for cmd in cmds:
- txn.add(cmd)
- raise periodics.NeverAgain()
-
# A static spacing value is used here, but this method will only run
# once per lock due to the use of periodics.NeverAgain().
@periodics.periodic(spacing=600, run_immediately=True)