diff --git a/common/deploy-steps-tasks-step-1.yaml b/common/deploy-steps-tasks-step-1.yaml index 35b52746b5..84adcb3bd8 100644 --- a/common/deploy-steps-tasks-step-1.yaml +++ b/common/deploy-steps-tasks-step-1.yaml @@ -255,13 +255,13 @@ - name: gather facts needed by role setup: gather_subset: "!min,python" - when: ansible_python is not defined + when: ansible_facts['python'] is not defined tags: - container_config_tasks - name: set python_cmd set_fact: - python_cmd: "python{{ ansible_python.version.major }}" + python_cmd: "python{{ ansible_facts['python']['version']['major'] }}" cacheable: true when: python_cmd is not defined tags: diff --git a/common/deploy-steps-tasks.yaml b/common/deploy-steps-tasks.yaml index b310e199f5..9c3a6df157 100644 --- a/common/deploy-steps-tasks.yaml +++ b/common/deploy-steps-tasks.yaml @@ -117,7 +117,7 @@ # Bootstrap tasks - run any tasks that have been defined ######################################################## -- name: "Clean container_puppet_tasks for {{ansible_hostname | lower}} step {{step}}" +- name: "Clean container_puppet_tasks for {{ansible_facts['hostname'] | lower}} step {{step}}" become: true file: path: /var/lib/container-puppet/container-puppet-tasks{{step}}.json @@ -125,11 +125,11 @@ tags: - container_config_tasks -- name: Calculate container_puppet_tasks for {{ansible_hostname | lower}} step {{step}} +- name: Calculate container_puppet_tasks for {{ansible_facts['hostname'] | lower}} step {{step}} set_fact: "{{'host_container_puppet_tasks_' ~ step}}": "{{lookup('vars', 'host_container_puppet_tasks_' ~ step, default=[]) | union([item])}}" loop: "{{container_puppet_tasks.get('step_' ~ step, [])}}" - when: (groups[item.service_name] | default ([]) | map('extract', hostvars, 'inventory_hostname') | sort | first | lower) == ansible_hostname | lower + when: (groups[item.service_name] | default ([]) | map('extract', hostvars, 'inventory_hostname') | sort | first | lower) == ansible_facts['hostname'] | lower vars: container_puppet_tasks: "{{ lookup('file', tripleo_role_name + '/container_puppet_tasks.yaml', errors='ignore') | default({}, True) | from_yaml }}" tags: diff --git a/common/deploy-steps.j2 b/common/deploy-steps.j2 index eea1ff680a..b013cf3d0f 100644 --- a/common/deploy-steps.j2 +++ b/common/deploy-steps.j2 @@ -1123,9 +1123,9 @@ outputs: loop_control: loop_var: step tags: always - # we use ansible_hostname to determine if the host is alive + # we use ansible_facts['hostname'] to determine if the host is alive # or not. - when: ansible_hostname is defined + when: ansible_facts['hostname'] is defined # We don't want to run the scale tasks on dead nodes, to allow # the operator to scale down the cloud no matter the state of # the servers. @@ -1136,9 +1136,9 @@ outputs: msg: "Node is unreachable. No scale tasks will be run." ignore_errors: True tags: always - # we use ansible_hostname to determine if the host is alive + # we use ansible_facts['hostname'] to determine if the host is alive # or not. - when: ansible_hostname is not defined + when: ansible_facts['hostname'] is not defined tags: - scale post_update_steps_tasks: | diff --git a/common/generate-config-tasks.yaml b/common/generate-config-tasks.yaml index 7af8993c57..471edf266e 100644 --- a/common/generate-config-tasks.yaml +++ b/common/generate-config-tasks.yaml @@ -18,7 +18,7 @@ net_host: true no_archive: false puppet_config: "/var/lib/container-puppet/{{ ansible_check_mode | bool | ternary('check-mode/', '') }}container-puppet.json" - short_hostname: "{{ ansible_hostname | lower }}" + short_hostname: "{{ ansible_facts['hostname'] | lower }}" step: "{{ step }}" - name: "Manage Puppet containers (generate config) for step {{ step }} with tripleo-ansible" diff --git a/common/host-container-puppet-tasks.yaml b/common/host-container-puppet-tasks.yaml index 3e206efb91..b172ba5bff 100644 --- a/common/host-container-puppet-tasks.yaml +++ b/common/host-container-puppet-tasks.yaml @@ -1,4 +1,4 @@ -- name: Write container-puppet-tasks json file for {{ansible_hostname | lower}} step {{step}} +- name: Write container-puppet-tasks json file for {{ansible_facts['hostname'] | lower}} step {{step}} no_log: True copy: content: "{{lookup ('vars', 'host_container_puppet_tasks_' ~ step, default=[]) | to_nice_json}}" @@ -21,7 +21,7 @@ net_host: true no_archive: true puppet_config: "/var/lib/container-puppet/{{ ansible_check_mode | bool | ternary('check-mode/', '') }}container-puppet-tasks{{ step }}.json" - short_hostname: "{{ ansible_hostname | lower }}" + short_hostname: "{{ ansible_facts['hostname'] | lower }}" step: "{{ step }}" - name: "Manage Puppet containers (bootstrap tasks) for step {{ step }} with tripleo-ansible" diff --git a/deployment/cinder/cinder-backup-pacemaker-puppet.yaml b/deployment/cinder/cinder-backup-pacemaker-puppet.yaml index 6182acae99..8d62f940e0 100644 --- a/deployment/cinder/cinder-backup-pacemaker-puppet.yaml +++ b/deployment/cinder/cinder-backup-pacemaker-puppet.yaml @@ -315,7 +315,7 @@ outputs: cinder_backup_pcs_res: "{{cinder_backup_pcs_res_result.rc == 0}}" - name: set is_cinder_backup_bootstrap_node fact tags: common - set_fact: is_cinder_backup_bootstrap_node={{cinder_backup_short_bootstrap_node_name|lower == ansible_hostname|lower}} + set_fact: is_cinder_backup_bootstrap_node={{cinder_backup_short_bootstrap_node_name|lower == ansible_facts['hostname']|lower}} - name: Update cinder_backup pcs resource bundle for new container image when: - step|int == 1 diff --git a/deployment/cinder/cinder-volume-pacemaker-puppet.yaml b/deployment/cinder/cinder-volume-pacemaker-puppet.yaml index 9d187932e3..6e32323b3f 100644 --- a/deployment/cinder/cinder-volume-pacemaker-puppet.yaml +++ b/deployment/cinder/cinder-volume-pacemaker-puppet.yaml @@ -296,7 +296,7 @@ outputs: cinder_volume_pcs_res: "{{cinder_volume_pcs_res_result.rc == 0}}" - name: set is_cinder_volume_bootstrap_node fact tags: common - set_fact: is_cinder_volume_bootstrap_node={{cinder_volume_short_bootstrap_node_name|lower == ansible_hostname|lower}} + set_fact: is_cinder_volume_bootstrap_node={{cinder_volume_short_bootstrap_node_name|lower == ansible_facts['hostname']|lower}} - name: Update cinder_volume pcs resource bundle for new container image when: - step|int == 1 diff --git a/deployment/database/mysql-pacemaker-puppet.yaml b/deployment/database/mysql-pacemaker-puppet.yaml index 12b111953c..33293a72c1 100644 --- a/deployment/database/mysql-pacemaker-puppet.yaml +++ b/deployment/database/mysql-pacemaker-puppet.yaml @@ -483,7 +483,7 @@ outputs: galera_pcs_res: "{{galera_pcs_res_result.rc == 0}}" - name: set is_mysql_bootstrap_node fact tags: common - set_fact: is_mysql_bootstrap_node={{mysql_short_bootstrap_node_name|lower == ansible_hostname|lower}} + set_fact: is_mysql_bootstrap_node={{mysql_short_bootstrap_node_name|lower == ansible_facts['hostname']|lower}} - name: Update galera pcs resource bundle for new container image when: - step|int == 1 diff --git a/deployment/database/redis-pacemaker-puppet.yaml b/deployment/database/redis-pacemaker-puppet.yaml index 5c1b4c5a8c..8b9e92f43f 100644 --- a/deployment/database/redis-pacemaker-puppet.yaml +++ b/deployment/database/redis-pacemaker-puppet.yaml @@ -404,7 +404,7 @@ outputs: - name: Set upgrade redis facts set_fact: redis_pcs_res: "{{redis_pcs_res_result.rc == 0}}" - is_redis_bootstrap_node: "{{redis_short_bootstrap_node_name|lower == ansible_hostname|lower}}" + is_redis_bootstrap_node: "{{redis_short_bootstrap_node_name|lower == ansible_facts['hostname']|lower}}" - name: Update redis-bundle pcs resource bundle for new container image when: - step|int == 1 diff --git a/deployment/deprecated/novajoin/novajoin-container-puppet.yaml b/deployment/deprecated/novajoin/novajoin-container-puppet.yaml index 087d167fd5..0f4b0b6f41 100644 --- a/deployment/deprecated/novajoin/novajoin-container-puppet.yaml +++ b/deployment/deprecated/novajoin/novajoin-container-puppet.yaml @@ -238,8 +238,8 @@ outputs: # openssl-perl is installed which provides /etc/pki/CA on RHEL8 - name: Ensure openssl-perl package is present on RHEL8 when: - - ansible_os_family == 'RedHat' - - ansible_distribution_major_version == '8' + - ansible_facts['os_family'] == 'RedHat' + - ansible_facts['distribution_major_version'] is version('8', '==') package: name: openssl-perl state: present @@ -265,6 +265,6 @@ outputs: creates: /etc/ipa/default.conf when: ipa_otp != '' - name: Request kerberos keytab - shell: "/usr/bin/kinit -kt /etc/krb5.keytab && ipa-getkeytab -s $(grep xmlrpc_uri /etc/ipa/default.conf | cut -d/ -f3) -p nova/{{ ansible_nodename }} -k /etc/novajoin/krb5.keytab" + shell: "/usr/bin/kinit -kt /etc/krb5.keytab && ipa-getkeytab -s $(grep xmlrpc_uri /etc/ipa/default.conf | cut -d/ -f3) -p nova/{{ ansible_facts['nodename'] }} -k /etc/novajoin/krb5.keytab" args: creates: /etc/novajoin/krb5.keytab diff --git a/deployment/haproxy/haproxy-pacemaker-puppet.yaml b/deployment/haproxy/haproxy-pacemaker-puppet.yaml index 5823e840c7..bb68d8446b 100644 --- a/deployment/haproxy/haproxy-pacemaker-puppet.yaml +++ b/deployment/haproxy/haproxy-pacemaker-puppet.yaml @@ -356,7 +356,7 @@ outputs: block: &haproxy_update_upgrade_facts - name: set is_haproxy_bootstrap_node fact tags: common - set_fact: is_haproxy_bootstrap_node={{haproxy_short_bootstrap_node_name|lower == ansible_hostname|lower}} + set_fact: is_haproxy_bootstrap_node={{haproxy_short_bootstrap_node_name|lower == ansible_facts['hostname']|lower}} when: - haproxy_short_bootstrap_node_name|default(false) - name: Mount TLS cert if needed @@ -471,7 +471,7 @@ outputs: - name: Set upgrade haproxy facts set_fact: haproxy_pcs_res: "{{haproxy_pcs_res_result.rc == 0}}" - is_haproxy_bootstrap_node: "{{haproxy_short_bootstrap_node_name|lower == ansible_hostname|lower}}" + is_haproxy_bootstrap_node: "{{haproxy_short_bootstrap_node_name|lower == ansible_facts['hostname']|lower}}" - name: Update haproxy pcs resource bundle for new container image when: diff --git a/deployment/haproxy/haproxy-public-tls-inject.yaml b/deployment/haproxy/haproxy-public-tls-inject.yaml index afd5703184..6701856d18 100644 --- a/deployment/haproxy/haproxy-public-tls-inject.yaml +++ b/deployment/haproxy/haproxy-public-tls-inject.yaml @@ -85,7 +85,7 @@ outputs: path: "{{cert_path}}" - name: set is_haproxy_bootstrap_node fact - set_fact: is_haproxy_bootstrap_node={{haproxy_short_bootstrap_node_name | lower == ansible_hostname | lower}} + set_fact: is_haproxy_bootstrap_node={{haproxy_short_bootstrap_node_name | lower == ansible_facts['hostname'] | lower}} when: - haproxy_short_bootstrap_node_name|default(false) diff --git a/deployment/ipa/ipaservices-baremetal-ansible.yaml b/deployment/ipa/ipaservices-baremetal-ansible.yaml index 1e58ee7677..0675cf0502 100644 --- a/deployment/ipa/ipaservices-baremetal-ansible.yaml +++ b/deployment/ipa/ipaservices-baremetal-ansible.yaml @@ -116,9 +116,9 @@ outputs: if: - idm_server_provided - IPA_HOST: {get_param: IdMServer} - IPA_USER: "nova/{{ ansible_fqdn }}" + IPA_USER: "nova/{{ ansible_facts['fqdn'] }}" KRB5_CLIENT_KTNAME: {get_param: IdMNovaKeytab} - - IPA_USER: "nova/{{ ansible_fqdn }}" + - IPA_USER: "nova/{{ ansible_facts['fqdn'] }}" KRB5_CLIENT_KTNAME: {get_param: IdMNovaKeytab} deploy_steps_tasks: - name: enroll the node as an ipa client diff --git a/deployment/manila/manila-share-pacemaker-puppet.yaml b/deployment/manila/manila-share-pacemaker-puppet.yaml index d71eac93fe..4a0dd3159f 100644 --- a/deployment/manila/manila-share-pacemaker-puppet.yaml +++ b/deployment/manila/manila-share-pacemaker-puppet.yaml @@ -272,7 +272,7 @@ outputs: manila_share_pcs_res: "{{manila_share_pcs_res_result.rc == 0}}" - name: set is_manila_share_bootstrap_node fact tags: common - set_fact: is_manila_share_bootstrap_node={{manila_share_short_bootstrap_node_name|lower == ansible_hostname|lower}} + set_fact: is_manila_share_bootstrap_node={{manila_share_short_bootstrap_node_name|lower == ansible_facts['hostname']|lower}} - name: Update openstack-manila-share pcs resource bundle for new container image when: - step|int == 1 diff --git a/deployment/nova/nova-compute-container-puppet.yaml b/deployment/nova/nova-compute-container-puppet.yaml index 6572c1afcc..f3e57a2533 100644 --- a/deployment/nova/nova-compute-container-puppet.yaml +++ b/deployment/nova/nova-compute-container-puppet.yaml @@ -1215,7 +1215,7 @@ outputs: register: iha_nodes - name: If instance HA is enabled on the node activate the evacuation completed check file: path=/var/lib/nova/instanceha/enabled state=touch - when: iha_nodes.stdout|lower is search('"'+ansible_hostname|lower+'"') + when: iha_nodes.stdout|lower is search('"'+ansible_facts['hostname']|lower+'"') - name: Is irqbalance enabled set_fact: compute_irqbalance_disabled: {get_attr: [RoleParametersValue, value, compute_disable_irqbalance]} @@ -1297,21 +1297,21 @@ outputs: name: qemu-kvm-common-ev state: present when: - - ansible_distribution == 'CentOS' - - ansible_distribution_major_version == '7' + - ansible_facts['distribution'] == 'CentOS' + - ansible_facts['distribution_major_version'] is version('7', '==') - name: make sure package providing ksmtuned is installed (RHEL7) package: name: qemu-kvm-common-rhev state: present when: - - ansible_distribution == 'RedHat' - - ansible_distribution_major_version == '7' + - ansible_facts['distribution'] == 'RedHat' + - ansible_facts['distribution_major_version'] is version('7', '==') - name: make sure package providing ksmtuned is installed (RHEL8 or CentOS8) package: name: qemu-kvm-common state: present when: - - ansible_distribution_major_version == '8' + - ansible_facts['distribution_major_version'] is version('8', '==') - name: enable ksmtunded service: name: "{{ item }}" @@ -1367,13 +1367,13 @@ outputs: is_additional_cell: {get_param: NovaAdditionalCell} - name: Set fact for nova_compute services set_fact: - nova_compute_service: "{{ nova_compute_service_result.stdout | from_yaml | selectattr('Host', 'match', ansible_fqdn ~ '.*') | list }}" + nova_compute_service: "{{ nova_compute_service_result.stdout | from_yaml | selectattr('Host', 'match', ansible_facts['fqdn'] ~ '.*') | list }}" delegate_to: localhost check_mode: no - name: Check search output fail: msg: >- - Found multiple `{{ ansible_fqdn }}`, which is unexpected. + Found multiple `{{ ansible_facts['fqdn'] }}`, which is unexpected. This means that the FQDN of the selected device to disable is either wrong or is sharing a name with another host, which is also wrong. Please correct this issue before continuing. Nova diff --git a/deployment/octavia/octavia-deployment-config.j2.yaml b/deployment/octavia/octavia-deployment-config.j2.yaml index ff8295463d..72594cb62f 100644 --- a/deployment/octavia/octavia-deployment-config.j2.yaml +++ b/deployment/octavia/octavia-deployment-config.j2.yaml @@ -305,7 +305,7 @@ outputs: {%- if 'octavia_' ~ octavia_groups %} {% for host in groups['octavia_' ~ octavia_group] -%} - {{ hostvars.raw_get(host)['ansible_hostname'] | lower}}: + {{ hostvars.raw_get(host)['ansible_facts']['hostname'] | lower}}: ansible_user: {{ hostvars.raw_get(host)['ansible_ssh_user'] | default('heat-admin') }} ansible_host: {{ hostvars.raw_get(host)['ansible_host'] | default(host) | lower }} canonical_hostname: {{ hostvars.raw_get(host)['canonical_hostname'] | default(host) | lower }} @@ -318,7 +318,7 @@ outputs: Undercloud: hosts: {% for host in groups['Undercloud'] -%} - {{ hostvars.raw_get(host)['ansible_hostname'] | lower}}: + {{ hostvars.raw_get(host)['ansible_facts']['hostname'] | lower}}: ansible_host: {{ hostvars.raw_get(host)['ansible_host'] | default(host) | lower }} ansible_become: false ansible_connection: local diff --git a/deployment/octavia/octavia-worker-container-puppet.yaml b/deployment/octavia/octavia-worker-container-puppet.yaml index b28c798f59..09aa61e202 100644 --- a/deployment/octavia/octavia-worker-container-puppet.yaml +++ b/deployment/octavia/octavia-worker-container-puppet.yaml @@ -151,7 +151,7 @@ outputs: - python2-openstackclient - openssl state: present - when: ansible_distribution_major_version is version(8, '<') + when: ansible_facts['distribution_major_version'] is version(8, '<') - name: Ensure packages required for configuring octavia are present for CentOS/RHEL 8 package: @@ -160,7 +160,7 @@ outputs: - python3-openstackclient - openssl state: present - when: ansible_distribution_major_version is version(8, '>=') + when: ansible_facts['distribution_major_version'] is version(8, '>=') when: {get_param: EnablePackageInstall} - name: enable virt_sandbox_use_netlink for healthcheck seboolean: diff --git a/deployment/ovn/ovn-dbs-pacemaker-puppet.yaml b/deployment/ovn/ovn-dbs-pacemaker-puppet.yaml index 37b1ee93f5..e6365a87cb 100644 --- a/deployment/ovn/ovn-dbs-pacemaker-puppet.yaml +++ b/deployment/ovn/ovn-dbs-pacemaker-puppet.yaml @@ -382,7 +382,7 @@ outputs: - name: Update ovn-dbs-bundle resource to use pcmklatest tag image if not used when: - step|int == 5 - - ovn_dbs_short_bootstrap_node_name|lower == ansible_hostname|lower + - ovn_dbs_short_bootstrap_node_name|lower == ansible_facts['hostname']|lower block: - name: Get the present image used by ovn-dbs-bundle shell: "pcs resource config ovn-dbs-bundle | grep -Eo 'image=[^ ]+' | awk -F= '{print $2;}'" @@ -446,7 +446,7 @@ outputs: - name: Update ovn_dbs pcs resource bundle for new container image when: - step|int == 1 - - ovn_dbs_short_bootstrap_node_name|lower == ansible_hostname|lower + - ovn_dbs_short_bootstrap_node_name|lower == ansible_facts['hostname']|lower - ovn_dbs_pcs_res|bool - ovn_dbs_image_current != ovn_dbs_image_latest block: @@ -528,7 +528,7 @@ outputs: - name: Restart ovn-dbs service (pacemaker) when: - step|int == 1 - - ovn_dbs_short_bootstrap_node_name|lower == ansible_hostname|lower + - ovn_dbs_short_bootstrap_node_name|lower == ansible_facts['hostname']|lower pacemaker_resource: resource: ovn-dbs-bundle state: restart diff --git a/deployment/rabbitmq/rabbitmq-messaging-notify-pacemaker-puppet.yaml b/deployment/rabbitmq/rabbitmq-messaging-notify-pacemaker-puppet.yaml index 9bd3c2c644..48a436f6db 100644 --- a/deployment/rabbitmq/rabbitmq-messaging-notify-pacemaker-puppet.yaml +++ b/deployment/rabbitmq/rabbitmq-messaging-notify-pacemaker-puppet.yaml @@ -359,7 +359,7 @@ outputs: set_fact: rabbitmq_pcs_res: "{{rabbitmq_pcs_res_result.rc == 0}}" - name: set is_notify_rabbitmq_bootstrap_node fact - set_fact: is_notify_rabbitmq_bootstrap_node={{oslo_messaging_notify_short_bootstrap_node_name|lower == ansible_hostname|lower}} + set_fact: is_notify_rabbitmq_bootstrap_node={{oslo_messaging_notify_short_bootstrap_node_name|lower == ansible_facts['hostname']|lower}} - name: Update rabbitmq-bundle pcs resource bundle for new container image when: - step|int == 1 diff --git a/deployment/rabbitmq/rabbitmq-messaging-pacemaker-puppet.yaml b/deployment/rabbitmq/rabbitmq-messaging-pacemaker-puppet.yaml index f929b971e9..1692376d86 100644 --- a/deployment/rabbitmq/rabbitmq-messaging-pacemaker-puppet.yaml +++ b/deployment/rabbitmq/rabbitmq-messaging-pacemaker-puppet.yaml @@ -360,7 +360,7 @@ outputs: rabbitmq_pcs_res: "{{rabbitmq_pcs_res_result.rc == 0}}" - name: set is_rabbitmq_bootstrap_node fact tags: common - set_fact: is_rabbitmq_bootstrap_node={{rabbitmq_short_bootstrap_node_name|lower == ansible_hostname|lower}} + set_fact: is_rabbitmq_bootstrap_node={{rabbitmq_short_bootstrap_node_name|lower == ansible_facts['hostname']|lower}} - name: Update rabbitmq-bundle pcs resource bundle for new container image when: - step|int == 1 diff --git a/deployment/rabbitmq/rabbitmq-messaging-rpc-pacemaker-puppet.yaml b/deployment/rabbitmq/rabbitmq-messaging-rpc-pacemaker-puppet.yaml index 8dff3bfe3f..92132d13cf 100644 --- a/deployment/rabbitmq/rabbitmq-messaging-rpc-pacemaker-puppet.yaml +++ b/deployment/rabbitmq/rabbitmq-messaging-rpc-pacemaker-puppet.yaml @@ -363,7 +363,7 @@ outputs: set_fact: rabbitmq_pcs_res: "{{rabbitmq_pcs_res_result.rc == 0}}" - name: set is_rpc_rabbitmq_bootstrap_node fact - set_fact: is_rpc_rabbitmq_bootstrap_node={{oslo_messaging_rpc_short_bootstrap_node_name|lower == ansible_hostname|lower}} + set_fact: is_rpc_rabbitmq_bootstrap_node={{oslo_messaging_rpc_short_bootstrap_node_name|lower == ansible_facts['hostname']|lower}} - name: Update rabbitmq-bundle pcs resource bundle for new container image when: - step|int == 1 diff --git a/deployment/swift/swift-storage-container-puppet.yaml b/deployment/swift/swift-storage-container-puppet.yaml index 7b34fd570d..7da68ea0ec 100644 --- a/deployment/swift/swift-storage-container-puppet.yaml +++ b/deployment/swift/swift-storage-container-puppet.yaml @@ -628,11 +628,12 @@ outputs: - '!all' - '!min' - 'hardware' + filter: 'ansible_device_links' when: swift_raw_disks - name: Mount devices defined in SwiftRawDisks mount: name: /srv/node/{{ item }} - src: "{% if lsblk.results['uuids'][item] is defined %}UUID={{ ansible_device_links['uuids'][item][0] }}{% else %}{{ swift_raw_disks[item]['base_dir']|default('/dev') }}/{{ item }}{% endif %}" + src: "{% if lsblk.results['uuids'][item] is defined %}UUID={{ ansible_facts['device_links']['uuids'][item][0] }}{% else %}{{ swift_raw_disks[item]['base_dir']|default('/dev') }}/{{ item }}{% endif %}" fstype: xfs opts: noatime state: mounted diff --git a/deployment/tls/undercloud-tls.yaml b/deployment/tls/undercloud-tls.yaml index 7e32c95987..d7f781b40a 100644 --- a/deployment/tls/undercloud-tls.yaml +++ b/deployment/tls/undercloud-tls.yaml @@ -50,8 +50,8 @@ outputs: # openssl-perl is installed which provides /etc/pki/CA on RHEL8 - name: Ensure openssl-perl package is present on RHEL8 when: - - ansible_os_family == 'RedHat' - - ansible_distribution_major_version == '8' + - ansible_facts['os_family'] == 'RedHat' + - ansible_facts['distribution_major_version'] is version('8', '==') package: name: openssl-perl state: present @@ -71,7 +71,7 @@ outputs: when: ipa_otp != '' - name: Set keytab permission facts set_fact: - nova_service: "nova/{{ ansible_nodename }}" + nova_service: "nova/{{ ansible_facts['nodename'] }}" nova_keytab: "/etc/novajoin/krb5.keytab" nova_keytab_group: "tripleo-admin" - name: Add directory for keytab diff --git a/deployment/tripleo-packages/tripleo-packages-baremetal-puppet.yaml b/deployment/tripleo-packages/tripleo-packages-baremetal-puppet.yaml index 8c483f7cda..291a4ff4ce 100644 --- a/deployment/tripleo-packages/tripleo-packages-baremetal-puppet.yaml +++ b/deployment/tripleo-packages/tripleo-packages-baremetal-puppet.yaml @@ -261,7 +261,7 @@ outputs: state: present loop: "{{ dnf_module_list|list }}" when: - - ansible_distribution_major_version >= '8' + - ansible_facts['distribution_major_version'] is version('8', '>=') - dnf_module_list|length > 0 - name: Ensure TripleO prerequisite packages are installed package: @@ -275,7 +275,7 @@ outputs: - python3-heat-agent* - rsync state: present - when: ansible_distribution_major_version == '8' + when: ansible_facts['distribution_major_version'] is version('8', '==') - name: Special treatment for OpenvSwitch tripleo_ovs_upgrade: when: @@ -350,7 +350,7 @@ outputs: skip_rhel_enforcement: {get_param: SkipRhelEnforcement} when: - step|int == 0 - - ansible_distribution == 'RedHat' + - ansible_facts['distribution'] == 'RedHat' - not (skip_rhel_enforcement | bool) - name: Ensure DNF modules have the right stream vars: @@ -361,7 +361,7 @@ outputs: loop: "{{ dnf_module_list|list }}" when: - step|int == 0 - - ansible_distribution_major_version >= '8' + - ansible_facts['distribution_major_version'] is version('8', '>=') - dnf_module_list|length > 0 - name: Check for existing yum.pid stat: path=/run/yum.pid diff --git a/deployment/undercloud/undercloud-upgrade.yaml b/deployment/undercloud/undercloud-upgrade.yaml index 177ecc9245..572233d920 100644 --- a/deployment/undercloud/undercloud-upgrade.yaml +++ b/deployment/undercloud/undercloud-upgrade.yaml @@ -65,7 +65,7 @@ outputs: skip_rhel_enforcement: {get_param: SkipRhelEnforcement} when: - step|int == 0 - - ansible_distribution == 'RedHat' + - ansible_facts['distribution'] == 'RedHat' - not (skip_rhel_enforcement | bool) - name: Ensure DNF modules have the right stream vars: @@ -76,7 +76,7 @@ outputs: loop: "{{ dnf_module_list|list }}" when: - step|int == 0 - - ansible_distribution_major_version >= '8' + - ansible_facts['distribution_major_version'] is version('8', '>=') - dnf_module_list|length > 0 - name: Special treatment for OpenvSwitch tripleo_ovs_upgrade: