diff options
| author | Brenton Leanhardt <bleanhar@redhat.com> | 2016-01-18 15:48:36 -0500 |
|---|---|---|
| committer | Brenton Leanhardt <bleanhar@redhat.com> | 2016-01-18 15:48:36 -0500 |
| commit | d6a42ecaf39ba679a15ceb4e85832c45a8997640 (patch) | |
| tree | fd3759e9c2ed4d26dcf33491dc2750cd6f490cbc | |
| parent | 6fe8100badf6be308f89a690646811af9891633e (diff) | |
| parent | 29594da3ab999bd2f3869959d16d215468c050f5 (diff) | |
Merge pull request #1211 from abutcher/pacemaker-master-check
Improve pacemaker 'is-active' check.
| -rw-r--r-- | playbooks/common/openshift-master/restart.yml | 10 |
1 files changed, 9 insertions, 1 deletions
diff --git a/playbooks/common/openshift-master/restart.yml b/playbooks/common/openshift-master/restart.yml index d9d857b1a..052892863 100644 --- a/playbooks/common/openshift-master/restart.yml +++ b/playbooks/common/openshift-master/restart.yml @@ -68,14 +68,22 @@ - name: Determine which masters are currently active hosts: oo_masters_to_config + any_errors_fatal: true tasks: - name: Check master service status command: > systemctl is-active {{ openshift.common.service_type }}-master register: active_check_output when: openshift.master.cluster_method | default(None) == 'pacemaker' - failed_when: active_check_output.stdout not in ['active', 'inactive', 'unknown'] + failed_when: false changed_when: false + # Any master which did not report 'active' or 'inactive' is likely + # unhealthy. Other possible states are 'unknown' or 'failed'. + - fail: + msg: > + Got invalid service state from {{ openshift.common.service_type }}-master + on {{ inventory_hostname }}. Please verify pacemaker cluster. + when: openshift.master.cluster_method | default(None) == 'pacemaker' and active_check_output.stdout not in ['active', 'inactive'] - set_fact: is_active: "{{ active_check_output.stdout == 'active' }}" when: openshift.master.cluster_method | default(None) == 'pacemaker' |