Only create local facts folder when necessary

If we do not plan to set any local facts, we should
not be creating this folder. If we do need to set the
facts, then the tasks will need to be run via sudo to
ensure that they has the appropriate rights to create
the folder/files under /etc.

If this change is not implemented, and the playbook
running this role is not executed as root, then the
task fails because it does not have rights to create
the folder - even when there are no facts to set.

Change-Id: Idcb093775507728a7c68a877a05b8895d4c8c39f
This commit is contained in:
Jesse Pretorius (odyssey4me) 2020-07-28 16:59:26 +01:00
parent aabd3c07c2
commit 880ed4b185

View File

@ -14,11 +14,15 @@
# limitations under the License.
- name: Ensure local facts folder exists
become: true
file:
path: /etc/ansible/facts.d
state: directory
when:
- venv_facts_when_changed != []
- name: Record the necessary facts
become: true
ini_file:
dest: "/etc/ansible/facts.d/{{ venv_facts_dest }}.fact"
section: "{{ item.section }}"