Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Client: Enable to ignore all facts #579

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

superstes
Copy link

@superstes superstes commented Jan 10, 2025

The Ansible-facts gather_subset allows for the value all to be supplied: https://docs.ansible.com/ansible/latest/collections/ansible/builtin/setup_module.html#parameter-gather_subset

It would be logical that ARA allows users to ignore all facts the same way.
This PR adds that functionality.

This is especially useful as some facts are dynamically named - like network-interface facts.

@superstes superstes changed the title Enable to ignore all facts Client: Enable to ignore all facts Jan 10, 2025
Copy link

Build succeeded.
https://ansible.softwarefactory-project.io/zuul/buildset/d7a4cb8aa7954ce4a44c8eea7e021d17

✔️ ara-tox-py3 SUCCESS in 2m 19s
✔️ ara-tox-linters SUCCESS in 2m 13s
✔️ ara-basic-ansible-core-devel SUCCESS in 5m 00s (non-voting)
✔️ ara-basic-ansible-10 SUCCESS in 5m 03s
✔️ ara-basic-ansible-core-2.16 SUCCESS in 5m 01s
✔️ ara-basic-ansible-core-2.17 SUCCESS in 5m 58s
✔️ ara-container-images SUCCESS in 9m 16s

Copy link

Build succeeded.
https://ansible.softwarefactory-project.io/zuul/buildset/4ffa8ae403534702a15f9c33ce3230df

✔️ ara-tox-py3 SUCCESS in 2m 36s
✔️ ara-tox-linters SUCCESS in 2m 26s
✔️ ara-basic-ansible-core-devel SUCCESS in 5m 44s (non-voting)
✔️ ara-basic-ansible-10 SUCCESS in 5m 36s
✔️ ara-basic-ansible-core-2.16 SUCCESS in 5m 26s
✔️ ara-basic-ansible-core-2.17 SUCCESS in 4m 56s
✔️ ara-container-images SUCCESS in 9m 22s

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant