-
Notifications
You must be signed in to change notification settings - Fork 23.8k
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
checks if tags_dict exists and has data before using it. #63169
Conversation
From what I can see CI failure is not related to the PR. @ffreitasalves Would you be able to add an integration test for this change? /cc @jillr |
It looks like the error message changed again - we'd modified the assertion in |
Will do. |
It would be good to get this patch merged, would you still be able to add integration tests to cover this case @ffreitasalves? CI is passing. |
Thank you very much for your interest in Ansible. Ansible has migrated much of the content into separate repositories to allow for more rapid, independent development. We are closing this issue/PR because this content has been moved to one or more collection repositories.
For further information, please see: |
SUMMARY
Checks if tags_dict was provided either as an empty dictionary or not provided at all (NoneType) before trying to access its items
Fixes #58821
ISSUE TYPE
COMPONENT NAME
ec2 module_utils