-
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
Restore BOOLEANS import in basic.py #42008
Conversation
Maybe add a comment that BOOLENS is there for backwards compatibility. Otherwise +1 to merge. Needs to be backported to stable-2.6 and stable-2.5 |
Closing and re-opening for CI trigger. |
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.
+1
Code looks good. We need to have a changelog entry for this as well. Something like restore module_utils.basic.BOOLEANS variable for backwards compatibility with the module API in older ansible releases. This was mistakenly removed in 2.5.0. |
This import was removed by mistake. This is required for backward compatibility. Fixes: ansible#41988 Signed-off-by: Abhijeet Kasurde <akasurde@redhat.com>
96f9efe
to
d83be40
Compare
rebuild_merge |
This import was removed by mistake. This is required for backward compatibility. Fixes: ansible#41988 Signed-off-by: Abhijeet Kasurde <akasurde@redhat.com>
This import was removed by mistake. This is required for backward compatibility. Fixes: ansible#41988 Signed-off-by: Abhijeet Kasurde <akasurde@redhat.com>
This import was removed by mistake. This is required for backward compatibility. Fixes: #41988 Signed-off-by: Abhijeet Kasurde <akasurde@redhat.com>
SUMMARY
This import was removed by mistake. This is required for backward
compatibility.
Fixes: #41988
Signed-off-by: Abhijeet Kasurde akasurde@redhat.com
ISSUE TYPE
COMPONENT NAME
lib/ansible/module_utils/basic.py
ANSIBLE VERSION