projects
/
osm
/
MON.git
/ blobdiff
commit
grep
author
committer
pickaxe
?
search:
re
summary
|
shortlog
|
log
|
commit
|
commitdiff
|
tree
raw
|
inline
| side by side
Minor fixes to additional metric labels
[osm/MON.git]
/
osm_mon
/
collector
/
vnf_collectors
/
openstack.py
diff --git
a/osm_mon/collector/vnf_collectors/openstack.py
b/osm_mon/collector/vnf_collectors/openstack.py
index
58721a9
..
d1cd2a1
100644
(file)
--- a/
osm_mon/collector/vnf_collectors/openstack.py
+++ b/
osm_mon/collector/vnf_collectors/openstack.py
@@
-24,10
+24,11
@@
from enum import Enum
from typing import List
import gnocchiclient.exceptions
from typing import List
import gnocchiclient.exceptions
-from ceilometerclient.v2 import client as ceilometer_client
+from ceilometerclient import client as ceilometer_client
+from ceilometerclient.exc import HTTPException
from gnocchiclient.v1 import client as gnocchi_client
from gnocchiclient.v1 import client as gnocchi_client
-from keystoneauth1.exceptions.catalog import EndpointNotFound
from keystoneclient.v3 import client as keystone_client
from keystoneclient.v3 import client as keystone_client
+from keystoneauth1.exceptions.catalog import EndpointNotFound
from neutronclient.v2_0 import client as neutron_client
from osm_mon.collector.metric import Metric
from neutronclient.v2_0 import client as neutron_client
from osm_mon.collector.metric import Metric
@@
-80,6
+81,15
@@
class OpenstackCollector(BaseVimCollector):
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]
+ else:
+ tags['project_id'] = None
+
metrics = []
for vdur in vnfr['vdur']:
# This avoids errors when vdur records have not been completely filled
metrics = []
for vdur in vnfr['vdur']:
# This avoids errors when vdur records have not been completely filled
@@
-106,7
+116,6
@@
class OpenstackCollector(BaseVimCollector):
value = self.backend.collect_metric(metric_type, openstack_metric_name, resource_id,
interface_name)
if value is not None:
value = self.backend.collect_metric(metric_type, openstack_metric_name, resource_id,
interface_name)
if value is not None:
- tags = {}
if interface_name:
tags['interface'] = interface_name
metric = VnfMetric(nsr_id, vnf_member_index, vdur['name'], metric_name, value, tags)
if interface_name:
tags['interface'] = interface_name
metric = VnfMetric(nsr_id, vnf_member_index, vdur['name'], metric_name, value, tags)
@@
-120,7
+129,7
@@
class OpenstackCollector(BaseVimCollector):
ceilometer = CeilometerBackend(vim_account)
ceilometer.client.capabilities.get()
return ceilometer
ceilometer = CeilometerBackend(vim_account)
ceilometer.client.capabilities.get()
return ceilometer
- except
EndpointNotFound
:
+ except
(HTTPException, EndpointNotFound)
:
gnocchi = GnocchiBackend(vim_account)
gnocchi.client.metric.list(limit=1)
return gnocchi
gnocchi = GnocchiBackend(vim_account)
gnocchi.client.metric.list(limit=1)
return gnocchi
@@
-220,7
+229,7
@@
class CeilometerBackend(OpenstackBackend):
def _build_ceilometer_client(self, vim_account: dict) -> ceilometer_client.Client:
sess = OpenstackUtils.get_session(vim_account)
def _build_ceilometer_client(self, vim_account: dict) -> ceilometer_client.Client:
sess = OpenstackUtils.get_session(vim_account)
- return ceilometer_client.Client(session=sess)
+ return ceilometer_client.Client(
"2",
session=sess)
def collect_metric(self, metric_type: MetricType, metric_name: str, resource_id: str, interface_name: str):
if metric_type != MetricType.INSTANCE:
def collect_metric(self, metric_type: MetricType, metric_name: str, resource_id: str, interface_name: str):
if metric_type != MetricType.INSTANCE: