-
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
Fixes #38787: azure_rm_resourcegroup idempotency broke when specifying location #40055
Conversation
Try the PR by using devel branch |
@@ -162,7 +162,7 @@ def exec_module(self, **kwargs): | |||
if update_tags: | |||
changed = True | |||
|
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 was wondering, perhaps we could normalize location before, e.g.:
self.location = normalize_location_name(self.location)
but just a suggestion. not sure perhaps that would have some disadvantages...
SUMMARY
User can set location as the display name, but it will break the idempotent,
Fixes #38787
ISSUE TYPE
COMPONENT NAME
azure_rm-resourcegroup.py
ANSIBLE VERSION
ADDITIONAL INFORMATION