diff options
author | Luke Meyer <lmeyer@redhat.com> | 2017-09-20 21:35:57 -0400 |
---|---|---|
committer | Luke Meyer <lmeyer@redhat.com> | 2017-09-21 13:43:47 -0400 |
commit | 9698f76b641b7cc5964d2f4f318ea71702aa2245 (patch) | |
tree | 04f9ec59ae00ce8b79c82dd0526051ed6a0a3192 /roles/openshift_health_checker/openshift_checks/logging | |
parent | 4f5839e585a78243eae2d215176047299c52a1eb (diff) | |
download | openshift-9698f76b641b7cc5964d2f4f318ea71702aa2245.tar.gz openshift-9698f76b641b7cc5964d2f4f318ea71702aa2245.tar.bz2 openshift-9698f76b641b7cc5964d2f4f318ea71702aa2245.tar.xz openshift-9698f76b641b7cc5964d2f4f318ea71702aa2245.zip |
health checks: add diagnostics check
Also, moved is_first_master method into superclass for reuse. And look
at oo_first_master and ansible_host instead of masters and
ansible_ssh_host.
Diffstat (limited to 'roles/openshift_health_checker/openshift_checks/logging')
-rw-r--r-- | roles/openshift_health_checker/openshift_checks/logging/logging.py | 8 |
1 files changed, 0 insertions, 8 deletions
diff --git a/roles/openshift_health_checker/openshift_checks/logging/logging.py b/roles/openshift_health_checker/openshift_checks/logging/logging.py index 06bdfebf6..05ba73ca1 100644 --- a/roles/openshift_health_checker/openshift_checks/logging/logging.py +++ b/roles/openshift_health_checker/openshift_checks/logging/logging.py @@ -30,14 +30,6 @@ class LoggingCheck(OpenShiftCheck): logging_deployed = self.get_var("openshift_hosted_logging_deploy", convert=bool, default=False) return logging_deployed and super(LoggingCheck, self).is_active() and self.is_first_master() - def is_first_master(self): - """Determine if running on first master. Returns: bool""" - # Note: It would be nice to use membership in oo_first_master group, however for now it - # seems best to avoid requiring that setup and just check this is the first master. - hostname = self.get_var("ansible_ssh_host") or [None] - masters = self.get_var("groups", "masters", default=None) or [None] - return masters[0] == hostname - def run(self): return {} |