Browse Source

Fixes for Some oschecks

Currently, (Ceilometer, cinder and nova) checks will throw an error if
result is empty (for ex: nova flavor list) even though the api is working.
With the proposed fix it will just look for the api response and
doesn't matter about the result indeed.

Change-Id: Ib27d2856fa0f0c794e9ebb348cb84d3d839adffe
Closes-Bug: #1784650
changes/64/584464/4
Nagasai Vinaykumar Kapalavai 3 years ago
parent
commit
a2ff2ebf02
  1. 3
      monitoring-for-openstack/oschecks/ceilometer.py
  2. 3
      monitoring-for-openstack/oschecks/cinder.py
  3. 3
      monitoring-for-openstack/oschecks/nova.py

3
monitoring-for-openstack/oschecks/ceilometer.py

@ -45,9 +45,6 @@ def _check_ceilometer_api():
elapsed, meters = utils.timeit(meters_list)
if not meters:
utils.critical("Unable to contact Ceilometer API.")
if elapsed > options.critical:
utils.critical("Get meters took more than %d seconds, "
"it's too long.|response_time=%d" %

3
monitoring-for-openstack/oschecks/cinder.py

@ -50,9 +50,6 @@ def _check_cinder_api():
utils.critical(str(ex))
elapsed, quotas = utils.timeit(quotas_list)
if not quotas:
utils.critical("Unable to contact cinder API.")
if elapsed > options.critical:
utils.critical("Get quotas took more than %d seconds, "
"it's too long.|response_time=%d" %

3
monitoring-for-openstack/oschecks/nova.py

@ -47,9 +47,6 @@ def _check_nova_api():
utils.critical(str(ex))
elapsed, flavors = utils.timeit(flavors_list)
if not flavors:
utils.critical("Unable to contact nova API.")
if elapsed > options.critical:
utils.critical("Get flavors took more than %d seconds, "
"it's too long.|response_time=%d" %

Loading…
Cancel
Save