diff options
author | Emma Foley <emma.l.foley@intel.com> | 2017-11-21 16:56:42 +0000 |
---|---|---|
committer | Emma Foley <emma.l.foley@intel.com> | 2018-01-23 11:02:08 +0000 |
commit | 12aaf2fd5a8511570fa20d9a5ca3643113475f24 (patch) | |
tree | a0350b5518b1b4cc7a9b3f97823bf5e5be259400 /yardstick/network_services/collector | |
parent | 36f0563490a4ee8f5b30eb7fe51a62d3b55c5b32 (diff) |
Rename ResourceProfile.check_if_sa_running()
* Update log message and rename method to make debugging easier
* Fix pep8 violations in touched files
JIRA: YARDSTICK-859
Change-Id: Id138d72cd8557ffacd5dcc81d06feeb484e13625
Signed-off-by: Emma Foley <emma.l.foley@intel.com>
Diffstat (limited to 'yardstick/network_services/collector')
-rw-r--r-- | yardstick/network_services/collector/subscriber.py | 7 |
1 files changed, 4 insertions, 3 deletions
diff --git a/yardstick/network_services/collector/subscriber.py b/yardstick/network_services/collector/subscriber.py index 4dc5a796e..7e18302eb 100644 --- a/yardstick/network_services/collector/subscriber.py +++ b/yardstick/network_services/collector/subscriber.py @@ -62,12 +62,13 @@ class Collector(object): # Result example: # {"VNF1: { "tput" : [1000, 999] }, "VNF2": { "latency": 100 }} LOG.debug("collect KPI for %s", node_name) - if resource.check_if_sa_running("collectd")[0] != 0: + if resource.check_if_system_agent_running("collectd")[0] != 0: continue try: results[node_name] = {"core": resource.amqp_collect_nfvi_kpi()} LOG.debug("%s collect KPIs %s", node_name, results[node_name]['core']) - except Exception: - LOG.exception("") + # NOTE(elfoley): catch a more specific error + except Exception as exc: # pylint: disable=broad-except + LOG.exception(exc) return results |