-
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
changed if logic Type to boolean #49628
changed if logic Type to boolean #49628
Conversation
There is need some tests for me |
Also change the default value for this parameter to |
@samdoiron @abadger could you pls review again ? |
@Alexander198961 Wrong ping 😄. @samdoran ^^ |
Sorry |
Please create a changelog fragment. See fragments for examples. |
@samdoran Done |
Please create a backport PR for this to be included in previous versions. |
The previous version (stable-2.7) seems doesn't have such check |
* changed Type to boolean * fix after review * created fragment
SUMMARY
ISSUE TYPE
COMPONENT NAME
s3_module
ADDITIONAL INFORMATION
We have requester_pays=dict(default='no', type='bool') variable and seems it can be True or False otherwise. not sure is it possible to have boolean type as None ?
but we tried to check against is None type before fix which seems to be not right ?
Ansible version
ansible 2.8.0.dev0