-
Notifications
You must be signed in to change notification settings - Fork 3.4k
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
Update metadata for the AWX collection #6016
Update metadata for the AWX collection #6016
Conversation
|
||
- Removing tower-cli as a dependency | ||
- Renaming the modules, for example `tower_organization` to just `organization` | ||
and giving a deprecation period for the switch |
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.
This isn't really on the agenda now (we could revisit it, but I wouldn't say the decision is made)
and this isn't the best place, we should handle these in issues.
I actually made this with the specific intention of merging before #5747 that makes some changes to the README.md but shouldn't conflict. |
Build succeeded.
|
12c6c27
to
d62994e
Compare
Build succeeded.
|
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.
👍
Build succeeded (gate pipeline).
|
SUMMARY
We know a lot more now than we did when this was written. This is just trying to update some of the meta stuff to the current reality.
ISSUE TYPE
COMPONENT NAME
AWX VERSION
ADDITIONAL INFORMATION
Tested as
alancoding.awx:9.2.1
https://galaxy.ansible.com/alancoding/awx
The playbook changes are to make this much much more compact:
Additionally, this allows me to run the sanity tests as alancoding.awx.
Also, here's testing for this new target:
with
and if I put in an exception to the file:
This doesn't work with
ansible-test
, and it might not even work forever in this form. But nothing lasts forever anyway.