From 3b52d968e678a9055a5b3e38185c889e1ddc027f Mon Sep 17 00:00:00 2001 From: Matt Davis Date: Fri, 21 Sep 2018 16:23:21 -0700 Subject: [PATCH] fix premature exit when RG doesn't exist (#46013) * fixes #45941 * corrects regression introduced by #26104; when the resource group doesn't exist, the module exits prematurely with an error instead of creating it. --- changelogs/fragments/azure_rm_deployment_fix_45941.yaml | 2 ++ lib/ansible/modules/cloud/azure/azure_rm_deployment.py | 3 ++- 2 files changed, 4 insertions(+), 1 deletion(-) create mode 100644 changelogs/fragments/azure_rm_deployment_fix_45941.yaml diff --git a/changelogs/fragments/azure_rm_deployment_fix_45941.yaml b/changelogs/fragments/azure_rm_deployment_fix_45941.yaml new file mode 100644 index 0000000000..bbfa8c7878 --- /dev/null +++ b/changelogs/fragments/azure_rm_deployment_fix_45941.yaml @@ -0,0 +1,2 @@ +bugfixes: +- azure_rm_deployment - fixed regression that prevents resource group from being created (https://github.com/ansible/ansible/issues/45941) diff --git a/lib/ansible/modules/cloud/azure/azure_rm_deployment.py b/lib/ansible/modules/cloud/azure/azure_rm_deployment.py index 43f09f0617..c8e777c9d0 100644 --- a/lib/ansible/modules/cloud/azure/azure_rm_deployment.py +++ b/lib/ansible/modules/cloud/azure/azure_rm_deployment.py @@ -491,7 +491,8 @@ class AzureRMDeploymentManager(AzureRMModuleBase): if self.append_tags and self.tags: try: - rg = self.get_resource_group(self.resource_group_name) + # fetch the RG directly (instead of using the base helper) since we don't want to exit if it's missing + rg = self.rm_client.resource_groups.get(self.resource_group_name) if rg.tags: self.tags = dict(self.tags, **rg.tags) except CloudError: