diff options
author | Luke Meyer <lmeyer@redhat.com> | 2017-06-16 15:36:06 -0400 |
---|---|---|
committer | Luke Meyer <lmeyer@redhat.com> | 2017-07-25 13:23:58 -0400 |
commit | 3f3f87b7982b570ffb976865e3eace2d36960bd4 (patch) | |
tree | cd469bc7ca6b1047b75a69642383f6b9f7305251 /roles/openshift_health_checker/openshift_checks/logging/elasticsearch.py | |
parent | 5b0525f509906a43071f824b02ceb8d170c300ed (diff) | |
download | openshift-3f3f87b7982b570ffb976865e3eace2d36960bd4.tar.gz openshift-3f3f87b7982b570ffb976865e3eace2d36960bd4.tar.bz2 openshift-3f3f87b7982b570ffb976865e3eace2d36960bd4.tar.xz openshift-3f3f87b7982b570ffb976865e3eace2d36960bd4.zip |
openshift_checks: improve comments/names
Diffstat (limited to 'roles/openshift_health_checker/openshift_checks/logging/elasticsearch.py')
-rw-r--r-- | roles/openshift_health_checker/openshift_checks/logging/elasticsearch.py | 8 |
1 files changed, 3 insertions, 5 deletions
diff --git a/roles/openshift_health_checker/openshift_checks/logging/elasticsearch.py b/roles/openshift_health_checker/openshift_checks/logging/elasticsearch.py index 01cb35b81..d57cb9376 100644 --- a/roles/openshift_health_checker/openshift_checks/logging/elasticsearch.py +++ b/roles/openshift_health_checker/openshift_checks/logging/elasticsearch.py @@ -1,6 +1,4 @@ -""" -Module for performing checks on an Elasticsearch logging deployment -""" +"""Check for an aggregated logging Elasticsearch deployment""" import json import re @@ -10,7 +8,7 @@ from openshift_checks.logging.logging import LoggingCheck class Elasticsearch(LoggingCheck): - """Module that checks an integrated logging Elasticsearch deployment""" + """Check for an aggregated logging Elasticsearch deployment""" name = "elasticsearch" tags = ["health", "logging"] @@ -41,7 +39,7 @@ class Elasticsearch(LoggingCheck): return {"failed": False, "changed": False, "msg": 'No problems found with Elasticsearch deployment.'} def _not_running_elasticsearch_pods(self, es_pods): - """Returns: list of running pods, list of errors about non-running pods""" + """Returns: list of pods that are not running, list of errors about non-running pods""" not_running = super(Elasticsearch, self).not_running_pods(es_pods) if not_running: return not_running, [( |