diff options
Diffstat (limited to 'playbooks')
21 files changed, 63 insertions, 59 deletions
| diff --git a/playbooks/adhoc/uninstall.yml b/playbooks/adhoc/uninstall.yml index 584117e6b..0e0e2b425 100644 --- a/playbooks/adhoc/uninstall.yml +++ b/playbooks/adhoc/uninstall.yml @@ -18,9 +18,8 @@    # Since we're not calling openshift_facts we'll do this for now    - set_fact: -      is_atomic: "{{ ostree_output.rc == 0 }}" -  - set_fact: -      is_containerized: "{{ is_atomic or containerized | default(false) | bool }}" +      openshift_is_atomic: "{{ ostree_output.rc == 0 }}" +      openshift_is_containerized: "{{ ostree_output.rc == 0 or containerized | default(false) | bool }}"  # Stop services on all hosts prior to removing files.  - hosts: nodes @@ -133,7 +132,7 @@          when: openshift_use_flannel | default(false) | bool          register: result          until: result is succeeded -      when: not is_atomic | bool +      when: not openshift_is_atomic | bool      - shell: systemctl reset-failed        changed_when: False @@ -363,7 +362,7 @@    - name: Remove packages      package: name={{ item }} state=absent -    when: not is_atomic | bool and openshift_remove_all | default(True) | bool +    when: not openshift_is_atomic | bool and openshift_remove_all | default(True) | bool      with_items:      - atomic-openshift      - atomic-openshift-clients @@ -487,14 +486,14 @@    - name: Stop additional atomic services      service: name={{ item }} state=stopped -    when: is_containerized | bool +    when: openshift_is_containerized | bool      with_items:      - etcd_container      failed_when: false    - name: Remove packages      package: name={{ item }} state=absent -    when: not is_atomic | bool and openshift_remove_all | default(True) | bool +    when: not openshift_is_atomic | bool and openshift_remove_all | default(True) | bool      with_items:      - etcd      - etcd3 @@ -554,7 +553,7 @@    - name: Remove packages      package: name={{ item }} state=absent -    when: not is_atomic | bool and openshift_remove_all | default(True) | bool +    when: not openshift_is_atomic | bool and openshift_remove_all | default(True) | bool      with_items:      - haproxy      register: result diff --git a/playbooks/common/openshift-cluster/upgrades/create_service_signer_cert.yml b/playbooks/common/openshift-cluster/upgrades/create_service_signer_cert.yml index 23cf8cf76..372a39e74 100644 --- a/playbooks/common/openshift-cluster/upgrades/create_service_signer_cert.yml +++ b/playbooks/common/openshift-cluster/upgrades/create_service_signer_cert.yml @@ -22,7 +22,7 @@    - name: Create service signer certificate      command: > -      {{ openshift.common.client_binary }} adm ca create-signer-cert +      {{ openshift_client_binary }} adm ca create-signer-cert        --cert="{{ remote_cert_create_tmpdir.stdout }}/"service-signer.crt        --key="{{ remote_cert_create_tmpdir.stdout }}/"service-signer.key        --name="{{ remote_cert_create_tmpdir.stdout }}/"openshift-service-serving-signer diff --git a/playbooks/common/openshift-cluster/upgrades/docker/docker_upgrade.yml b/playbooks/common/openshift-cluster/upgrades/docker/docker_upgrade.yml index 42cd51bd9..5b8746f2a 100644 --- a/playbooks/common/openshift-cluster/upgrades/docker/docker_upgrade.yml +++ b/playbooks/common/openshift-cluster/upgrades/docker/docker_upgrade.yml @@ -17,7 +17,7 @@    - fail:        msg: Cannot upgrade Docker on Atomic operating systems. -    when: openshift.common.is_atomic | bool +    when: openshift_is_atomic | bool    - include_role:        name: container_runtime @@ -54,7 +54,7 @@    - name: Drain Node for Kubelet upgrade      command: > -      {{ openshift.common.client_binary }} adm drain {{ openshift.node.nodename }} --config={{ openshift.common.config_base }}/master/admin.kubeconfig --force --delete-local-data --ignore-daemonsets +      {{ openshift_client_binary }} adm drain {{ openshift.node.nodename }} --config={{ openshift.common.config_base }}/master/admin.kubeconfig --force --delete-local-data --ignore-daemonsets      delegate_to: "{{ groups.oo_first_master.0 }}"      when: l_docker_upgrade is defined and l_docker_upgrade | bool and inventory_hostname in groups.oo_nodes_to_upgrade      register: l_docker_upgrade_drain_result diff --git a/playbooks/common/openshift-cluster/upgrades/docker/tasks/restart.yml b/playbooks/common/openshift-cluster/upgrades/docker/tasks/restart.yml index 385a141ea..3b47a11e0 100644 --- a/playbooks/common/openshift-cluster/upgrades/docker/tasks/restart.yml +++ b/playbooks/common/openshift-cluster/upgrades/docker/tasks/restart.yml @@ -15,7 +15,7 @@      - "{{ openshift_service_type }}-master-controllers"      - "{{ openshift_service_type }}-node"    failed_when: false -  when: openshift.common.is_containerized | bool +  when: openshift_is_containerized | bool  - name: Wait for master API to come back online    wait_for: diff --git a/playbooks/common/openshift-cluster/upgrades/docker/tasks/upgrade.yml b/playbooks/common/openshift-cluster/upgrades/docker/tasks/upgrade.yml index b5000d3a1..54eeb2ef5 100644 --- a/playbooks/common/openshift-cluster/upgrades/docker/tasks/upgrade.yml +++ b/playbooks/common/openshift-cluster/upgrades/docker/tasks/upgrade.yml @@ -10,7 +10,7 @@      - etcd_container      - openvswitch    failed_when: false -  when: openshift.common.is_containerized | bool +  when: openshift_is_containerized | bool  - name: Check Docker image count    shell: "docker images -aq | wc -l" diff --git a/playbooks/common/openshift-cluster/upgrades/pre/verify_cluster.yml b/playbooks/common/openshift-cluster/upgrades/pre/verify_cluster.yml index 3fc18c9b7..4713f8633 100644 --- a/playbooks/common/openshift-cluster/upgrades/pre/verify_cluster.yml +++ b/playbooks/common/openshift-cluster/upgrades/pre/verify_cluster.yml @@ -71,7 +71,7 @@        local_facts:          ha: "{{ groups.oo_masters_to_config | length > 1 }}" -  - when: openshift.common.is_containerized | bool +  - when: openshift_is_containerized | bool      block:      - set_fact:          master_services: diff --git a/playbooks/common/openshift-cluster/upgrades/pre/verify_upgrade_targets.yml b/playbooks/common/openshift-cluster/upgrades/pre/verify_upgrade_targets.yml index 065a9a8ab..95c37c38c 100644 --- a/playbooks/common/openshift-cluster/upgrades/pre/verify_upgrade_targets.yml +++ b/playbooks/common/openshift-cluster/upgrades/pre/verify_upgrade_targets.yml @@ -15,9 +15,9 @@      docker pull {{ openshift_cli_image }}:{{ openshift_image_tag }}    register: pull_result    changed_when: "'Downloaded newer image' in pull_result.stdout" -  when: openshift.common.is_containerized | bool +  when: openshift_is_containerized | bool -- when: not openshift.common.is_containerized | bool +- when: not openshift_is_containerized | bool    block:    - name: Check latest available OpenShift RPM version      repoquery: diff --git a/playbooks/common/openshift-cluster/upgrades/upgrade_control_plane.yml b/playbooks/common/openshift-cluster/upgrades/upgrade_control_plane.yml index 0ddccfa98..0263e721d 100644 --- a/playbooks/common/openshift-cluster/upgrades/upgrade_control_plane.yml +++ b/playbooks/common/openshift-cluster/upgrades/upgrade_control_plane.yml @@ -25,7 +25,7 @@    tasks:    - name: Upgrade all storage      command: > -      {{ openshift.common.client_binary }} adm --config={{ openshift.common.config_base }}/master/admin.kubeconfig +      {{ openshift_client_binary }} adm --config={{ openshift.common.config_base }}/master/admin.kubeconfig        migrate storage --include=* --confirm      register: l_pb_upgrade_control_plane_pre_upgrade_storage      when: openshift_upgrade_pre_storage_migration_enabled | default(true) | bool @@ -86,7 +86,7 @@    - name: Post master upgrade - Upgrade clusterpolicies storage      command: > -      {{ openshift.common.client_binary }} adm --config={{ openshift.common.config_base }}/master/admin.kubeconfig +      {{ openshift_client_binary }} adm --config={{ openshift.common.config_base }}/master/admin.kubeconfig        migrate storage --include=clusterpolicies --confirm      register: l_pb_upgrade_control_plane_post_upgrade_storage      when: @@ -133,7 +133,7 @@    tasks:    - name: Reconcile Cluster Roles      command: > -      {{ openshift.common.client_binary }} adm --config={{ openshift.common.config_base }}/master/admin.kubeconfig +      {{ openshift_client_binary }} adm --config={{ openshift.common.config_base }}/master/admin.kubeconfig        policy reconcile-cluster-roles --additive-only=true --confirm -o name      register: reconcile_cluster_role_result      when: openshift_version is version_compare('3.7','<') @@ -144,7 +144,7 @@    - name: Reconcile Cluster Role Bindings      command: > -      {{ openshift.common.client_binary }} adm --config={{ openshift.common.config_base }}/master/admin.kubeconfig +      {{ openshift_client_binary }} adm --config={{ openshift.common.config_base }}/master/admin.kubeconfig        policy reconcile-cluster-role-bindings        --exclude-groups=system:authenticated        --exclude-groups=system:authenticated:oauth @@ -160,7 +160,7 @@    - name: Reconcile Jenkins Pipeline Role Bindings      command: > -      {{ openshift.common.client_binary }} adm --config={{ openshift.common.config_base }}/master/admin.kubeconfig policy reconcile-cluster-role-bindings system:build-strategy-jenkinspipeline --confirm -o name +      {{ openshift_client_binary }} adm --config={{ openshift.common.config_base }}/master/admin.kubeconfig policy reconcile-cluster-role-bindings system:build-strategy-jenkinspipeline --confirm -o name      run_once: true      register: reconcile_jenkins_role_binding_result      changed_when: @@ -214,7 +214,7 @@    - name: Reconcile Security Context Constraints      command: > -      {{ openshift.common.client_binary }} adm policy --config={{ openshift.common.config_base }}/master/admin.kubeconfig reconcile-sccs --confirm --additive-only=true -o name +      {{ openshift_client_binary }} adm policy --config={{ openshift.common.config_base }}/master/admin.kubeconfig reconcile-sccs --confirm --additive-only=true -o name      register: reconcile_scc_result      changed_when:      - reconcile_scc_result.stdout != '' @@ -223,7 +223,7 @@    - name: Migrate storage post policy reconciliation      command: > -      {{ openshift.common.client_binary }} adm --config={{ openshift.common.config_base }}/master/admin.kubeconfig +      {{ openshift_client_binary }} adm --config={{ openshift.common.config_base }}/master/admin.kubeconfig        migrate storage --include=* --confirm      run_once: true      register: l_pb_upgrade_control_plane_post_upgrade_storage @@ -262,7 +262,7 @@    - openshift_facts    tasks:    - include_tasks: docker/tasks/upgrade.yml -    when: l_docker_upgrade is defined and l_docker_upgrade | bool and not openshift.common.is_atomic | bool +    when: l_docker_upgrade is defined and l_docker_upgrade | bool and not openshift_is_atomic | bool  - name: Drain and upgrade master nodes    hosts: oo_masters_to_config:&oo_nodes_to_upgrade @@ -291,7 +291,7 @@    - name: Drain Node for Kubelet upgrade      command: > -      {{ hostvars[groups.oo_first_master.0].openshift.common.client_binary }} adm drain {{ openshift.node.nodename | lower }} --config={{ openshift.common.config_base }}/master/admin.kubeconfig --force --delete-local-data --ignore-daemonsets +      {{ hostvars[groups.oo_first_master.0]['first_master_client_binary'] }} adm drain {{ openshift.node.nodename | lower }} --config={{ openshift.common.config_base }}/master/admin.kubeconfig --force --delete-local-data --ignore-daemonsets      delegate_to: "{{ groups.oo_first_master.0 }}"      register: l_upgrade_control_plane_drain_result      until: not (l_upgrade_control_plane_drain_result is failed) diff --git a/playbooks/common/openshift-cluster/upgrades/upgrade_nodes.yml b/playbooks/common/openshift-cluster/upgrades/upgrade_nodes.yml index 956ad0d53..ece69a3d5 100644 --- a/playbooks/common/openshift-cluster/upgrades/upgrade_nodes.yml +++ b/playbooks/common/openshift-cluster/upgrades/upgrade_nodes.yml @@ -35,7 +35,7 @@    - name: Drain Node for Kubelet upgrade      command: > -      {{ hostvars[groups.oo_first_master.0].openshift.common.client_binary }} adm drain {{ openshift.node.nodename | lower }} --config={{ openshift.common.config_base }}/master/admin.kubeconfig --force --delete-local-data --ignore-daemonsets +      {{ hostvars[groups.oo_first_master.0]['first_master_client_binary'] }} adm drain {{ openshift.node.nodename | lower }} --config={{ openshift.common.config_base }}/master/admin.kubeconfig --force --delete-local-data --ignore-daemonsets      delegate_to: "{{ groups.oo_first_master.0 }}"      register: l_upgrade_nodes_drain_result      until: not (l_upgrade_nodes_drain_result is failed) diff --git a/playbooks/common/openshift-cluster/upgrades/upgrade_scale_group.yml b/playbooks/common/openshift-cluster/upgrades/upgrade_scale_group.yml index e8c0f361a..a90082760 100644 --- a/playbooks/common/openshift-cluster/upgrades/upgrade_scale_group.yml +++ b/playbooks/common/openshift-cluster/upgrades/upgrade_scale_group.yml @@ -43,7 +43,7 @@    tasks:    - name: Drain Node for Kubelet upgrade      command: > -      {{ hostvars[groups.oo_first_master.0].openshift.common.client_binary }} adm drain {{ openshift.node.nodename | lower }} +      {{ hostvars[groups.oo_first_master.0]['first_master_client_binary'] }} adm drain {{ openshift.node.nodename | lower }}        --config={{ openshift.common.config_base }}/master/admin.kubeconfig        --force --delete-local-data --ignore-daemonsets        --timeout={{ openshift_upgrade_nodes_drain_timeout | default(0) }}s diff --git a/playbooks/common/openshift-cluster/upgrades/v3_7/validator.yml b/playbooks/common/openshift-cluster/upgrades/v3_7/validator.yml index c8c87a9c3..49e691352 100644 --- a/playbooks/common/openshift-cluster/upgrades/v3_7/validator.yml +++ b/playbooks/common/openshift-cluster/upgrades/v3_7/validator.yml @@ -14,7 +14,7 @@    # DO NOT DISABLE THIS, YOUR UPGRADE WILL FAIL IF YOU DO SO    - name: Confirm OpenShift authorization objects are in sync      command: > -      {{ openshift.common.client_binary }} adm migrate authorization +      {{ openshift_client_binary }} adm migrate authorization      when:      - openshift_currently_installed_version is version_compare('3.7','<')      - openshift_upgrade_pre_authorization_migration_enabled | default(true) | bool diff --git a/playbooks/init/facts.yml b/playbooks/init/facts.yml index 9fec95b17..ac4429b23 100644 --- a/playbooks/init/facts.yml +++ b/playbooks/init/facts.yml @@ -21,14 +21,10 @@        path: /run/ostree-booted      register: ostree_booted -  # Locally setup containerized facts for now -  - name: initialize_facts set fact l_is_atomic +  - name: initialize_facts set fact openshift_is_atomic and openshift_is_containerized      set_fact: -      l_is_atomic: "{{ ostree_booted.stat.exists }}" - -  - name: initialize_facts set fact for containerized and l_is_*_system_container -    set_fact: -      l_is_containerized: "{{ (l_is_atomic | bool) or (containerized | default(false) | bool) }}" +      openshift_is_atomic: "{{ ostree_booted.stat.exists }}" +      openshift_is_containerized: "{{ ostree_booted.stat.exists or (containerized | default(false) | bool) }}"    # TODO: Should this be moved into health checks??    # Seems as though any check that happens with a corresponding fail should move into health_checks @@ -54,7 +50,7 @@    # Seems as though any check that happens with a corresponding fail should move into health_checks    # Fail as early as possible if Atomic and old version of Docker    - when: -    - l_is_atomic | bool +    - openshift_is_atomic | bool      block:      # See https://access.redhat.com/articles/2317361 @@ -73,7 +69,7 @@          msg: Installation on Atomic Host requires Docker 1.12 or later. Please upgrade and restart the Atomic Host.    - when: -    - not l_is_atomic | bool +    - not openshift_is_atomic | bool      block:      - name: Ensure openshift-ansible installer package deps are installed        package: @@ -105,7 +101,7 @@        register: result        until: result is succeeded -  - name: Gather Cluster facts and set is_containerized if needed +  - name: Gather Cluster facts      openshift_facts:        role: common        local_facts: @@ -113,7 +109,6 @@          deployment_subtype: "{{ openshift_deployment_subtype | default(None) }}"          hostname: "{{ openshift_hostname | default(None) }}"          ip: "{{ openshift_ip | default(None) }}" -        is_containerized: "{{ l_is_containerized | default(None) }}"          public_hostname: "{{ openshift_public_hostname | default(None) }}"          public_ip: "{{ openshift_public_ip | default(None) }}"          portal_net: "{{ openshift_portal_net | default(openshift_master_portal_net) | default(None) }}" @@ -145,3 +140,13 @@      set_fact:        repoquery_cmd: "{{ 'dnf repoquery --latest-limit 1 -d 0' if ansible_pkg_mgr == 'dnf' else 'repoquery --plugins' }}"        repoquery_installed: "{{ 'dnf repoquery --latest-limit 1 -d 0 --disableexcludes=all --installed' if ansible_pkg_mgr == 'dnf' else 'repoquery --plugins --installed' }}" + +- name: Initialize special first-master variables +  hosts: oo_first_master +  roles: +  - role: openshift_facts +  tasks: +  - set_fact: +      # We need to setup openshift_client_binary here for special uses of delegate_to in +      # later roles and plays. +      first_master_client_binary: "{{  openshift_client_binary }}" diff --git a/playbooks/openshift-etcd/private/upgrade_image_members.yml b/playbooks/openshift-etcd/private/upgrade_image_members.yml index 339fc6b74..d4386249e 100644 --- a/playbooks/openshift-etcd/private/upgrade_image_members.yml +++ b/playbooks/openshift-etcd/private/upgrade_image_members.yml @@ -1,7 +1,7 @@  ---  # INPUT etcd_upgrade_version  # INPUT etcd_container_version -# INPUT openshift.common.is_containerized +# INPUT openshift_is_containerized  - name: Upgrade containerized hosts to {{ etcd_upgrade_version }}    hosts: oo_etcd_hosts_to_upgrade    serial: 1 @@ -14,4 +14,4 @@        etcd_peer: "{{ openshift.common.hostname }}"      when:      - etcd_container_version | default('99') is version_compare(etcd_upgrade_version,'<') -    - openshift.common.is_containerized | bool +    - openshift_is_containerized | bool diff --git a/playbooks/openshift-etcd/private/upgrade_rpm_members.yml b/playbooks/openshift-etcd/private/upgrade_rpm_members.yml index 327a35b09..f7fe6cd9c 100644 --- a/playbooks/openshift-etcd/private/upgrade_rpm_members.yml +++ b/playbooks/openshift-etcd/private/upgrade_rpm_members.yml @@ -1,7 +1,7 @@  ---  # INPUT etcd_upgrade_version  # INPUT etcd_rpm_version -# INPUT openshift.common.is_containerized +# INPUT openshift_is_containerized  - name: Upgrade to {{ etcd_upgrade_version }}    hosts: oo_etcd_hosts_to_upgrade    serial: 1 @@ -15,4 +15,4 @@      when:      - etcd_rpm_version.stdout | default('99') is version_compare(etcd_upgrade_version, '<')      - ansible_distribution == 'RedHat' -    - not openshift.common.is_containerized | bool +    - not openshift_is_containerized | bool diff --git a/playbooks/openshift-etcd/private/upgrade_step.yml b/playbooks/openshift-etcd/private/upgrade_step.yml index 60127fc68..05c543d62 100644 --- a/playbooks/openshift-etcd/private/upgrade_step.yml +++ b/playbooks/openshift-etcd/private/upgrade_step.yml @@ -61,4 +61,4 @@        etcd_peer: "{{ openshift.common.hostname }}"      when:      - ansible_distribution == 'Fedora' -    - not openshift.common.is_containerized | bool +    - not openshift_is_containerized | bool diff --git a/playbooks/openshift-hosted/private/redeploy-registry-certificates.yml b/playbooks/openshift-hosted/private/redeploy-registry-certificates.yml index 3943720e3..b817221b8 100644 --- a/playbooks/openshift-hosted/private/redeploy-registry-certificates.yml +++ b/playbooks/openshift-hosted/private/redeploy-registry-certificates.yml @@ -17,7 +17,7 @@    - name: Determine if docker-registry exists      command: > -      {{ openshift.common.client_binary }} get dc/docker-registry -o json +      {{ openshift_client_binary }} get dc/docker-registry -o json        --config={{ mktemp.stdout }}/admin.kubeconfig        -n default      register: l_docker_registry_dc @@ -38,7 +38,7 @@    # Replace dc/docker-registry environment variable certificate data if set.    - name: Update docker-registry environment variables      shell: > -      {{ openshift.common.client_binary }} env dc/docker-registry +      {{ openshift_client_binary }} env dc/docker-registry        OPENSHIFT_CA_DATA="$(cat /etc/origin/master/ca.crt)"        OPENSHIFT_CERT_DATA="$(cat /etc/origin/master/openshift-registry.crt)"        OPENSHIFT_KEY_DATA="$(cat /etc/origin/master/openshift-registry.key)" @@ -62,7 +62,7 @@      - name: Generate registry certificate        command: > -        {{ openshift.common.client_binary }} adm ca create-server-cert +        {{ openshift_client_binary }} adm ca create-server-cert          --signer-cert={{ openshift.common.config_base }}/master/ca.crt          --signer-key={{ openshift.common.config_base }}/master/ca.key          --signer-serial={{ openshift.common.config_base }}/master/ca.serial.txt @@ -88,7 +88,7 @@    - name: Redeploy docker registry      command: > -      {{ openshift.common.client_binary }} deploy dc/docker-registry +      {{ openshift_client_binary }} deploy dc/docker-registry        --latest        --config={{ mktemp.stdout }}/admin.kubeconfig        -n default diff --git a/playbooks/openshift-hosted/private/redeploy-router-certificates.yml b/playbooks/openshift-hosted/private/redeploy-router-certificates.yml index b1f60f1ae..c19147d41 100644 --- a/playbooks/openshift-hosted/private/redeploy-router-certificates.yml +++ b/playbooks/openshift-hosted/private/redeploy-router-certificates.yml @@ -17,7 +17,7 @@    - name: Determine if router exists      command: > -      {{ openshift.common.client_binary }} get dc/router -o json +      {{ openshift_client_binary }} get dc/router -o json        --config={{ router_cert_redeploy_tempdir.stdout }}/admin.kubeconfig        -n default      register: l_router_dc @@ -26,7 +26,7 @@    - name: Determine if router service exists      command: > -      {{ openshift.common.client_binary }} get svc/router -o json +      {{ openshift_client_binary }} get svc/router -o json        --config={{ router_cert_redeploy_tempdir.stdout }}/admin.kubeconfig        -n default      register: l_router_svc @@ -52,7 +52,7 @@    - name: Update router environment variables      shell: > -      {{ openshift.common.client_binary }} env dc/router +      {{ openshift_client_binary }} env dc/router        OPENSHIFT_CA_DATA="$(cat /etc/origin/master/ca.crt)"        OPENSHIFT_CERT_DATA="$(cat /etc/origin/master/openshift-router.crt)"        OPENSHIFT_KEY_DATA="$(cat /etc/origin/master/openshift-router.key)" @@ -78,7 +78,7 @@      - name: Remove router service annotations        command: > -        {{ openshift.common.client_binary }} annotate service/router +        {{ openshift_client_binary }} annotate service/router          service.alpha.openshift.io/serving-cert-secret-name-          service.alpha.openshift.io/serving-cert-signed-by-          --config={{ router_cert_redeploy_tempdir.stdout }}/admin.kubeconfig @@ -86,7 +86,7 @@      - name: Add serving-cert-secret annotation to router service        command: > -        {{ openshift.common.client_binary }} annotate service/router +        {{ openshift_client_binary }} annotate service/router          service.alpha.openshift.io/serving-cert-secret-name=router-certs          --config={{ router_cert_redeploy_tempdir.stdout }}/admin.kubeconfig          -n default @@ -129,7 +129,7 @@    - name: Redeploy router      command: > -      {{ openshift.common.client_binary }} deploy dc/router +      {{ openshift_client_binary }} deploy dc/router        --latest        --config={{ router_cert_redeploy_tempdir.stdout }}/admin.kubeconfig        -n default diff --git a/playbooks/openshift-master/private/additional_config.yml b/playbooks/openshift-master/private/additional_config.yml index a90cd6b22..81bb8cc5c 100644 --- a/playbooks/openshift-master/private/additional_config.yml +++ b/playbooks/openshift-master/private/additional_config.yml @@ -30,7 +30,7 @@      when: openshift_use_manageiq | default(true) | bool    - role: cockpit      when: -    - not openshift.common.is_atomic | bool +    - not openshift_is_atomic | bool      - deployment_type == 'openshift-enterprise'      - osm_use_cockpit is undefined or osm_use_cockpit | bool      - openshift.common.deployment_subtype != 'registry' diff --git a/playbooks/openshift-master/private/tasks/wire_aggregator.yml b/playbooks/openshift-master/private/tasks/wire_aggregator.yml index 4f55d5c82..59e2b515c 100644 --- a/playbooks/openshift-master/private/tasks/wire_aggregator.yml +++ b/playbooks/openshift-master/private/tasks/wire_aggregator.yml @@ -21,7 +21,7 @@  # TODO: this currently has a bug where hostnames are required  - name: Creating First Master Aggregator signer certs    command: > -    {{ hostvars[groups.oo_first_master.0].openshift.common.client_binary }} adm ca create-signer-cert +    {{ hostvars[groups.oo_first_master.0]['first_master_client_binary'] }} adm ca create-signer-cert      --cert=/etc/origin/master/front-proxy-ca.crt      --key=/etc/origin/master/front-proxy-ca.key      --serial=/etc/origin/master/ca.serial.txt @@ -84,7 +84,7 @@  - block:    - name: Create first master api-client config for Aggregator      command: > -      {{ hostvars[groups.oo_first_master.0].openshift.common.client_binary }} adm create-api-client-config +      {{ hostvars[groups.oo_first_master.0]['first_master_client_binary'] }} adm create-api-client-config        --certificate-authority=/etc/origin/master/front-proxy-ca.crt        --signer-cert=/etc/origin/master/front-proxy-ca.crt        --signer-key=/etc/origin/master/front-proxy-ca.key diff --git a/playbooks/openshift-node/private/restart.yml b/playbooks/openshift-node/private/restart.yml index c2092b23c..7249ced70 100644 --- a/playbooks/openshift-node/private/restart.yml +++ b/playbooks/openshift-node/private/restart.yml @@ -28,7 +28,7 @@      - "{{ openshift_service_type }}-master-controllers"      - "{{ openshift_service_type }}-node"      failed_when: false -    when: openshift.common.is_containerized | bool +    when: openshift_is_containerized | bool    - name: Wait for master API to come back online      wait_for: diff --git a/playbooks/openshift-node/private/setup.yml b/playbooks/openshift-node/private/setup.yml index 541913aef..802dce37e 100644 --- a/playbooks/openshift-node/private/setup.yml +++ b/playbooks/openshift-node/private/setup.yml @@ -21,6 +21,6 @@      when:      - hostvars[item].openshift is defined      - hostvars[item].openshift.common is defined -    - hostvars[item].openshift.common.is_containerized | bool +    - hostvars[item].openshift_is_containerized | bool      - (item in groups.oo_nodes_to_config and item in groups.oo_masters_to_config)      changed_when: False | 
