Browse Source

Fixes typo when setting facts to record whether master/node has been restarted already, to decide whether notify handler should do so or not.
Currently, this causes random SDN network setup failures as openshift-node gets restarted while the setup script is running, and the subsequent start fails to configure the SDN because it thinks it's already done.

Eric Mountain 9 years ago
parent
commit
0b61ab5559
2 changed files with 4 additions and 4 deletions
  1. 3 3
      roles/openshift_master/tasks/main.yml
  2. 1 1
      roles/openshift_node/tasks/main.yml

+ 3 - 3
roles/openshift_master/tasks/main.yml

@@ -228,7 +228,7 @@
   register: start_result
 
 - set_fact:
-    master_service_status_changed = start_result | changed
+    master_service_status_changed: start_result | changed
   when: not openshift_master_ha | bool
 
 - name: Start and enable master api
@@ -237,7 +237,7 @@
   register: start_result
 
 - set_fact:
-    master_api_service_status_changed = start_result | changed
+    master_api_service_status_changed: start_result | changed
   when: openshift_master_ha | bool and openshift.master.cluster_method == 'native'
 
 - name: Start and enable master controller
@@ -246,7 +246,7 @@
   register: start_result
 
 - set_fact:
-    master_controllers_service_status_changed = start_result | changed
+    master_controllers_service_status_changed: start_result | changed
   when: openshift_master_ha | bool and openshift.master.cluster_method == 'native'
 
 - name: Install cluster packages

+ 1 - 1
roles/openshift_node/tasks/main.yml

@@ -131,4 +131,4 @@
   register: start_result
 
 - set_fact:
-    node_service_status_changed = start_result | changed
+    node_service_status_changed: start_result | changed