From 815ef4c9c9fed4455cc8b151a9f28919763d6e45 Mon Sep 17 00:00:00 2001 From: Nico Ohnezat Date: Fri, 8 Oct 2021 23:30:07 +0200 Subject: [PATCH] related #11229 consider previous set json_output changed in controller_api job_template module sets self.json_output['changed'] to true before calling create_or_update_if_needed. Signed-off-by: Nico Ohnezat --- awx_collection/plugins/module_utils/controller_api.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/awx_collection/plugins/module_utils/controller_api.py b/awx_collection/plugins/module_utils/controller_api.py index ba542f00fe..21562602a5 100644 --- a/awx_collection/plugins/module_utils/controller_api.py +++ b/awx_collection/plugins/module_utils/controller_api.py @@ -909,7 +909,7 @@ class ControllerAPIModule(ControllerModule): if response['status_code'] == 200: # compare apples-to-apples, old API data to new API data # but do so considering the fields given in parameters - self.json_output['changed'] = self.objects_could_be_different(existing_item, response['json'], field_set=new_item.keys(), warning=True) + self.json_output['changed'] |= self.objects_could_be_different(existing_item, response['json'], field_set=new_item.keys(), warning=True) elif 'json' in response and '__all__' in response['json']: self.fail_json(msg=response['json']['__all__']) else: