Skip to content

Commit

Permalink
Merge pull request #6842 from mgugino-upstream-stage/protect-upgrade-…
Browse files Browse the repository at this point in the history
…master-version

Protect master installed version during node upgrades
  • Loading branch information
sdodson committed Jan 25, 2018
2 parents 0905e3f + d125aa5 commit eccd64e
Show file tree
Hide file tree
Showing 9 changed files with 10 additions and 1 deletion.
2 changes: 1 addition & 1 deletion playbooks/common/openshift-cluster/upgrades/pre/config.yml
Original file line number Diff line number Diff line change
Expand Up @@ -49,7 +49,7 @@
# to a more specific version, respecting openshift_image_tag and openshift_pkg_version if
# defined, and overriding the normal behavior of protecting the installed version
openshift_release: "{{ openshift_upgrade_target }}"
openshift_protect_installed_version: False
# openshift_protect_installed_version is passed n via upgrade_control_plane.yml
# l_openshift_version_set_hosts is passed via upgrade_control_plane.yml
# l_openshift_version_check_hosts is passed via upgrade_control_plane.yml

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -23,6 +23,7 @@
l_upgrade_verify_targets_hosts: "oo_masters_to_config:oo_nodes_to_upgrade"
l_upgrade_docker_target_hosts: "oo_masters_to_config:oo_nodes_to_upgrade:oo_etcd_to_config"
l_upgrade_excluder_hosts: "oo_nodes_to_config:oo_masters_to_config"
openshift_protect_installed_version: False

- import_playbook: validator.yml

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -35,6 +35,7 @@
l_upgrade_verify_targets_hosts: "oo_masters_to_config"
l_upgrade_docker_target_hosts: "oo_masters_to_config:oo_etcd_to_config"
l_upgrade_excluder_hosts: "oo_masters_to_config"
openshift_protect_installed_version: False

- import_playbook: validator.yml

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -23,6 +23,7 @@
l_upgrade_verify_targets_hosts: "oo_masters_to_config:oo_nodes_to_upgrade"
l_upgrade_docker_target_hosts: "oo_masters_to_config:oo_nodes_to_upgrade:oo_etcd_to_config"
l_upgrade_excluder_hosts: "oo_nodes_to_config:oo_masters_to_config"
openshift_protect_installed_version: False

- import_playbook: validator.yml

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -35,6 +35,7 @@
l_upgrade_verify_targets_hosts: "oo_masters_to_config"
l_upgrade_docker_target_hosts: "oo_masters_to_config:oo_etcd_to_config"
l_upgrade_excluder_hosts: "oo_masters_to_config"
openshift_protect_installed_version: False

- import_playbook: validator.yml

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -23,6 +23,7 @@
l_upgrade_verify_targets_hosts: "oo_masters_to_config:oo_nodes_to_upgrade"
l_upgrade_docker_target_hosts: "oo_masters_to_config:oo_nodes_to_upgrade:oo_etcd_to_config"
l_upgrade_excluder_hosts: "oo_nodes_to_config:oo_masters_to_config"
openshift_protect_installed_version: False

- import_playbook: validator.yml

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -36,6 +36,7 @@
l_upgrade_verify_targets_hosts: "oo_masters_to_config"
l_upgrade_docker_target_hosts: "oo_masters_to_config:oo_etcd_to_config"
l_upgrade_excluder_hosts: "oo_masters_to_config"
openshift_protect_installed_version: False

- import_playbook: validator.yml

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -20,6 +20,7 @@
l_upgrade_verify_targets_hosts: "oo_masters_to_config:oo_nodes_to_upgrade"
l_upgrade_docker_target_hosts: "oo_masters_to_config:oo_nodes_to_upgrade:oo_etcd_to_config"
l_upgrade_excluder_hosts: "oo_nodes_to_config:oo_masters_to_config"
openshift_protect_installed_version: False

- import_playbook: validator.yml

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -41,6 +41,7 @@
l_upgrade_verify_targets_hosts: "oo_masters_to_config"
l_upgrade_docker_target_hosts: "oo_masters_to_config:oo_etcd_to_config"
l_upgrade_excluder_hosts: "oo_masters_to_config"
openshift_protect_installed_version: False
when: hostvars[groups.oo_first_master.0].openshift_currently_installed_version | version_compare('3.8','<')

- name: Flag pre-upgrade checks complete for hosts without errors 3.8
Expand Down Expand Up @@ -82,6 +83,7 @@
l_upgrade_verify_targets_hosts: "oo_masters_to_config"
l_upgrade_docker_target_hosts: "oo_masters_to_config:oo_etcd_to_config"
l_upgrade_excluder_hosts: "oo_masters_to_config"
openshift_protect_installed_version: False

- name: Flag pre-upgrade checks complete for hosts without errors
hosts: oo_masters_to_config:oo_etcd_to_config
Expand Down

0 comments on commit eccd64e

Please sign in to comment.