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
This commit is contained in:
parent
965579723c
commit
a2ff2ebf02
|
@ -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" %
|
||||
|
|
|
@ -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" %
|
||||
|
|
|
@ -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…
Reference in New Issue