Bug fixes for GCP modules (#60721)

lib/ansible/modules/cloud/google/gcp_compute_instance_template.py

@@ -1117,7 +1117,11 @@ def wait_for_operation(module, response):
         return {}
     status = navigate_hash(op_result, ['status'])
     wait_done = wait_for_completion(status, op_result, module)
-    return decode_response(fetch_resource(module, navigate_hash(wait_done, ['targetLink']), 'compute#instanceTemplate'), module)
+    response = fetch_resource(module, navigate_hash(wait_done, ['targetLink']), 'compute#instanceTemplate')
+    if response:
+        return decode_response(response, module)
+    else:
+        return {}
 
 
 def wait_for_completion(status, op_result, module):

lib/ansible/modules/cloud/google/gcp_compute_instance_template_info.py

@@ -488,12 +488,7 @@ def main():
     if not module.params['scopes']:
         module.params['scopes'] = ['https://www.googleapis.com/auth/compute']
 
-    items = fetch_list(module, collection(module), query_options(module.params['filters']))
+    return_value = {'resources': fetch_list(module, collection(module), query_options(module.params['filters']))}
-    if items.get('items'):
-        items = items.get('items')
-    else:
-        items = []
-    return_value = {'resources': items}
     module.exit_json(**return_value)
 
 
@@ -503,8 +498,7 @@ def collection(module):
 
 def fetch_list(module, link, query):
     auth = GcpSession(module, 'compute')
-    response = auth.get(link, params={'filter': query})
+    return auth.list(link, return_if_object, array_name='items', params={'filter': query})
-    return return_if_object(module, response)
 
 
 def query_options(filters):

lib/ansible/modules/cloud/google/gcp_compute_interconnect_attachment_info.py

@@ -225,12 +225,7 @@ def main():
     if not module.params['scopes']:
         module.params['scopes'] = ['https://www.googleapis.com/auth/compute']
 
-    items = fetch_list(module, collection(module), query_options(module.params['filters']))
+    return_value = {'resources': fetch_list(module, collection(module), query_options(module.params['filters']))}
-    if items.get('items'):
-        items = items.get('items')
-    else:
-        items = []
-    return_value = {'resources': items}
     module.exit_json(**return_value)
 
 
@@ -240,8 +235,7 @@ def collection(module):
 
 def fetch_list(module, link, query):
     auth = GcpSession(module, 'compute')
-    response = auth.get(link, params={'filter': query})
+    return auth.list(link, return_if_object, array_name='items', params={'filter': query})
-    return return_if_object(module, response)
 
 
 def query_options(filters):

lib/ansible/modules/cloud/google/gcp_compute_network_info.py

@@ -162,12 +162,7 @@ def main():
     if not module.params['scopes']:
         module.params['scopes'] = ['https://www.googleapis.com/auth/compute']
 
-    items = fetch_list(module, collection(module), query_options(module.params['filters']))
+    return_value = {'resources': fetch_list(module, collection(module), query_options(module.params['filters']))}
-    if items.get('items'):
-        items = items.get('items')
-    else:
-        items = []
-    return_value = {'resources': items}
     module.exit_json(**return_value)
 
 
@@ -177,8 +172,7 @@ def collection(module):
 
 def fetch_list(module, link, query):
     auth = GcpSession(module, 'compute')
-    response = auth.get(link, params={'filter': query})
+    return auth.list(link, return_if_object, array_name='items', params={'filter': query})
-    return return_if_object(module, response)
 
 
 def query_options(filters):

lib/ansible/modules/cloud/google/gcp_compute_region_disk_info.py

@@ -248,12 +248,7 @@ def main():
     if not module.params['scopes']:
         module.params['scopes'] = ['https://www.googleapis.com/auth/compute']
 
-    items = fetch_list(module, collection(module), query_options(module.params['filters']))
+    return_value = {'resources': fetch_list(module, collection(module), query_options(module.params['filters']))}
-    if items.get('items'):
-        items = items.get('items')
-    else:
-        items = []
-    return_value = {'resources': items}
     module.exit_json(**return_value)
 
 
@@ -263,8 +258,7 @@ def collection(module):
 
 def fetch_list(module, link, query):
     auth = GcpSession(module, 'compute')
-    response = auth.get(link, params={'filter': query})
+    return auth.list(link, return_if_object, array_name='items', params={'filter': query})
-    return return_if_object(module, response)
 
 
 def query_options(filters):

lib/ansible/modules/cloud/google/gcp_compute_route_info.py

@@ -162,12 +162,7 @@ def main():
     if not module.params['scopes']:
         module.params['scopes'] = ['https://www.googleapis.com/auth/compute']
 
-    items = fetch_list(module, collection(module), query_options(module.params['filters']))
+    return_value = {'resources': fetch_list(module, collection(module), query_options(module.params['filters']))}
-    if items.get('items'):
-        items = items.get('items')
-    else:
-        items = []
-    return_value = {'resources': items}
     module.exit_json(**return_value)
 
 
@@ -177,8 +172,7 @@ def collection(module):
 
 def fetch_list(module, link, query):
     auth = GcpSession(module, 'compute')
-    response = auth.get(link, params={'filter': query})
+    return auth.list(link, return_if_object, array_name='items', params={'filter': query})
-    return return_if_object(module, response)
 
 
 def query_options(filters):

lib/ansible/modules/cloud/google/gcp_compute_router_info.py

@@ -178,12 +178,7 @@ def main():
     if not module.params['scopes']:
         module.params['scopes'] = ['https://www.googleapis.com/auth/compute']
 
-    items = fetch_list(module, collection(module), query_options(module.params['filters']))
+    return_value = {'resources': fetch_list(module, collection(module), query_options(module.params['filters']))}
-    if items.get('items'):
-        items = items.get('items')
-    else:
-        items = []
-    return_value = {'resources': items}
     module.exit_json(**return_value)
 
 
@@ -193,8 +188,7 @@ def collection(module):
 
 def fetch_list(module, link, query):
     auth = GcpSession(module, 'compute')
-    response = auth.get(link, params={'filter': query})
+    return auth.list(link, return_if_object, array_name='items', params={'filter': query})
-    return return_if_object(module, response)
 
 
 def query_options(filters):

lib/ansible/modules/cloud/google/gcp_compute_snapshot_info.py

@@ -191,12 +191,7 @@ def main():
     if not module.params['scopes']:
         module.params['scopes'] = ['https://www.googleapis.com/auth/compute']
 
-    items = fetch_list(module, collection(module), query_options(module.params['filters']))
+    return_value = {'resources': fetch_list(module, collection(module), query_options(module.params['filters']))}
-    if items.get('items'):
-        items = items.get('items')
-    else:
-        items = []
-    return_value = {'resources': items}
     module.exit_json(**return_value)
 
 
@@ -206,8 +201,7 @@ def collection(module):
 
 def fetch_list(module, link, query):
     auth = GcpSession(module, 'compute')
-    response = auth.get(link, params={'filter': query})
+    return auth.list(link, return_if_object, array_name='items', params={'filter': query})
-    return return_if_object(module, response)
 
 
 def query_options(filters):

lib/ansible/modules/cloud/google/gcp_compute_ssl_certificate_info.py

@@ -126,12 +126,7 @@ def main():
     if not module.params['scopes']:
         module.params['scopes'] = ['https://www.googleapis.com/auth/compute']
 
-    items = fetch_list(module, collection(module), query_options(module.params['filters']))
+    return_value = {'resources': fetch_list(module, collection(module), query_options(module.params['filters']))}
-    if items.get('items'):
-        items = items.get('items')
-    else:
-        items = []
-    return_value = {'resources': items}
     module.exit_json(**return_value)
 
 
@@ -141,8 +136,7 @@ def collection(module):
 
 def fetch_list(module, link, query):
     auth = GcpSession(module, 'compute')
-    response = auth.get(link, params={'filter': query})
+    return auth.list(link, return_if_object, array_name='items', params={'filter': query})
-    return return_if_object(module, response)
 
 
 def query_options(filters):

lib/ansible/modules/cloud/google/gcp_compute_ssl_policy_info.py

@@ -164,12 +164,7 @@ def main():
     if not module.params['scopes']:
         module.params['scopes'] = ['https://www.googleapis.com/auth/compute']
 
-    items = fetch_list(module, collection(module), query_options(module.params['filters']))
+    return_value = {'resources': fetch_list(module, collection(module), query_options(module.params['filters']))}
-    if items.get('items'):
-        items = items.get('items')
-    else:
-        items = []
-    return_value = {'resources': items}
     module.exit_json(**return_value)
 
 
@@ -179,8 +174,7 @@ def collection(module):
 
 def fetch_list(module, link, query):
     auth = GcpSession(module, 'compute')
-    response = auth.get(link, params={'filter': query})
+    return auth.list(link, return_if_object, array_name='items', params={'filter': query})
-    return return_if_object(module, response)
 
 
 def query_options(filters):

lib/ansible/modules/cloud/google/gcp_compute_subnetwork_info.py

@@ -187,12 +187,7 @@ def main():
     if not module.params['scopes']:
         module.params['scopes'] = ['https://www.googleapis.com/auth/compute']
 
-    items = fetch_list(module, collection(module), query_options(module.params['filters']))
+    return_value = {'resources': fetch_list(module, collection(module), query_options(module.params['filters']))}
-    if items.get('items'):
-        items = items.get('items')
-    else:
-        items = []
-    return_value = {'resources': items}
     module.exit_json(**return_value)
 
 
@@ -202,8 +197,7 @@ def collection(module):
 
 def fetch_list(module, link, query):
     auth = GcpSession(module, 'compute')
-    response = auth.get(link, params={'filter': query})
+    return auth.list(link, return_if_object, array_name='items', params={'filter': query})
-    return return_if_object(module, response)
 
 
 def query_options(filters):

lib/ansible/modules/cloud/google/gcp_compute_target_http_proxy_info.py

@@ -120,12 +120,7 @@ def main():
     if not module.params['scopes']:
         module.params['scopes'] = ['https://www.googleapis.com/auth/compute']
 
-    items = fetch_list(module, collection(module), query_options(module.params['filters']))
+    return_value = {'resources': fetch_list(module, collection(module), query_options(module.params['filters']))}
-    if items.get('items'):
-        items = items.get('items')
-    else:
-        items = []
-    return_value = {'resources': items}
     module.exit_json(**return_value)
 
 
@@ -135,8 +130,7 @@ def collection(module):
 
 def fetch_list(module, link, query):
     auth = GcpSession(module, 'compute')
-    response = auth.get(link, params={'filter': query})
+    return auth.list(link, return_if_object, array_name='items', params={'filter': query})
-    return return_if_object(module, response)
 
 
 def query_options(filters):