From b9111860245e2f831f4e1552f2d71ac4e5024b5e Mon Sep 17 00:00:00 2001 From: Matt Davis Date: Tue, 21 Nov 2017 17:21:24 -0800 Subject: [PATCH] Ensure that readonly result members are serialized (#33170) * fix for breaking metadata change in various Azure Python SDK bits; some members were marked `readonly` for validation, which the default msrest serializer ignores. Added `keep_readonly` flag to serializer call to ensure they're preserved. (cherry picked from commit 70e351036dfdeb0c862db2e642085a648e23a47f) --- CHANGELOG.md | 2 ++ lib/ansible/module_utils/azure_rm_common.py | 2 +- 2 files changed, 3 insertions(+), 1 deletion(-) diff --git a/CHANGELOG.md b/CHANGELOG.md index e9b1a77ce02..9cfd2f8d8c2 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -188,6 +188,8 @@ Ansible Changes By Release https://github.com/ansible/ansible/pull/32990 * Fix for breaking change to Azure Python SDK DNS RecordSet constructor in azure-mgmt-dns==1.2.0 https://github.com/ansible/ansible/pull/33165 +* Fix for breaking change to Azure Python SDK that prevented some members from being returned in facts modules + https://github.com/ansible/ansible/pull/33169 diff --git a/lib/ansible/module_utils/azure_rm_common.py b/lib/ansible/module_utils/azure_rm_common.py index 8c234bfdd48..24626fd2e58 100644 --- a/lib/ansible/module_utils/azure_rm_common.py +++ b/lib/ansible/module_utils/azure_rm_common.py @@ -508,7 +508,7 @@ class AzureRMModuleBase(object): self.log("dependencies: ") self.log(str(dependencies)) serializer = Serializer(classes=dependencies) - return serializer.body(obj, class_name) + return serializer.body(obj, class_name, keep_readonly=True) def get_poller_result(self, poller, wait=5): '''