diff options
author | Rodolfo Alonso Hernandez <ralonsoh@redhat.com> | 2023-02-15 00:09:32 +0100 |
---|---|---|
committer | Rodolfo Alonso <ralonsoh@redhat.com> | 2023-03-09 10:14:20 +0000 |
commit | 745497a1124d9ebf98cd29fd0819b7489a70cf6a (patch) | |
tree | bd74c4eb570a0537e48ef3fe6a3639a8a349adc2 /neutron/plugins | |
parent | 9d5d706f7b149cb8d152b0afd490464ee3d66196 (diff) | |
download | neutron-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.py | 36 |
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) |