-
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
added diff as new keyword #28581
added diff as new keyword #28581
Conversation
@@ -242,11 +242,9 @@ class PlayContext(Base): | |||
_verbosity = FieldAttribute(isa='int', default=0) | |||
_only_tags = FieldAttribute(isa='set', default=set()) | |||
_skip_tags = FieldAttribute(isa='set', default=set()) | |||
_check_mode = FieldAttribute(isa='bool', default=False) |
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 assume this is being removed because it's redundant with the one in lib/ansible/playbook/base.py
, but should the default be moved to that file?
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.
yes, was redundant and no, as we check for None in task, which means its 'not set' vs True/False which would override the config and cli options
added diff to fake options in ec2_vpc_nat_gateway test
Can we get this merged ASAP ? We need it for doing integration tests in the XML module, so the tests provide proper guidance to why a test failed. |
A leap of faith, I trust @bcoca ! |
added diff to fake options in ec2_vpc_nat_gateway test
Tasks that deal with secrets may leak sensitive information when running in Check Mode. This change updates the documentation explaining that the diff can be deactivated on task level. The feature was requested in ansible#14860 and got introduced in Ansible 2.4 with ansible#28581.
* docs: Document disabling diff on task level Tasks that deal with secrets may leak sensitive information when running in Check Mode. This change updates the documentation explaining that the diff can be deactivated on task level. The feature was requested in #14860 and got introduced in Ansible 2.4 with #28581. * Updated for clarity
* docs: Document disabling diff on task level Tasks that deal with secrets may leak sensitive information when running in Check Mode. This change updates the documentation explaining that the diff can be deactivated on task level. The feature was requested in #14860 and got introduced in Ansible 2.4 with #28581. * Updated for clarity (cherry picked from commit 9cca9a9)
* docs: Document disabling diff on task level Tasks that deal with secrets may leak sensitive information when running in Check Mode. This change updates the documentation explaining that the diff can be deactivated on task level. The feature was requested in ansible#14860 and got introduced in Ansible 2.4 with ansible#28581. * Updated for clarity
SUMMARY
allows to override diff config/cli options in play
ISSUE TYPE
COMPONENT NAME
core
ANSIBLE VERSION