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

add support for ansible action_groups #138

Merged
merged 1 commit into from
Feb 12, 2024

Conversation

netsandbox
Copy link
Contributor

see https://docs.ansible.com/ansible/latest/dev_guide/developing_collections_structure.html#meta-directory-and-runtime-yml

The list was generated with:

find plugins/modules/ -type f | xargs -n1 basename -s .py | sort | awk '$0="    - "$0'

@mdellweg
Copy link
Member

mdellweg commented Dec 5, 2023

How does that work (what is it for)? And can we find a way to automate generating this list (and add a lint step to make sure it is up to date)?

@netsandbox
Copy link
Contributor Author

How does that work (what is it for)?

You can define module defaults for all modules in this collection: https://docs.ansible.com/ansible/latest/playbook_guide/playbooks_module_defaults.html#module-defaults-groups

And can we find a way to automate generating this list (and add a lint step to make sure it is up to date)?

I currently have no idea how to do this, but I'm open for any suggestion.

@mdellweg
Copy link
Member

mdellweg commented Dec 5, 2023

I'll try to take a look.

@netsandbox
Copy link
Contributor Author

@mdellweg I have added a basic check that the action_groups list is in sync with the modules.

@mdellweg
Copy link
Member

Thank you. If i find the time, maybe I will rewrite it into a script that can both generate an d check this list. ;)

@mdellweg mdellweg merged commit a6426e2 into pulp:develop Feb 12, 2024
12 checks passed
@netsandbox netsandbox deleted the ansible_action_groups branch March 5, 2024 13:03
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.

2 participants