openstack
/
neutron
Code Issues Proposed changes

Metering fix error that might happen when labels are removed during processing 

If a metering label is removed during a data collection and processing, and
error might happen. This patch address such situations, and creates an execution
flow that avoids such problems.

Change-Id: Iba68173c304dccf97521dcec9a87fce67e21cec2
This commit is contained in:
Rafael Weingärtner 2020-09-16 21:27:07 -03:00
parent 6599b7c786
commit 95558d0bf5
2 changed files with 11 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

11
neutron/services/metering/agents/metering_agent.py
View File

@ -134,10 +134,17 @@ class MeteringAgent(MeteringPluginRpc, manager.Manager):
if driver.BASE_LABEL_TRAFFIC_COUNTER_KEY in key:
other_ids, actual_label_id = key.split(
driver.BASE_LABEL_TRAFFIC_COUNTER_KEY)
is_label_shared = self.metering_labels[actual_label_id]['shared']
is_label_shared = None
label_name = actual_label_id
metering_label = self.metering_labels.get(actual_label_id)
if metering_label:
is_label_shared = metering_label['shared']
label_name = metering_label['name']
data['label_id'] = actual_label_id
data['label_name'] = self.metering_labels[actual_label_id]['name']
data['label_name'] = label_name
data['label_shared'] = is_label_shared
if is_label_shared:

3
neutron/services/metering/drivers/iptables/iptables_driver.py
View File

@ -309,7 +309,8 @@ class IptablesMeteringDriver(abstract_driver.MeteringAbstractDriver):
labels = router.get(constants.METERING_LABEL_KEY, [])
for label in labels:
label_id = label['id']
del rm.metering_labels[label_id]
if rm.metering_labels.get(label_id):
del rm.metering_labels[label_id]
@log_helpers.log_method_call
def add_metering_label(self, context, routers):