-
Notifications
You must be signed in to change notification settings - Fork 39
add CRD for tower modules (#42) #43
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
base: devel
Are you sure you want to change the base?
Conversation
|
needs testing and review |
| name: | ||
| description: | | ||
| List of modules available: | ||
| https://docs.ansible.com/ansible/latest/collections/awx/awx/index.html#modules |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think the collection module changed. The prefix tower_ is no longer need.
|
What's the goal with this? Are you basically creating a wrapper around all of the existing functionality in awx.awx ? |
|
@the-it-jaeger @rooftopcellist I'm wondering if this should be archived as it is a pretty old PR at this point |
|
I think the idea behind this PR is to just wrap all of the awx.awx modules' functionality in one fell swoop rather than sort of implementing around each one individually. It might be a genius approach, but I haven't figured out how it consumes parameters yet. I wish the author would chime in. |
|
Hello... this PR creates CRDs for AWX.AWX modules, and can be used to whatever module available. You can refer to the following repo in order to get an idea of what was behind this PR: https://github.com/victorock/ansible-operator-tower-config. The individual implementations are based on a role that templatize the yaml task file for whatever module: https://github.com/victorock/ansible-operator-tower-config/tree/main/roles/module In particular: https://github.com/victorock/ansible-operator-tower-config/blob/main/roles/module/tasks/module.yml Let me know if it worth to update this PR based on what is proposed. |
No description provided.