diff options
author | OpenShift Merge Robot <openshift-merge-robot@users.noreply.github.com> | 2017-11-06 15:04:12 -0800 |
---|---|---|
committer | GitHub <noreply@github.com> | 2017-11-06 15:04:12 -0800 |
commit | 4bf7a5b7472a48438f23542f127e8ab988d6f453 (patch) | |
tree | d65ffc98c152b3c932538f8636b103fa21865e80 /roles/nuage_master | |
parent | a62b37b9109ce17f36254b9374e7fda803198de6 (diff) | |
parent | eb9914420e8c327a998531a4bb7a6b8406b4316f (diff) | |
download | openshift-4bf7a5b7472a48438f23542f127e8ab988d6f453.tar.gz openshift-4bf7a5b7472a48438f23542f127e8ab988d6f453.tar.bz2 openshift-4bf7a5b7472a48438f23542f127e8ab988d6f453.tar.xz openshift-4bf7a5b7472a48438f23542f127e8ab988d6f453.zip |
Merge pull request #6027 from mgugino-upstream-stage/retry-restart-master-controllers
Automatic merge from submit-queue.
Retry restarting master controllers
Currently, master controller services may fail to restart
if master api services are not fully initialized.
This commit enables retry of master controllers.
Fixes: https://bugzilla.redhat.com/show_bug.cgi?id=1509837
Diffstat (limited to 'roles/nuage_master')
-rw-r--r-- | roles/nuage_master/handlers/main.yaml | 7 |
1 files changed, 6 insertions, 1 deletions
diff --git a/roles/nuage_master/handlers/main.yaml b/roles/nuage_master/handlers/main.yaml index 21da6b953..410b739e9 100644 --- a/roles/nuage_master/handlers/main.yaml +++ b/roles/nuage_master/handlers/main.yaml @@ -7,8 +7,13 @@ openshift.master.cluster_method == 'native' # TODO: need to fix up ignore_errors here +# We retry the controllers because the API may not be 100% initialized yet. - name: restart master controllers - systemd: name={{ openshift.common.service_type }}-master-controllers state=restarted + command: "systemctl restart {{ openshift.common.service_type }}-master-controllers" + retries: 3 + delay: 5 + register: result + until: result.rc == 0 when: > (openshift_master_ha | bool) and (not master_controllers_service_status_changed | default(false)) and |