From 556df3017c6088450bfdec68ba964cd85e63ab94 Mon Sep 17 00:00:00 2001 From: Dwane Pottratz Date: Mon, 7 Feb 2022 09:01:06 -0800 Subject: [PATCH] ansible setup filter is not respecting a string according to https://docs.ansible.com/ansible/latest/collections/ansible/builtin/setup_module.html filter is supposed to be able to use a string. However it does not accept a string. Signed-off-by: Dwane Pottratz --- playbooks/ceph-install.yml | 15 ++++++++++----- playbooks/os-masakari-install.yml | 3 ++- playbooks/security-hardening.yml | 3 ++- 3 files changed, 14 insertions(+), 7 deletions(-) diff --git a/playbooks/ceph-install.yml b/playbooks/ceph-install.yml index cf2a126c7..3cf64b7ff 100644 --- a/playbooks/ceph-install.yml +++ b/playbooks/ceph-install.yml @@ -30,7 +30,8 @@ - name: Gather additional facts for monitor_address_block include_tasks: "common-tasks/gather-hardware-facts.yml" vars: - deployment_extra_facts_filter: "ansible_all_ipv[4,6]_addresses" + deployment_extra_facts_filter: + - "ansible_all_ipv[4,6]_addresses" deployment_extra_facts_subset: "!all,network" when: monitor_address_block is defined tags: @@ -39,7 +40,8 @@ - name: Gather additional facts for monitor_interface include_tasks: "common-tasks/gather-hardware-facts.yml" vars: - deployment_extra_facts_filter: "{{ 'ansible_' ~ monitor_interface | replace('-','_') }}" + deployment_extra_facts_filter: + - "{{ 'ansible_' ~ monitor_interface | replace('-','_') }}" deployment_extra_facts_subset: "!all,network" when: monitor_interface is defined tags: @@ -48,7 +50,8 @@ - name: Gather memory facts include_tasks: "common-tasks/gather-hardware-facts.yml" vars: - deployment_extra_facts_filter: "ansible_memtotal*" + deployment_extra_facts_filter: + - "ansible_memtotal*" deployment_extra_facts_subset: "!all,hardware" tags: - always @@ -131,7 +134,8 @@ - name: Gather memory facts include_tasks: "common-tasks/gather-hardware-facts.yml" vars: - deployment_extra_facts_filter: "ansible_memtotal*" + deployment_extra_facts_filter: + - "ansible_memtotal*" deployment_extra_facts_subset: "!all,hardware" tags: - always @@ -189,7 +193,8 @@ - name: Gather memory facts include_tasks: "common-tasks/gather-hardware-facts.yml" vars: - deployment_extra_facts_filter: "ansible_memtotal*" + deployment_extra_facts_filter: + - "ansible_memtotal*" deployment_extra_facts_subset: "!all,hardware" tags: - always diff --git a/playbooks/os-masakari-install.yml b/playbooks/os-masakari-install.yml index c094ae5b9..b9cb41d49 100644 --- a/playbooks/os-masakari-install.yml +++ b/playbooks/os-masakari-install.yml @@ -32,7 +32,8 @@ - name: Gather additional facts include_tasks: "common-tasks/gather-hardware-facts.yml" vars: - deployment_extra_facts_filter: "{{ 'ansible_' ~ pacemaker_corosync_ring_interface | replace('-','_') }}" + deployment_extra_facts_filter: + - "{{ 'ansible_' ~ pacemaker_corosync_ring_interface | replace('-','_') }}" deployment_extra_facts_subset: "!all,network" tags: - always diff --git a/playbooks/security-hardening.yml b/playbooks/security-hardening.yml index 66835f532..6bb378060 100644 --- a/playbooks/security-hardening.yml +++ b/playbooks/security-hardening.yml @@ -31,7 +31,8 @@ - name: Gather additional facts include_tasks: "common-tasks/gather-hardware-facts.yml" vars: - deployment_extra_facts_filter: ansible_mounts + deployment_extra_facts_filter: + - ansible_mounts deployment_extra_facts_subset: "!all,hardware" tags: - always -- 2.25.1