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
azure_rm_rg_info: fix KeyError: 'ansible_facts' #66729
azure_rm_rg_info: fix KeyError: 'ansible_facts' #66729
Conversation
Fix facts for using old azure_rm_resourcegroup_facts, closes #66727
The test
|
@derkoe, just so you are aware we have a dedicated Working Group for azure. |
ready_for_review |
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.
Looks good to me. You need a changelog fragment, though.
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
Thanks for fixing and reviewing this! Who wants to create a backport for stable-2.9? |
@haiyuazhang Can you submit backport PR to back up this change to version 2.9? Thank you very much |
here you go #67479 |
Fix facts for using old azure_rm_resourcegroup_facts,
SUMMARY
Fixes #66727
Adds the
ansible_facts
as a sub map to fix the KeyErrorISSUE TYPE
COMPONENT NAME
azure / azure_rm_resourcegroup_info
ADDITIONAL INFORMATION
None