-
Notifications
You must be signed in to change notification settings - Fork 23.7k
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
aci_tenant: Update to support check_mode #28090
Conversation
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.
LGTM. As I already reviewed this in https://github.com/datacenter/aci-ansible
The integration tests require ACI infrastructure, but we are going to add them as soon as possible in a form acceptable to this project. Working on it. |
@privateip @devarshishah3 @jedelman8 @calfonso Your feedback is appreciated. As this module is the first module in a larger set, so any feedback we can take into account for the other modules. |
shipit |
shipit |
SUMMARY
Updates to aci_tenant to add support for check_mode.
This fixes issues raised in #26836
ISSUE TYPE
COMPONENT NAME
aci_tenant
ANSIBLE VERSION