tests: fixing 7 tests #505

Fixing additional tests, primarily adjusting to new GCP behavior.

- gcp_appengine_firewall_rule
- gcp_cloudfunctions_cloud_function
- gcp_compute_external_backend_service
- gcp_compute_forwarding_rule
- gcp_compute_route
- gcp_compute_router
- gcp_compute_region_url_map

raising the number to 29/64 tests.

plugins/modules/gcp_cloudfunctions_cloud_function.py

@@ -25,9 +25,13 @@ __metaclass__ = type
 # Documentation
 ################################################################################
 
-ANSIBLE_METADATA = {'metadata_version': '1.1', 'status': ["preview"], 'supported_by': 'community'}
+ANSIBLE_METADATA = {
+    "metadata_version": "1.1",
+    "status": ["preview"],
+    "supported_by": "community",
+}
 
-DOCUMENTATION = '''
+DOCUMENTATION = """
 ---
 module: gcp_cloudfunctions_cloud_function
 description:
@@ -69,8 +73,8 @@ options:
     type: str
   runtime:
     description:
-    - The runtime in which the function is going to run. If empty, defaults to Node.js
+    - The runtime in which to run the function. Required when deploying a new function,
-      6.
+      optional when updating an existing function.
     required: false
     type: str
   timeout:
@@ -195,9 +199,9 @@ options:
     - This should not be set unless you know what you're doing.
     - This only alters the User Agent string for any API requests.
     type: str
-'''
+"""
 
-EXAMPLES = '''
+EXAMPLES = """
 - name: create a cloud function
   google.cloud.gcp_cloudfunctions_cloud_function:
     name: test_object
@@ -209,9 +213,9 @@ EXAMPLES = '''
     auth_kind: serviceaccount
     service_account_file: "/tmp/auth.pem"
     state: present
-'''
+"""
 
-RETURN = '''
+RETURN = """
 name:
   description:
   - A user-defined name of the function. Function names must be unique globally and
@@ -353,7 +357,7 @@ trigger_http:
   - Use HTTP to trigger this function.
   returned: success
   type: bool
-'''
+"""
 
 ################################################################################
 # Imports
@@ -381,43 +385,50 @@ def main():
 
     module = GcpModule(
         argument_spec=dict(
-            state=dict(default='present', choices=['present', 'absent'], type='str'),
+            state=dict(default="present", choices=["present", "absent"], type="str"),
-            name=dict(required=True, type='str'),
+            name=dict(required=True, type="str"),
-            description=dict(type='str'),
+            description=dict(type="str"),
-            entry_point=dict(type='str'),
+            entry_point=dict(type="str"),
-            runtime=dict(type='str'),
+            runtime=dict(type="str"),
-            timeout=dict(type='str'),
+            timeout=dict(type="str"),
-            available_memory_mb=dict(type='int'),
+            available_memory_mb=dict(type="int"),
-            labels=dict(type='dict'),
+            labels=dict(type="dict"),
-            environment_variables=dict(type='dict'),
+            environment_variables=dict(type="dict"),
-            source_archive_url=dict(type='str'),
+            source_archive_url=dict(type="str"),
-            source_upload_url=dict(type='str'),
+            source_upload_url=dict(type="str"),
-            source_repository=dict(type='dict', options=dict(url=dict(required=True, type='str'))),
+            source_repository=dict(
-            https_trigger=dict(type='dict', options=dict()),
+                type="dict", options=dict(url=dict(required=True, type="str"))
-            event_trigger=dict(
-                type='dict', options=dict(event_type=dict(required=True, type='str'), resource=dict(required=True, type='str'), service=dict(type='str'))
             ),
-            location=dict(required=True, type='str'),
+            https_trigger=dict(type="dict", options=dict()),
-            trigger_http=dict(type='bool'),
+            event_trigger=dict(
+                type="dict",
+                options=dict(
+                    event_type=dict(required=True, type="str"),
+                    resource=dict(required=True, type="str"),
+                    service=dict(type="str"),
+                ),
+            ),
+            location=dict(required=True, type="str"),
+            trigger_http=dict(type="bool"),
         )
     )
 
-    if not module.params['scopes']:
+    if not module.params["scopes"]:
-        module.params['scopes'] = ['https://www.googleapis.com/auth/cloud-platform']
+        module.params["scopes"] = ["https://www.googleapis.com/auth/cloud-platform"]
 
-    state = module.params['state']
+    state = module.params["state"]
 
     fetch = fetch_resource(module, self_link(module))
     changed = False
 
     # Need to set triggerHttps to {} if boolean true.
-    if fetch and fetch.get('httpsTrigger') and module.params['trigger_http']:
+    if fetch and fetch.get("httpsTrigger") and module.params["trigger_http"]:
-        module.params['https_trigger'] = fetch.get('httpsTrigger')
+        module.params["https_trigger"] = fetch.get("httpsTrigger")
-    elif module.params['trigger_http']:
+    elif module.params["trigger_http"]:
-        module.params['https_trigger'] = {}
+        module.params["https_trigger"] = {}
 
     if fetch:
-        if state == 'present':
+        if state == "present":
             if is_different(module, fetch):
                 update(module, self_link(module), fetch)
                 fetch = fetch_resource(module, self_link(module))
@@ -427,101 +438,115 @@ def main():
             fetch = {}
             changed = True
     else:
-        if state == 'present':
+        if state == "present":
             fetch = create(module, collection(module))
             changed = True
         else:
             fetch = {}
 
-    fetch.update({'changed': changed})
+    fetch.update({"changed": changed})
 
     module.exit_json(**fetch)
 
 
 def create(module, link):
-    auth = GcpSession(module, 'cloudfunctions')
+    auth = GcpSession(module, "cloudfunctions")
     return wait_for_operation(module, auth.post(link, resource_to_request(module)))
 
 
 def update(module, link, fetch):
-    auth = GcpSession(module, 'cloudfunctions')
+    auth = GcpSession(module, "cloudfunctions")
-    params = {'updateMask': updateMask(resource_to_request(module), response_to_hash(module, fetch))}
+    params = {
+        "updateMask": updateMask(
+            resource_to_request(module), response_to_hash(module, fetch)
+        )
+    }
     request = resource_to_request(module)
-    del request['name']
+    del request["name"]
     return wait_for_operation(module, auth.put(link, request, params=params))
 
 
 def updateMask(request, response):
     update_mask = []
-    if request.get('name') != response.get('name'):
+    if request.get("name") != response.get("name"):
-        update_mask.append('name')
+        update_mask.append("name")
-    if request.get('description') != response.get('description'):
+    if request.get("description") != response.get("description"):
-        update_mask.append('description')
+        update_mask.append("description")
-    if request.get('entryPoint') != response.get('entryPoint'):
+    if request.get("entryPoint") != response.get("entryPoint"):
-        update_mask.append('entryPoint')
+        update_mask.append("entryPoint")
-    if request.get('runtime') != response.get('runtime'):
+    if request.get("runtime") != response.get("runtime"):
-        update_mask.append('runtime')
+        update_mask.append("runtime")
-    if request.get('timeout') != response.get('timeout'):
+    if request.get("timeout") != response.get("timeout"):
-        update_mask.append('timeout')
+        update_mask.append("timeout")
-    if request.get('availableMemoryMb') != response.get('availableMemoryMb'):
+    if request.get("availableMemoryMb") != response.get("availableMemoryMb"):
-        update_mask.append('availableMemoryMb')
+        update_mask.append("availableMemoryMb")
-    if request.get('labels') != response.get('labels'):
+    if request.get("labels") != response.get("labels"):
-        update_mask.append('labels')
+        update_mask.append("labels")
-    if request.get('environmentVariables') != response.get('environmentVariables'):
+    if request.get("environmentVariables") != response.get("environmentVariables"):
-        update_mask.append('environmentVariables')
+        update_mask.append("environmentVariables")
-    if request.get('sourceArchiveUrl') != response.get('sourceArchiveUrl'):
+    if request.get("sourceArchiveUrl") != response.get("sourceArchiveUrl"):
-        update_mask.append('sourceArchiveUrl')
+        update_mask.append("sourceArchiveUrl")
-    if request.get('sourceUploadUrl') != response.get('sourceUploadUrl'):
+    if request.get("sourceUploadUrl") != response.get("sourceUploadUrl"):
-        update_mask.append('sourceUploadUrl')
+        update_mask.append("sourceUploadUrl")
-    if request.get('sourceRepository') != response.get('sourceRepository'):
+    if request.get("sourceRepository") != response.get("sourceRepository"):
-        update_mask.append('sourceRepository')
+        update_mask.append("sourceRepository")
-    if request.get('httpsTrigger') != response.get('httpsTrigger'):
+    if request.get("httpsTrigger") != response.get("httpsTrigger"):
-        update_mask.append('httpsTrigger')
+        update_mask.append("httpsTrigger")
-    if request.get('eventTrigger') != response.get('eventTrigger'):
+    if request.get("eventTrigger") != response.get("eventTrigger"):
-        update_mask.append('eventTrigger')
+        update_mask.append("eventTrigger")
-    if request.get('location') != response.get('location'):
+    if request.get("location") != response.get("location"):
-        update_mask.append('location')
+        update_mask.append("location")
-    if request.get('trigger_http') != response.get('trigger_http'):
+    if request.get("trigger_http") != response.get("trigger_http"):
-        update_mask.append('trigger_http')
+        update_mask.append("trigger_http")
-    return ','.join(update_mask)
+    return ",".join(update_mask)
 
 
 def delete(module, link):
-    auth = GcpSession(module, 'cloudfunctions')
+    auth = GcpSession(module, "cloudfunctions")
     return wait_for_operation(module, auth.delete(link))
 
 
 def resource_to_request(module):
     request = {
-        u'name': name_pattern(module.params.get('name'), module),
+        "name": name_pattern(module.params.get("name"), module),
-        u'description': module.params.get('description'),
+        "description": module.params.get("description"),
-        u'entryPoint': module.params.get('entry_point'),
+        "entryPoint": module.params.get("entry_point"),
-        u'runtime': module.params.get('runtime'),
+        "runtime": module.params.get("runtime"),
-        u'timeout': module.params.get('timeout'),
+        "timeout": module.params.get("timeout"),
-        u'availableMemoryMb': module.params.get('available_memory_mb'),
+        "availableMemoryMb": module.params.get("available_memory_mb"),
-        u'labels': module.params.get('labels'),
+        "labels": module.params.get("labels"),
-        u'environmentVariables': module.params.get('environment_variables'),
+        "environmentVariables": module.params.get("environment_variables"),
-        u'sourceArchiveUrl': module.params.get('source_archive_url'),
+        "sourceArchiveUrl": module.params.get("source_archive_url"),
-        u'sourceUploadUrl': module.params.get('source_upload_url'),
+        "sourceUploadUrl": module.params.get("source_upload_url"),
-        u'sourceRepository': CloudFunctionSourcerepository(module.params.get('source_repository', {}), module).to_request(),
+        "sourceRepository": CloudFunctionSourcerepository(
-        u'httpsTrigger': CloudFunctionHttpstrigger(module.params.get('https_trigger', {}), module).to_request(),
+            module.params.get("source_repository", {}), module
-        u'eventTrigger': CloudFunctionEventtrigger(module.params.get('event_trigger', {}), module).to_request(),
+        ).to_request(),
+        "httpsTrigger": CloudFunctionHttpstrigger(
+            module.params.get("https_trigger", {}), module
+        ).to_request(),
+        "eventTrigger": CloudFunctionEventtrigger(
+            module.params.get("event_trigger", {}), module
+        ).to_request(),
     }
     request = encode_request(request, module)
     return request
 
 
 def fetch_resource(module, link, allow_not_found=True):
-    auth = GcpSession(module, 'cloudfunctions')
+    auth = GcpSession(module, "cloudfunctions")
     return return_if_object(module, auth.get(link), allow_not_found)
 
 
 def self_link(module):
-    return "https://cloudfunctions.googleapis.com/v1/projects/{project}/locations/{location}/functions/{name}".format(**module.params)
+    return "https://cloudfunctions.googleapis.com/v1/projects/{project}/locations/{location}/functions/{name}".format(
+        **module.params
+    )
 
 
 def collection(module):
-    return "https://cloudfunctions.googleapis.com/v1/projects/{project}/locations/{location}/functions".format(**module.params)
+    return "https://cloudfunctions.googleapis.com/v1/projects/{project}/locations/{location}/functions".format(
+        **module.params
+    )
 
 
 def return_if_object(module, response, allow_not_found=False):
@@ -536,11 +561,11 @@ def return_if_object(module, response, allow_not_found=False):
     try:
         module.raise_for_status(response)
         result = response.json()
-    except getattr(json.decoder, 'JSONDecodeError', ValueError):
+    except getattr(json.decoder, "JSONDecodeError", ValueError):
         module.fail_json(msg="Invalid JSON response with error: %s" % response.text)
 
-    if navigate_hash(result, ['error', 'errors']):
+    if navigate_hash(result, ["error", "errors"]):
-        module.fail_json(msg=navigate_hash(result, ['error', 'errors']))
+        module.fail_json(msg=navigate_hash(result, ["error", "errors"]))
 
     return result
 
@@ -567,23 +592,29 @@ def is_different(module, response):
 # This is for doing comparisons with Ansible's current parameters.
 def response_to_hash(module, response):
     return {
-        u'name': response.get(u'name'),
+        "name": response.get("name"),
-        u'description': response.get(u'description'),
+        "description": response.get("description"),
-        u'status': response.get(u'status'),
+        "status": response.get("status"),
-        u'entryPoint': response.get(u'entryPoint'),
+        "entryPoint": response.get("entryPoint"),
-        u'runtime': response.get(u'runtime'),
+        "runtime": response.get("runtime"),
-        u'timeout': response.get(u'timeout'),
+        "timeout": response.get("timeout"),
-        u'availableMemoryMb': response.get(u'availableMemoryMb'),
+        "availableMemoryMb": response.get("availableMemoryMb"),
-        u'serviceAccountEmail': response.get(u'serviceAccountEmail'),
+        "serviceAccountEmail": response.get("serviceAccountEmail"),
-        u'updateTime': response.get(u'updateTime'),
+        "updateTime": response.get("updateTime"),
-        u'versionId': response.get(u'versionId'),
+        "versionId": response.get("versionId"),
-        u'labels': response.get(u'labels'),
+        "labels": response.get("labels"),
-        u'environmentVariables': response.get(u'environmentVariables'),
+        "environmentVariables": response.get("environmentVariables"),
-        u'sourceArchiveUrl': response.get(u'sourceArchiveUrl'),
+        "sourceArchiveUrl": response.get("sourceArchiveUrl"),
-        u'sourceUploadUrl': response.get(u'sourceUploadUrl'),
+        "sourceUploadUrl": response.get("sourceUploadUrl"),
-        u'sourceRepository': CloudFunctionSourcerepository(response.get(u'sourceRepository', {}), module).from_response(),
+        "sourceRepository": CloudFunctionSourcerepository(
-        u'httpsTrigger': CloudFunctionHttpstrigger(response.get(u'httpsTrigger', {}), module).from_response(),
+            response.get("sourceRepository", {}), module
-        u'eventTrigger': CloudFunctionEventtrigger(response.get(u'eventTrigger', {}), module).from_response(),
+        ).from_response(),
+        "httpsTrigger": CloudFunctionHttpstrigger(
+            response.get("httpsTrigger", {}), module
+        ).from_response(),
+        "eventTrigger": CloudFunctionEventtrigger(
+            response.get("eventTrigger", {}), module
+        ).from_response(),
     }
 
 
@@ -594,7 +625,9 @@ def name_pattern(name, module):
     regex = r"projects/.*/locations/.*/functions/.*"
 
     if not re.match(regex, name):
-        name = "projects/{project}/locations/{location}/functions/{name}".format(**module.params)
+        name = "projects/{project}/locations/{location}/functions/{name}".format(
+            **module.params
+        )
 
     return name
 
@@ -612,20 +645,20 @@ def wait_for_operation(module, response):
     op_result = return_if_object(module, response)
     if op_result is None:
         return {}
-    status = navigate_hash(op_result, ['done'])
+    status = navigate_hash(op_result, ["done"])
     wait_done = wait_for_completion(status, op_result, module)
-    raise_if_errors(wait_done, ['error'], module)
+    raise_if_errors(wait_done, ["error"], module)
-    return navigate_hash(wait_done, ['response'])
+    return navigate_hash(wait_done, ["response"])
 
 
 def wait_for_completion(status, op_result, module):
-    op_id = navigate_hash(op_result, ['name'])
+    op_id = navigate_hash(op_result, ["name"])
-    op_uri = async_op_url(module, {'op_id': op_id})
+    op_uri = async_op_url(module, {"op_id": op_id})
     while not status:
-        raise_if_errors(op_result, ['error'], module)
+        raise_if_errors(op_result, ["error"], module)
         time.sleep(1.0)
         op_result = fetch_resource(module, op_uri, False)
-        status = navigate_hash(op_result, ['done'])
+        status = navigate_hash(op_result, ["done"])
     return op_result
 
 
@@ -641,8 +674,8 @@ def encode_request(request, module):
         if v or v is False:
             return_vals[k] = v
 
-    if module.params['trigger_http'] and not return_vals.get('httpsTrigger'):
+    if module.params["trigger_http"] and not return_vals.get("httpsTrigger"):
-        return_vals['httpsTrigger'] = {}
+        return_vals["httpsTrigger"] = {}
 
     return return_vals
 
@@ -656,10 +689,10 @@ class CloudFunctionSourcerepository(object):
             self.request = {}
 
     def to_request(self):
-        return remove_nones_from_dict({u'url': self.request.get('url')})
+        return remove_nones_from_dict({"url": self.request.get("url")})
 
     def from_response(self):
-        return remove_nones_from_dict({u'url': self.request.get(u'url')})
+        return remove_nones_from_dict({"url": self.request.get("url")})
 
 
 class CloudFunctionHttpstrigger(object):
@@ -687,14 +720,22 @@ class CloudFunctionEventtrigger(object):
 
     def to_request(self):
         return remove_nones_from_dict(
-            {u'eventType': self.request.get('event_type'), u'resource': self.request.get('resource'), u'service': self.request.get('service')}
+            {
+                "eventType": self.request.get("event_type"),
+                "resource": self.request.get("resource"),
+                "service": self.request.get("service"),
+            }
         )
 
     def from_response(self):
         return remove_nones_from_dict(
-            {u'eventType': self.request.get(u'eventType'), u'resource': self.request.get(u'resource'), u'service': self.request.get(u'service')}
+            {
+                "eventType": self.request.get("eventType"),
+                "resource": self.request.get("resource"),
+                "service": self.request.get("service"),
+            }
         )
 
 
-if __name__ == '__main__':
+if __name__ == "__main__":
     main()

scripts/bootstrap-project.sh

@@ -11,6 +11,7 @@ SERVICE_ACCOUNT_NAME="${2}"
 SERVICE_LIST=(
     "appengine"
     "bigtable"
+    "cloudbuild.googleapis.com"
     "cloudfunctions"
     "cloudkms.googleapis.com"
     "cloudresourcemanager.googleapis.com"
@@ -50,11 +51,16 @@ if ! gcloud app describe --project="$PROJECT_ID" > /dev/null; then
     gcloud app create --project="$PROJECT_ID" --region=us-central
 fi
 
+# create and upload cloud function for testing
 
+BUCKET_NAME="gs://${PROJECT_ID}-ansible-testing"
 
-# Add bindings
+if ! gcloud storage buckets describe "${BUCKET_NAME}" > /dev/null; then
+    gcloud storage buckets create "${BUCKET_NAME}" --project="${PROJECT_ID}"
+fi
+
+gsutil cp ./test-fixtures/cloud-function.zip "${BUCKET_NAME}"
 
-# roles/storage.objectAdmin
 
 # The following is hard to automate, so echo
 echo "Done! It may take up to 10 minutes for some of the changes to fully propagate."

test-fixtures/cloud-function-source/build-function-zip.sh

@@ -0,0 +1,7 @@
+#!/usr/bin/env bash
+# Build the cloud function zip file,
+# in the desired cloud function source format.
+if [ -f ../cloud-function.zip ]; then
+    rm ../cloud-function.zip
+fi
+zip ../cloud-function.zip *

test-fixtures/cloud-function-source/main.py

@@ -0,0 +1,9 @@
+import functions_framework
+
+# Register an HTTP function with the Functions Framework
+@functions_framework.http
+def helloGET(request):
+    # Your code here
+
+    # Return an HTTP response
+    return "OK"

test-fixtures/cloud-function-source/requirements.txt

@@ -0,0 +1 @@
+functions-framework==3.*

tests/integration/targets/gcp_appengine_firewall_rule/aliases

@@ -1,2 +1 @@
-cloud/gcp
+cloud/gcp
-unsupported

tests/integration/targets/gcp_appengine_firewall_rule/tasks/autogen.yml

@@ -90,7 +90,9 @@
 - name: verify that command succeeded
   assert:
     that:
-      - results['resources'] | length == 0
+      # there is a default firewall rule that cannot be
+      # deleted, so the length should be 1.
+      - results['resources'] | length == 1
 # ----------------------------------------------------------------------------
 - name: delete a firewall rule that does not exist
   google.cloud.gcp_appengine_firewall_rule:

tests/integration/targets/gcp_cloudfunctions_cloud_function/aliases

@@ -1,2 +1 @@
 cloud/gcp
-unsupported

tests/integration/targets/gcp_cloudfunctions_cloud_function/tasks/autogen.yml

@@ -18,9 +18,10 @@
     name: "{{ resource_name }}"
     location: us-central1
     entry_point: helloGET
-    source_archive_url: gs://ansible-cloudfunctions-bucket/function.zip
+    source_archive_url: gs://{{ gcp_project }}-ansible-testing/cloud-function.zip
     trigger_http: 'true'
     project: "{{ gcp_project }}"
+    runtime: "python310"
     auth_kind: "{{ gcp_cred_kind }}"
     service_account_file: "{{ gcp_cred_file }}"
     state: absent
@@ -30,9 +31,10 @@
     name: "{{ resource_name }}"
     location: us-central1
     entry_point: helloGET
-    source_archive_url: gs://ansible-cloudfunctions-bucket/function.zip
+    source_archive_url: gs://{{ gcp_project }}-ansible-testing/cloud-function.zip
     trigger_http: 'true'
     project: "{{ gcp_project }}"
+    runtime: "python310"
     auth_kind: "{{ gcp_cred_kind }}"
     service_account_file: "{{ gcp_cred_file }}"
     state: present
@@ -60,10 +62,13 @@
     name: "{{ resource_name }}"
     location: us-central1
     entry_point: helloGET
-    source_archive_url: gs://ansible-cloudfunctions-bucket/function.zip
+    source_archive_url: gs://{{ gcp_project }}-ansible-testing/cloud-function.zip
     trigger_http: 'true'
     project: "{{ gcp_project }}"
     auth_kind: "{{ gcp_cred_kind }}"
+    # runtime is not sent as it is optional for
+    # existing functions.
+    # runtime: "python310"
     service_account_file: "{{ gcp_cred_file }}"
     state: present
   register: result
@@ -77,7 +82,7 @@
     name: "{{ resource_name }}"
     location: us-central1
     entry_point: helloGET
-    source_archive_url: gs://ansible-cloudfunctions-bucket/function.zip
+    source_archive_url: gs://{{ gcp_project }}-ansible-testing/cloud-function.zip
     trigger_http: 'true'
     project: "{{ gcp_project }}"
     auth_kind: "{{ gcp_cred_kind }}"
@@ -107,7 +112,7 @@
     name: "{{ resource_name }}"
     location: us-central1
     entry_point: helloGET
-    source_archive_url: gs://ansible-cloudfunctions-bucket/function.zip
+    source_archive_url: gs://{{ gcp_project }}-ansible-testing/cloud-function.zip
     trigger_http: 'true'
     project: "{{ gcp_project }}"
     auth_kind: "{{ gcp_cred_kind }}"

tests/integration/targets/gcp_compute_external_vpn_gateway/aliases

@@ -1,2 +1 @@
-cloud/gcp
+cloud/gcp
-unsupported

tests/integration/targets/gcp_compute_external_vpn_gateway/tasks/autogen.yml

@@ -17,7 +17,7 @@
   google.cloud.gcp_compute_external_vpn_gateway:
     name: "{{ resource_name }}"
     redundancy_type: SINGLE_IP_INTERNALLY_REDUNDANT
-    descrpition: An externalyl managed VPN gateway
+    description: An externaly managed VPN gateway
     interfaces:
     - id: 0
       ip_address: 8.8.8.8
@@ -30,7 +30,7 @@
   google.cloud.gcp_compute_external_vpn_gateway:
     name: "{{ resource_name }}"
     redundancy_type: SINGLE_IP_INTERNALLY_REDUNDANT
-    descrpition: An externalyl managed VPN gateway
+    description: An externalyl managed VPN gateway
     interfaces:
     - id: 0
       ip_address: 8.8.8.8
@@ -62,7 +62,7 @@
   google.cloud.gcp_compute_external_vpn_gateway:
     name: "{{ resource_name }}"
     redundancy_type: SINGLE_IP_INTERNALLY_REDUNDANT
-    descrpition: An externalyl managed VPN gateway
+    description: An externalyl managed VPN gateway
     interfaces:
     - id: 0
       ip_address: 8.8.8.8
@@ -80,7 +80,7 @@
   google.cloud.gcp_compute_external_vpn_gateway:
     name: "{{ resource_name }}"
     redundancy_type: SINGLE_IP_INTERNALLY_REDUNDANT
-    descrpition: An externalyl managed VPN gateway
+    description: An externalyl managed VPN gateway
     interfaces:
     - id: 0
       ip_address: 8.8.8.8
@@ -112,7 +112,7 @@
   google.cloud.gcp_compute_external_vpn_gateway:
     name: "{{ resource_name }}"
     redundancy_type: SINGLE_IP_INTERNALLY_REDUNDANT
-    descrpition: An externalyl managed VPN gateway
+    description: An externalyl managed VPN gateway
     interfaces:
     - id: 0
       ip_address: 8.8.8.8

tests/integration/targets/gcp_compute_forwarding_rule/tasks/autogen.yml

@@ -35,7 +35,7 @@
   google.cloud.gcp_compute_forwarding_rule:
     name: "{{ resource_name }}"
     region: us-west1
-    target: "{{ targetpool }}"
+    target: "{{ targetpool.selfLink }}"
     ip_protocol: TCP
     port_range: 80-80
     ip_address: "{{ address.address }}"
@@ -48,7 +48,7 @@
   google.cloud.gcp_compute_forwarding_rule:
     name: "{{ resource_name }}"
     region: us-west1
-    target: "{{ targetpool }}"
+    target: "{{ targetpool.selfLink }}"
     ip_protocol: TCP
     port_range: 80-80
     ip_address: "{{ address.address }}"
@@ -81,7 +81,7 @@
   google.cloud.gcp_compute_forwarding_rule:
     name: "{{ resource_name }}"
     region: us-west1
-    target: "{{ targetpool }}"
+    target: "{{ targetpool.selfLink }}"
     ip_protocol: TCP
     port_range: 80-80
     ip_address: "{{ address.address }}"
@@ -99,7 +99,7 @@
   google.cloud.gcp_compute_forwarding_rule:
     name: "{{ resource_name }}"
     region: us-west1
-    target: "{{ targetpool }}"
+    target: "{{ targetpool.selfLink }}"
     ip_protocol: TCP
     port_range: 80-80
     ip_address: "{{ address.address }}"
@@ -132,7 +132,7 @@
   google.cloud.gcp_compute_forwarding_rule:
     name: "{{ resource_name }}"
     region: us-west1
-    target: "{{ targetpool }}"
+    target: "{{ targetpool.selfLink }}"
     ip_protocol: TCP
     port_range: 80-80
     ip_address: "{{ address.address }}"

tests/integration/targets/gcp_compute_instance/aliases

@@ -1,2 +1,2 @@
 cloud/gcp
-unsupported
+unsupported

tests/integration/targets/gcp_compute_instance/tasks/autogen.yml

@@ -15,9 +15,9 @@
 # Pre-test setup
 - name: create a disk
   google.cloud.gcp_compute_disk:
-    name: disk-instance
+    name: "{{ resource_prefix }}"
     size_gb: 50
-    source_image: projects/ubuntu-os-cloud/global/images/family/ubuntu-1604-lts
+    source_image: projects/ubuntu-os-cloud/global/images/family/ubuntu-minimal-2204-lts
     zone: us-central1-a
     project: "{{ gcp_project }}"
     auth_kind: "{{ gcp_cred_kind }}"
@@ -26,10 +26,11 @@
   register: disk
 - name: create a network
   google.cloud.gcp_compute_network:
-    name: network-instance
+    name: "{{ resource_prefix }}"
     project: "{{ gcp_project }}"
     auth_kind: "{{ gcp_cred_kind }}"
     service_account_file: "{{ gcp_cred_file }}"
+    auto_create_subnetworks: true
     state: present
   register: network
 - name: create a address
@@ -55,7 +56,6 @@
       initialize_params:
         disk_type: local-ssd
     metadata:
-      startup-script-url: gs:://graphite-playground/bootstrap.sh
       cost-center: '12345'
     labels:
       environment: production
@@ -85,7 +85,6 @@
       initialize_params:
         disk_type: local-ssd
     metadata:
-      startup-script-url: gs:://graphite-playground/bootstrap.sh
       cost-center: '12345'
     labels:
       environment: production
@@ -135,7 +134,6 @@
       initialize_params:
         disk_type: local-ssd
     metadata:
-      startup-script-url: gs:://graphite-playground/bootstrap.sh
       cost-center: '12345'
     labels:
       environment: production
@@ -170,7 +168,6 @@
       initialize_params:
         disk_type: local-ssd
     metadata:
-      startup-script-url: gs:://graphite-playground/bootstrap.sh
       cost-center: '12345'
     labels:
       environment: production
@@ -220,7 +217,6 @@
       initialize_params:
         disk_type: local-ssd
     metadata:
-      startup-script-url: gs:://graphite-playground/bootstrap.sh
       cost-center: '12345'
     labels:
       environment: production
@@ -255,16 +251,17 @@
   ignore_errors: true
 - name: delete a network
   google.cloud.gcp_compute_network:
-    name: network-instance
+    name: "{{ resource_prefix }}"
     project: "{{ gcp_project }}"
     auth_kind: "{{ gcp_cred_kind }}"
     service_account_file: "{{ gcp_cred_file }}"
+    auto_create_subnetworks: true
     state: absent
   register: network
   ignore_errors: true
 - name: delete a disk
   google.cloud.gcp_compute_disk:
-    name: disk-instance
+    name: "{{ resource_prefix }}"
     size_gb: 50
     source_image: projects/ubuntu-os-cloud/global/images/family/ubuntu-1604-lts
     zone: us-central1-a

tests/integration/targets/gcp_compute_region_url_map/aliases

@@ -1,2 +1 @@
-cloud/gcp
+cloud/gcp
-unsupported

tests/integration/targets/gcp_compute_region_url_map/tasks/autogen.yml

@@ -13,29 +13,15 @@
 #
 # ----------------------------------------------------------------------------
 # Pre-test setup
-- name: create a health check
-  google.cloud.gcp_compute_health_check:
-    name: "{{ resource_name }}"
-    type: HTTP
-    http_health_check:
-      port: 80
-    check_interval_sec: 1
-    timeout_sec: 1
-    project: "{{ gcp_project }}"
-    auth_kind: "{{ gcp_cred_kind }}"
-    service_account_file: "{{ gcp_cred_file }}"
-    state: present
-  register: healthcheck
 - name: create a region backend service
   google.cloud.gcp_compute_region_backend_service:
     name: "{{ resource_name }}"
     region: us-central1
-    health_checks:
-    - "{{ healthcheck.selfLink }}"
     protocol: HTTP
     project: "{{ gcp_project }}"
     auth_kind: "{{ gcp_cred_kind }}"
     service_account_file: "{{ gcp_cred_file }}"
+    load_balancing_scheme: "EXTERNAL"
     state: present
   register: backendservice
 - name: delete a region URL map
@@ -144,26 +130,11 @@
   google.cloud.gcp_compute_region_backend_service:
     name: "{{ resource_name }}"
     region: us-central1
-    health_checks:
-    - "{{ healthcheck.selfLink }}"
     protocol: HTTP
     project: "{{ gcp_project }}"
     auth_kind: "{{ gcp_cred_kind }}"
     service_account_file: "{{ gcp_cred_file }}"
+    load_balancing_scheme: "EXTERNAL"
     state: absent
   register: backendservice
-  ignore_errors: true
+  ignore_errors: true
-- name: delete a health check
-  google.cloud.gcp_compute_health_check:
-    name: "{{ resource_name }}"
-    type: HTTP
-    http_health_check:
-      port: 80
-    check_interval_sec: 1
-    timeout_sec: 1
-    project: "{{ gcp_project }}"
-    auth_kind: "{{ gcp_cred_kind }}"
-    service_account_file: "{{ gcp_cred_file }}"
-    state: absent
-  register: healthcheck
-  ignore_errors: true

tests/integration/targets/gcp_compute_route/aliases

@@ -1,2 +1 @@
-cloud/gcp
+cloud/gcp
-unsupported

tests/integration/targets/gcp_compute_route/tasks/autogen.yml

@@ -19,6 +19,7 @@
     project: "{{ gcp_project }}"
     auth_kind: "{{ gcp_cred_kind }}"
     service_account_file: "{{ gcp_cred_file }}"
+    auto_create_subnetworks: true
     state: present
   register: network
 - name: delete a route
@@ -147,6 +148,7 @@
     project: "{{ gcp_project }}"
     auth_kind: "{{ gcp_cred_kind }}"
     service_account_file: "{{ gcp_cred_file }}"
+    auto_create_subnetworks: true
     state: absent
   register: network
   ignore_errors: true

tests/integration/targets/gcp_compute_router/aliases

@@ -1,2 +1 @@
-cloud/gcp
+cloud/gcp
-unsupported

tests/integration/targets/gcp_compute_router/tasks/autogen.yml

@@ -19,6 +19,7 @@
     project: "{{ gcp_project }}"
     auth_kind: "{{ gcp_cred_kind }}"
     service_account_file: "{{ gcp_cred_file }}"
+    auto_create_subnetworks: true
     state: present
   register: network
 - name: delete a router
@@ -169,6 +170,7 @@
     project: "{{ gcp_project }}"
     auth_kind: "{{ gcp_cred_kind }}"
     service_account_file: "{{ gcp_cred_file }}"
+    auto_create_subnetworks: true
     state: absent
   register: network
   ignore_errors: true