-
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
Support diff mode in awx.awx.* modules #14903
Comments
@sean-m-sullivan do have some opinions on diff mode support? I realize check mode probably doesn't work very well across all awx modules, but diff (without check) probably has a better chance of being consistent and useful |
I think a diff mode might be possible by reading the the api and determining if there is a change, and could be useful as well. it depends on how we want to do it, but could compare new fields to existing fields dict where they match in most modules. |
@sean-m-sullivan @fosterseth please have a look to my PR: #15118 We are using this in our custom Ansible EE and it works perfectly. |
Please confirm the following
Feature type
New Feature
Feature Summary
During the playbook development process, we heavily use the
--check
and--diff
modes to verify what will be changed on the controller before applying the configuration. However, as theawx.awx.*
modules do not yet support the diff mode, the playbook output is not helpful during a dry run.Select the relevant components
Steps to reproduce
n/a
Current results
n/a
Sugested feature result
show diff output
Additional information
We use
infra.controller_configuration
collection to configure all AWX resources (GitOps). Therefore, a dry run with proper diff output would be very helpful in detecting issues that may potentially destroy the entire AWX instance.The text was updated successfully, but these errors were encountered: