X-Git-Url: https://osm.etsi.org/gitweb/?a=blobdiff_plain;f=osm_mon%2Fcollector%2Fvnf_collectors%2Fopenstack.py;h=525bd00d3686d9b236383f407e6b4fcf4db0daaf;hb=refs%2Fchanges%2F03%2F10803%2F1;hp=a5c4980b6fb607489acaa03db1d827da4dfe8f74;hpb=a2eeb474200b8f9ebcaee6fa68fe52b6e1a5e337;p=osm%2FMON.git diff --git a/osm_mon/collector/vnf_collectors/openstack.py b/osm_mon/collector/vnf_collectors/openstack.py index a5c4980..525bd00 100644 --- a/osm_mon/collector/vnf_collectors/openstack.py +++ b/osm_mon/collector/vnf_collectors/openstack.py @@ -55,21 +55,22 @@ METRIC_MAPPINGS = { "cpu_utilization": "cpu", } -METRIC_MULTIPLIERS = { - "cpu": 0.0000001 -} +METRIC_MULTIPLIERS = {"cpu": 0.0000001} -METRIC_AGGREGATORS = { - "cpu": "rate:mean" -} +METRIC_AGGREGATORS = {"cpu": "rate:mean"} -INTERFACE_METRICS = ['packets_in_dropped', 'packets_out_dropped', 'packets_received', 'packets_sent'] +INTERFACE_METRICS = [ + "packets_in_dropped", + "packets_out_dropped", + "packets_received", + "packets_sent", +] class MetricType(Enum): - INSTANCE = 'instance' - INTERFACE_ALL = 'interface_all' - INTERFACE_ONE = 'interface_one' + INSTANCE = "instance" + INTERFACE_ALL = "interface_all" + INTERFACE_ONE = "interface_one" class OpenstackCollector(BaseVimCollector): @@ -83,59 +84,77 @@ class OpenstackCollector(BaseVimCollector): sess = OpenstackUtils.get_session(vim_account) return keystone_client.Client(session=sess) - def _get_resource_uuid(self, nsr_id: str, vnf_member_index: str, vdur_name: str) -> str: + def _get_resource_uuid( + self, nsr_id: str, vnf_member_index: str, vdur_name: str + ) -> str: vdur = self.common_db.get_vdur(nsr_id, vnf_member_index, vdur_name) - return vdur['vim-id'] + return vdur["vim-id"] def collect(self, vnfr: dict) -> List[Metric]: - nsr_id = vnfr['nsr-id-ref'] - vnf_member_index = vnfr['member-vnf-index-ref'] - vnfd = self.common_db.get_vnfd(vnfr['vnfd-id']) + nsr_id = vnfr["nsr-id-ref"] + vnf_member_index = vnfr["member-vnf-index-ref"] + vnfd = self.common_db.get_vnfd(vnfr["vnfd-id"]) # Populate extra tags for metrics tags = {} - tags['ns_name'] = self.common_db.get_nsr(nsr_id)['name'] - if vnfr['_admin']['projects_read']: - tags['project_id'] = vnfr['_admin']['projects_read'][0] + tags["ns_name"] = self.common_db.get_nsr(nsr_id)["name"] + if vnfr["_admin"]["projects_read"]: + tags["project_id"] = vnfr["_admin"]["projects_read"][0] else: - tags['project_id'] = '' + tags["project_id"] = "" metrics = [] - for vdur in vnfr['vdur']: + for vdur in vnfr["vdur"]: # This avoids errors when vdur records have not been completely filled - if 'name' not in vdur: + if "name" not in vdur: continue - vdu = next( - filter(lambda vdu: vdu['id'] == vdur['vdu-id-ref'], vnfd['vdu']) - ) - if 'monitoring-parameter' in vdu: - for param in vdu['monitoring-parameter']: - metric_name = param['performance-metric'] + vdu = next(filter(lambda vdu: vdu["id"] == vdur["vdu-id-ref"], vnfd["vdu"])) + if "monitoring-parameter" in vdu: + for param in vdu["monitoring-parameter"]: + metric_name = param["performance-metric"] openstack_metric_name = METRIC_MAPPINGS[metric_name] metric_type = self._get_metric_type(metric_name) try: - resource_id = self._get_resource_uuid(nsr_id, vnf_member_index, vdur['name']) + resource_id = self._get_resource_uuid( + nsr_id, vnf_member_index, vdur["name"] + ) except ValueError: log.warning( "Could not find resource_uuid for vdur %s, vnf_member_index %s, nsr_id %s. " "Was it recently deleted?", - vdur['name'], vnf_member_index, nsr_id) + vdur["name"], + vnf_member_index, + nsr_id, + ) continue try: log.info( "Collecting metric type: %s and metric_name: %s and resource_id %s and ", metric_type, metric_name, - resource_id) - value = self.backend.collect_metric(metric_type, openstack_metric_name, resource_id) + resource_id, + ) + value = self.backend.collect_metric( + metric_type, openstack_metric_name, resource_id + ) if value is not None: log.info("value: %s", value) - metric = VnfMetric(nsr_id, vnf_member_index, vdur['name'], metric_name, value, tags) + metric = VnfMetric( + nsr_id, + vnf_member_index, + vdur["name"], + metric_name, + value, + tags, + ) metrics.append(metric) else: log.info("metric value is empty") except Exception as e: - log.exception("Error collecting metric %s for vdu %s" % (metric_name, vdur['name'])) + log.exception( + "Error collecting metric %s for vdu %s" + % (metric_name, vdur["name"]) + ) log.info("Error in metric collection: %s" % e) return metrics @@ -159,12 +178,13 @@ class OpenstackCollector(BaseVimCollector): class OpenstackBackend: - def collect_metric(self, metric_type: MetricType, metric_name: str, resource_id: str): + def collect_metric( + self, metric_type: MetricType, metric_name: str, resource_id: str + ): pass class GnocchiBackend(OpenstackBackend): - def __init__(self, vim_account: dict): self.client = self._build_gnocchi_client(vim_account) self.neutron = self._build_neutron_client(vim_account) @@ -177,7 +197,9 @@ class GnocchiBackend(OpenstackBackend): sess = OpenstackUtils.get_session(vim_account) return neutron_client.Client(session=sess) - def collect_metric(self, metric_type: MetricType, metric_name: str, resource_id: str): + def collect_metric( + self, metric_type: MetricType, metric_name: str, resource_id: str + ): if metric_type == MetricType.INTERFACE_ALL: return self._collect_interface_all_metric(metric_name, resource_id) @@ -185,17 +207,19 @@ class GnocchiBackend(OpenstackBackend): return self._collect_instance_metric(metric_name, resource_id) else: - raise Exception('Unknown metric type %s' % metric_type.value) + raise Exception("Unknown metric type %s" % metric_type.value) def _collect_interface_all_metric(self, openstack_metric_name, resource_id): total_measure = None - interfaces = self.client.resource.search(resource_type='instance_network_interface', - query={'=': {'instance_id': resource_id}}) + interfaces = self.client.resource.search( + resource_type="instance_network_interface", + query={"=": {"instance_id": resource_id}}, + ) for interface in interfaces: try: - measures = self.client.metric.get_measures(openstack_metric_name, - resource_id=interface['id'], - limit=1) + measures = self.client.metric.get_measures( + openstack_metric_name, resource_id=interface["id"], limit=1 + ) if measures: if not total_measure: total_measure = 0.0 @@ -203,8 +227,12 @@ class GnocchiBackend(OpenstackBackend): except (gnocchiclient.exceptions.NotFound, TypeError) as e: # Gnocchi in some Openstack versions raise TypeError instead of NotFound - log.debug("No metric %s found for interface %s: %s", openstack_metric_name, - interface['id'], e) + log.debug( + "No metric %s found for interface %s: %s", + openstack_metric_name, + interface["id"], + e, + ) return total_measure def _collect_instance_metric(self, openstack_metric_name, resource_id): @@ -213,22 +241,36 @@ class GnocchiBackend(OpenstackBackend): aggregation = METRIC_AGGREGATORS.get(openstack_metric_name) try: - measures = self.client.metric.get_measures(openstack_metric_name, - aggregation=aggregation, - start=time.time() - 1200, - resource_id=resource_id) + measures = self.client.metric.get_measures( + openstack_metric_name, + aggregation=aggregation, + start=time.time() - 1200, + resource_id=resource_id, + ) if measures: value = measures[-1][2] - except (gnocchiclient.exceptions.NotFound, gnocchiclient.exceptions.BadRequest, TypeError) as e: + except ( + gnocchiclient.exceptions.NotFound, + gnocchiclient.exceptions.BadRequest, + TypeError, + ) as e: # CPU metric in previous Openstack versions do not support rate:mean aggregation method # Gnocchi in some Openstack versions raise TypeError instead of NotFound or BadRequest if openstack_metric_name == "cpu": - log.debug("No metric %s found for instance %s: %s", openstack_metric_name, resource_id, e) - log.info("Retrying to get metric %s for instance %s without aggregation", - openstack_metric_name, resource_id) - measures = self.client.metric.get_measures(openstack_metric_name, - resource_id=resource_id, - limit=1) + log.debug( + "No metric %s found for instance %s: %s", + openstack_metric_name, + resource_id, + e, + ) + log.info( + "Retrying to get metric %s for instance %s without aggregation", + openstack_metric_name, + resource_id, + ) + measures = self.client.metric.get_measures( + openstack_metric_name, resource_id=resource_id, limit=1 + ) else: raise e # measures[-1] is the last measure @@ -247,8 +289,12 @@ class GnocchiBackend(OpenstackBackend): if openstack_metric_name in METRIC_MULTIPLIERS: value = value * METRIC_MULTIPLIERS[openstack_metric_name] except gnocchiclient.exceptions.NotFound as e: - log.debug("No metric %s found for instance %s: %s", openstack_metric_name, resource_id, - e) + log.debug( + "No metric %s found for instance %s: %s", + openstack_metric_name, + resource_id, + e, + ) return value @@ -260,9 +306,16 @@ class CeilometerBackend(OpenstackBackend): sess = OpenstackUtils.get_session(vim_account) return ceilometer_client.Client("2", session=sess) - def collect_metric(self, metric_type: MetricType, metric_name: str, resource_id: str): + def collect_metric( + self, metric_type: MetricType, metric_name: str, resource_id: str + ): if metric_type != MetricType.INSTANCE: - raise NotImplementedError('Ceilometer backend only support instance metrics') - measures = self.client.samples.list(meter_name=metric_name, limit=1, q=[ - {'field': 'resource_id', 'op': 'eq', 'value': resource_id}]) + raise NotImplementedError( + "Ceilometer backend only support instance metrics" + ) + measures = self.client.samples.list( + meter_name=metric_name, + limit=1, + q=[{"field": "resource_id", "op": "eq", "value": resource_id}], + ) return measures[0].counter_volume if measures else None