Use FQCN for ansible builtin tasks
Using the FQCN for ansible builtin tasks is required by the new
ansible-lint 6.0.0. Otherwise, the pep8 job fails running ansible-lint.
Closes-Bug: #1964935
Change-Id: I82f65a39856bc86cbdb58de969f0392ea36c4e04
Signed-off-by: James Slagle <jslagle@redhat.com>
(cherry picked from commit de668b4ec2
)
This commit is contained in:
parent
4b13772c13
commit
524d8a8872
|
@ -8,10 +8,10 @@
|
||||||
- hosts: localhost
|
- hosts: localhost
|
||||||
name: Check if required variables are defined
|
name: Check if required variables are defined
|
||||||
tasks:
|
tasks:
|
||||||
- fail:
|
- ansible.builtin.fail:
|
||||||
msg: deploy_source_host variable is not defined
|
msg: deploy_source_host variable is not defined
|
||||||
when: deploy_source_host is not defined
|
when: deploy_source_host is not defined
|
||||||
- fail:
|
- ansible.builtin.fail:
|
||||||
msg: deploy_target_host variable is not defined
|
msg: deploy_target_host variable is not defined
|
||||||
when: deploy_target_host is not defined
|
when: deploy_target_host is not defined
|
||||||
# Keep in mind that you will still need to use defaults in order to be
|
# Keep in mind that you will still need to use defaults in order to be
|
||||||
|
@ -22,7 +22,7 @@
|
||||||
# We don't want to gather facts, just clear them
|
# We don't want to gather facts, just clear them
|
||||||
gather_facts: false
|
gather_facts: false
|
||||||
tasks:
|
tasks:
|
||||||
- meta: clear_facts
|
- ansible.builtin.meta: clear_facts
|
||||||
tags:
|
tags:
|
||||||
- facts
|
- facts
|
||||||
|
|
||||||
|
@ -32,7 +32,7 @@
|
||||||
gather_facts: true
|
gather_facts: true
|
||||||
tasks:
|
tasks:
|
||||||
- name: Set legacy facts
|
- name: Set legacy facts
|
||||||
set_fact:
|
ansible.builtin.set_fact:
|
||||||
ansible_distribution: "{{ ansible_facts['distribution'] }}"
|
ansible_distribution: "{{ ansible_facts['distribution'] }}"
|
||||||
ansible_distribution_major_version: "{{ ansible_facts['distribution_major_version'] }}"
|
ansible_distribution_major_version: "{{ ansible_facts['distribution_major_version'] }}"
|
||||||
ansible_distribution_release: "{{ ansible_facts['distribution_release'] }}"
|
ansible_distribution_release: "{{ ansible_facts['distribution_release'] }}"
|
||||||
|
@ -47,7 +47,7 @@
|
||||||
any_errors_fatal: false
|
any_errors_fatal: false
|
||||||
ignore_unreachable: true
|
ignore_unreachable: true
|
||||||
tasks:
|
tasks:
|
||||||
- include_vars: global_vars.yaml # noqa: missing-import
|
- ansible.builtin.include_vars: global_vars.yaml # noqa: missing-import
|
||||||
no_log: true
|
no_log: true
|
||||||
tags:
|
tags:
|
||||||
- always
|
- always
|
||||||
|
@ -59,12 +59,12 @@
|
||||||
ignore_unreachable: true
|
ignore_unreachable: true
|
||||||
tasks:
|
tasks:
|
||||||
- name: Include Service VIP vars
|
- name: Include Service VIP vars
|
||||||
include_vars:
|
ansible.builtin.include_vars:
|
||||||
file: "{{ playbook_dir }}/service_vip_vars.yaml"
|
file: "{{ playbook_dir }}/service_vip_vars.yaml"
|
||||||
name: service_vip_vars
|
name: service_vip_vars
|
||||||
ignore_errors: yes
|
ignore_errors: yes
|
||||||
- name: Include OVN bridge MAC address variables
|
- name: Include OVN bridge MAC address variables
|
||||||
include_vars:
|
ansible.builtin.include_vars:
|
||||||
file: "{{ playbook_dir }}/ovn_bridge_mac_address_vars.yaml"
|
file: "{{ playbook_dir }}/ovn_bridge_mac_address_vars.yaml"
|
||||||
name: ovn_bridge_mac_address_vars
|
name: ovn_bridge_mac_address_vars
|
||||||
ignore_errors: yes
|
ignore_errors: yes
|
||||||
|
@ -95,7 +95,7 @@
|
||||||
ignore_unreachable: true
|
ignore_unreachable: true
|
||||||
tasks:
|
tasks:
|
||||||
- name: Set all_nodes data as group_vars for overcloud
|
- name: Set all_nodes data as group_vars for overcloud
|
||||||
include_vars: "{{ playbook_dir }}/group_vars/overcloud.json"
|
ansible.builtin.include_vars: "{{ playbook_dir }}/group_vars/overcloud.json"
|
||||||
no_log: true
|
no_log: true
|
||||||
when: not ansible_check_mode|bool
|
when: not ansible_check_mode|bool
|
||||||
tags:
|
tags:
|
||||||
|
|
Loading…
Reference in New Issue