run excluders over selected set of hosts during control_plane/node upgrade

Disable/reset excluders over requested hosts

playbooks/common/openshift-cluster/config.yml

@@ -3,9 +3,15 @@
   tags:
   - always
 
-- include: disable_excluder.yml
+- name: Disable excluders
+  hosts: oo_masters_to_config:oo_nodes_to_config
   tags:
   - always
+  gather_facts: no
+  tasks:
+  - include_role:
+      name: openshift_excluder
+      tasks_from: disable
 
 - include: ../openshift-etcd/config.yml
   tags:
@@ -39,6 +45,12 @@
   tags:
   - hosted
 
-- include: reset_excluder.yml
+- name: Re-enable excluder if it was previously enabled
+  hosts: oo_masters_to_config:oo_nodes_to_config
   tags:
   - always
+  gather_facts: no
+  tasks:
+  - include_role:
+      name: openshift_excluder
+      tasks_from: enable

playbooks/common/openshift-cluster/disable_excluder.yml

@@ -1,17 +0,0 @@
----
-- name: Disable excluders
-  hosts: oo_masters_to_config:oo_nodes_to_config
-  gather_facts: no
-  tasks:
-
-  # During installation the excluders are installed with present state.
-  # So no pre-validation check here as the excluders are either to be installed (present = latest)
-  # or they are not going to be updated if already installed
-
-  # disable excluders based on their status
-  - include_role:
-      name: openshift_excluder
-      tasks_from: disable
-    vars:
-      openshift_excluder_package_state: present
-      docker_excluder_package_state: present

playbooks/common/openshift-cluster/reset_excluder.yml

@@ -1,8 +0,0 @@
----
-- name: Re-enable excluder if it was previously enabled
-  hosts: oo_masters_to_config:oo_nodes_to_config
-  gather_facts: no
-  tasks:
-  - include_role:
-      name: openshift_excluder
-      tasks_from: enable

playbooks/common/openshift-cluster/upgrades/disable_excluder.yml

@@ -1,14 +1,11 @@
 ---
-- name: Record excluder state and disable
-  hosts: oo_masters_to_config:oo_nodes_to_config
+- name: Disable excluders
+  hosts: oo_masters_to_config
   gather_facts: no
   tasks:
-  # verify the excluders can be upgraded
   - include_role:
       name: openshift_excluder
       tasks_from: verify_upgrade
-
-  # disable excluders based on their status
   - include_role:
       name: openshift_excluder
       tasks_from: disable

playbooks/common/openshift-cluster/upgrades/disable_node_excluders.yml

@@ -0,0 +1,14 @@
+---
+- name: Disable excluders
+  hosts: oo_nodes_to_config
+  gather_facts: no
+  tasks:
+  - include_role:
+      name: openshift_excluder
+      tasks_from: verify_upgrade
+  - include_role:
+      name: openshift_excluder
+      tasks_from: disable
+    vars:
+      openshift_excluder_package_state: latest
+      docker_excluder_package_state: latest

playbooks/common/openshift-cluster/upgrades/post_control_plane.yml

@@ -97,6 +97,12 @@
     - not grep_plugin_order_override | skipped
     - grep_plugin_order_override.rc == 0
 
-- include: ../reset_excluder.yml
+- name: Re-enable excluder if it was previously enabled
+  hosts: oo_masters_to_config
   tags:
   - always
+  gather_facts: no
+  tasks:
+  - include_role:
+      name: openshift_excluder
+      tasks_from: enable

playbooks/common/openshift-cluster/upgrades/upgrade_nodes.yml

@@ -47,6 +47,12 @@
     until: node_schedulable|succeeded
     when: node_unschedulable|changed
 
-- include: ../reset_excluder.yml
+- name: Re-enable excluder if it was previously enabled
+  hosts: oo_nodes_to_config
   tags:
   - always
+  gather_facts: no
+  tasks:
+  - include_role:
+      name: openshift_excluder
+      tasks_from: enable

playbooks/common/openshift-cluster/upgrades/v3_3/upgrade.yml

@@ -46,7 +46,11 @@
   tags:
   - pre_upgrade
 
-- include: ../disable_excluder.yml
+- include: ../disable_master_excluders.yml
+  tags:
+  - pre_upgrade
+
+- include: ../disable_node_excluders.yml
   tags:
   - pre_upgrade
 

playbooks/common/openshift-cluster/upgrades/v3_3/upgrade_control_plane.yml

@@ -54,7 +54,7 @@
   tags:
   - pre_upgrade
 
-- include: ../disable_excluder.yml
+- include: ../disable_master_excluders.yml
   tags:
   - pre_upgrade
 

playbooks/common/openshift-cluster/upgrades/v3_3/upgrade_nodes.yml

@@ -47,7 +47,7 @@
   tags:
   - pre_upgrade
 
-- include: ../disable_excluder.yml
+- include: ../disable_node_excluders.yml
   tags:
   - pre_upgrade
 

playbooks/common/openshift-cluster/upgrades/v3_4/upgrade.yml

@@ -46,7 +46,11 @@
   tags:
   - pre_upgrade
 
-- include: ../disable_excluder.yml
+- include: ../disable_master_excluders.yml
+  tags:
+  - pre_upgrade
+
+- include: ../disable_node_excluders.yml
   tags:
   - pre_upgrade
 

playbooks/common/openshift-cluster/upgrades/v3_4/upgrade_control_plane.yml

@@ -54,7 +54,7 @@
   tags:
   - pre_upgrade
 
-- include: ../disable_excluder.yml
+- include: ../disable_master_excluders.yml
   tags:
   - pre_upgrade
 

playbooks/common/openshift-cluster/upgrades/v3_4/upgrade_nodes.yml

@@ -47,7 +47,7 @@
   tags:
   - pre_upgrade
 
-- include: ../disable_excluder.yml
+- include: ../disable_node_excluders.yml
   tags:
   - pre_upgrade
 

playbooks/common/openshift-cluster/upgrades/v3_5/upgrade.yml

@@ -46,12 +46,14 @@
   tags:
   - pre_upgrade
 
-- include: ../disable_excluder.yml
+- include: ../disable_master_excluders.yml
+  tags:
+  - pre_upgrade
+
+- include: ../disable_node_excluders.yml
   tags:
   - pre_upgrade
 
-# Note: During upgrade the openshift excluder is not unexcluded inside the initialize_openshift_version.yml play.
-#       So it is necessary to run the play after running disable_excluder.yml.
 - include: ../../initialize_openshift_version.yml
   tags:
   - pre_upgrade

playbooks/common/openshift-cluster/upgrades/v3_5/upgrade_control_plane.yml

@@ -54,7 +54,7 @@
   tags:
   - pre_upgrade
 
-- include: ../disable_excluder.yml
+- include: ../disable_master_excluders.yml
   tags:
   - pre_upgrade
 

playbooks/common/openshift-cluster/upgrades/v3_5/upgrade_nodes.yml

@@ -47,7 +47,7 @@
   tags:
   - pre_upgrade
 
-- include: ../disable_excluder.yml
+- include: ../disable_node_excluders.yml
   tags:
   - pre_upgrade
 

playbooks/common/openshift-cluster/upgrades/v3_6/upgrade.yml

@@ -46,12 +46,14 @@
   tags:
   - pre_upgrade
 
-- include: ../disable_excluder.yml
+- include: ../disable_master_excluders.yml
+  tags:
+  - pre_upgrade
+
+- include: ../disable_node_excluders.yml
   tags:
   - pre_upgrade
 
-# Note: During upgrade the openshift excluder is not unexcluded inside the initialize_openshift_version.yml play.
-#       So it is necassary to run the play after running disable_excluder.yml.
 - include: ../../initialize_openshift_version.yml
   tags:
   - pre_upgrade

playbooks/common/openshift-cluster/upgrades/v3_6/upgrade_control_plane.yml

@@ -54,7 +54,7 @@
   tags:
   - pre_upgrade
 
-- include: ../disable_excluder.yml
+- include: ../disable_master_excluders.yml
   tags:
   - pre_upgrade
 

playbooks/common/openshift-cluster/upgrades/v3_6/upgrade_nodes.yml

@@ -47,7 +47,7 @@
   tags:
   - pre_upgrade
 
-- include: ../disable_excluder.yml
+- include: ../disable_node_excluders.yml
   tags:
   - pre_upgrade
 

playbooks/common/openshift-master/scaleup.yml

@@ -60,9 +60,15 @@
   - openshift_facts
   - openshift_docker
 
-- include: ../openshift-cluster/disable_excluder.yml
+- name: Disable excluders
+  hosts: oo_masters_to_config
   tags:
   - always
+  gather_facts: no
+  tasks:
+  - include_role:
+      name: openshift_excluder
+      tasks_from: disable
 
 - include: ../openshift-master/config.yml
 
@@ -70,6 +76,12 @@
 
 - include: ../openshift-node/config.yml
 
-- include: ../openshift-cluster/reset_excluder.yml
+- name: Re-enable excluder if it was previously enabled
+  hosts: oo_masters_to_config
   tags:
   - always
+  gather_facts: no
+  tasks:
+  - include_role:
+      name: openshift_excluder
+      tasks_from: enable

playbooks/common/openshift-node/scaleup.yml

@@ -27,12 +27,24 @@
   - openshift_facts
   - openshift_docker
 
-- include: ../openshift-cluster/disable_excluder.yml
+- name: Disable excluders
+  hosts: oo_nodes_to_config
   tags:
   - always
+  gather_facts: no
+  tasks:
+  - include_role:
+      name: openshift_excluder
+      tasks_from: disable
 
 - include: ../openshift-node/config.yml
 
-- include: ../openshift-cluster/reset_excluder.yml
+- name: Re-enable excluder if it was previously enabled
+  hosts: oo_nodes_to_config
   tags:
   - always
+  gather_facts: no
+  tasks:
+  - include_role:
+      name: openshift_excluder
+      tasks_from: enable