chore: update tests to use 1.0 image and update compat note (#72)

cloud-foundation-bot · morgante · web-flow · commit b9b7003df989 · 2021-09-29T18:03:52.000-04:00
Co-authored-by: Morgante Pell &lt;morgantep@google.com&gt;
diff --git a/.github/release-please.yml b/.github/release-please.yml
@@ -1,2 +1,16 @@
+# Copyright 2021 Google LLC
+#
+# Licensed under the Apache License, Version 2.0 (the "License");
+# you may not use this file except in compliance with the License.
+# You may obtain a copy of the License at
+#
+#      http://www.apache.org/licenses/LICENSE-2.0
+#
+# Unless required by applicable law or agreed to in writing, software
+# distributed under the License is distributed on an "AS IS" BASIS,
+# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+# See the License for the specific language governing permissions and
+# limitations under the License.
+
 releaseType: terraform-module
 handleGHRelease: true
diff --git a/.github/workflows/stale.yml b/.github/workflows/stale.yml
@@ -1,3 +1,17 @@
+# Copyright 2021 Google LLC
+#
+# Licensed under the Apache License, Version 2.0 (the "License");
+# you may not use this file except in compliance with the License.
+# You may obtain a copy of the License at
+#
+#      http://www.apache.org/licenses/LICENSE-2.0
+#
+# Unless required by applicable law or agreed to in writing, software
+# distributed under the License is distributed on an "AS IS" BASIS,
+# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+# See the License for the specific language governing permissions and
+# limitations under the License.
+
 name: "Close stale issues"
 on:
   schedule:
diff --git a/.gitignore b/.gitignore
@@ -56,3 +56,6 @@ examples/dynamic-files/function_source/terraform_created_file.txt
 
 node_modules
 yarn.lock
+
+# tf lock file
+.terraform.lock.hcl
diff --git a/Makefile b/Makefile
@@ -18,7 +18,7 @@
 # Make will use bash instead of sh
 SHELL := /usr/bin/env bash
 
-DOCKER_TAG_VERSION_DEVELOPER_TOOLS := 0.13
+DOCKER_TAG_VERSION_DEVELOPER_TOOLS := 1.0
 DOCKER_IMAGE_DEVELOPER_TOOLS := cft/developer-tools
 REGISTRY_URL := gcr.io/cloud-foundation-cicd
 
diff --git a/README.md b/README.md
@@ -14,8 +14,10 @@ configures a function sourced from a Cloud Source Repositories
 repository.
 
 ## Compatibility
-This module is meant for use with Terraform 0.13. If you haven't
-[upgraded](https://www.terraform.io/upgrade-guides/0-13.html) and need a Terraform
+This module is meant for use with Terraform 0.13+ and tested using Terraform 1.0+.
+If you find incompatibilities using Terraform `>=0.13`, please open an issue.
+
+If you haven't [upgraded][terraform-0.13-upgrade] and need a Terraform
 0.12.x-compatible version of this module, the last released version
 intended for Terraform 0.12.x is [v1.5.0](https://registry.terraform.io/modules/terraform-google-modules/-event-function/google/v1.5.0).
 
@@ -50,32 +52,32 @@ module "localhost_function" {
 ## Inputs
 
 | Name | Description | Type | Default | Required |
-|------|-------------|:----:|:-----:|:-----:|
-| available\_memory\_mb | The amount of memory in megabytes allotted for the function to use. | number | `"256"` | no |
-| bucket\_force\_destroy | When deleting the GCS bucket containing the cloud function, delete all objects in the bucket first. | bool | `"false"` | no |
-| bucket\_labels | A set of key/value label pairs to assign to the function source archive bucket. | map(string) | `<map>` | no |
-| bucket\_name | The name to apply to the bucket. Will default to a string of the function name. | string | `""` | no |
-| create\_bucket | Whether to create a new bucket or use an existing one. If false, `bucket_name` should reference the name of the alternate bucket to use. | bool | `"true"` | no |
-| description | The description of the function. | string | `"Processes events."` | no |
-| entry\_point | The name of a method in the function source which will be invoked when the function is executed. | string | n/a | yes |
-| environment\_variables | A set of key/value environment variable pairs to assign to the function. | map(string) | `<map>` | no |
-| event\_trigger | A source that fires events in response to a condition in another service. | map(string) | `<map>` | no |
-| event\_trigger\_failure\_policy\_retry | A toggle to determine if the function should be retried on failure. | bool | `"false"` | no |
-| files\_to\_exclude\_in\_source\_dir | Specify files to ignore when reading the source_dir | list(string) | `<list>` | no |
-| ingress\_settings | The ingress settings for the function. Allowed values are ALLOW_ALL, ALLOW_INTERNAL_AND_GCLB and ALLOW_INTERNAL_ONLY. Changes to this field will recreate the cloud function. | string | `"ALLOW_ALL"` | no |
-| labels | A set of key/value label pairs to assign to the Cloud Function. | map(string) | `<map>` | no |
-| max\_instances | The maximum number of parallel executions of the function. | number | `"0"` | no |
-| name | The name to apply to any nameable resources. | string | n/a | yes |
-| project\_id | The ID of the project to which resources will be applied. | string | n/a | yes |
-| region | The region in which resources will be applied. | string | n/a | yes |
-| runtime | The runtime in which the function will be executed. | string | n/a | yes |
-| service\_account\_email | The service account to run the function as. | string | `""` | no |
-| source\_dependent\_files | A list of any Terraform created `local_file`s that the module will wait for before creating the archive. | object | `<list>` | no |
-| source\_directory | The pathname of the directory which contains the function source code. | string | n/a | yes |
-| timeout\_s | The amount of time in seconds allotted for the execution of the function. | number | `"60"` | no |
-| trigger\_http | Wheter to use HTTP trigger instead of the event trigger. | bool | `"null"` | no |
-| vpc\_connector | The VPC Network Connector that this cloud function can connect to. It should be set up as fully-qualified URI. The format of this field is projects/*/locations/*/connectors/*. | string | `"null"` | no |
-| vpc\_connector\_egress\_settings | The egress settings for the connector, controlling what traffic is diverted through it. Allowed values are ALL_TRAFFIC and PRIVATE_RANGES_ONLY. If unset, this field preserves the previously set value. | string | `"null"` | no |
+|------|-------------|------|---------|:--------:|
+| available\_memory\_mb | The amount of memory in megabytes allotted for the function to use. | `number` | `256` | no |
+| bucket\_force\_destroy | When deleting the GCS bucket containing the cloud function, delete all objects in the bucket first. | `bool` | `false` | no |
+| bucket\_labels | A set of key/value label pairs to assign to the function source archive bucket. | `map(string)` | `{}` | no |
+| bucket\_name | The name to apply to the bucket. Will default to a string of the function name. | `string` | `""` | no |
+| create\_bucket | Whether to create a new bucket or use an existing one. If false, `bucket_name` should reference the name of the alternate bucket to use. | `bool` | `true` | no |
+| description | The description of the function. | `string` | `"Processes events."` | no |
+| entry\_point | The name of a method in the function source which will be invoked when the function is executed. | `string` | n/a | yes |
+| environment\_variables | A set of key/value environment variable pairs to assign to the function. | `map(string)` | `{}` | no |
+| event\_trigger | A source that fires events in response to a condition in another service. | `map(string)` | `{}` | no |
+| event\_trigger\_failure\_policy\_retry | A toggle to determine if the function should be retried on failure. | `bool` | `false` | no |
+| files\_to\_exclude\_in\_source\_dir | Specify files to ignore when reading the source\_dir | `list(string)` | `[]` | no |
+| ingress\_settings | The ingress settings for the function. Allowed values are ALLOW\_ALL, ALLOW\_INTERNAL\_AND\_GCLB and ALLOW\_INTERNAL\_ONLY. Changes to this field will recreate the cloud function. | `string` | `"ALLOW_ALL"` | no |
+| labels | A set of key/value label pairs to assign to the Cloud Function. | `map(string)` | `{}` | no |
+| max\_instances | The maximum number of parallel executions of the function. | `number` | `0` | no |
+| name | The name to apply to any nameable resources. | `string` | n/a | yes |
+| project\_id | The ID of the project to which resources will be applied. | `string` | n/a | yes |
+| region | The region in which resources will be applied. | `string` | n/a | yes |
+| runtime | The runtime in which the function will be executed. | `string` | n/a | yes |
+| service\_account\_email | The service account to run the function as. | `string` | `""` | no |
+| source\_dependent\_files | A list of any Terraform created `local_file`s that the module will wait for before creating the archive. | <pre>list(object({<br>    filename = string<br>    id       = string<br>  }))</pre> | `[]` | no |
+| source\_directory | The pathname of the directory which contains the function source code. | `string` | n/a | yes |
+| timeout\_s | The amount of time in seconds allotted for the execution of the function. | `number` | `60` | no |
+| trigger\_http | Wheter to use HTTP trigger instead of the event trigger. | `bool` | `null` | no |
+| vpc\_connector | The VPC Network Connector that this cloud function can connect to. It should be set up as fully-qualified URI. The format of this field is projects/\*/locations/\*/connectors/\*. | `string` | `null` | no |
+| vpc\_connector\_egress\_settings | The egress settings for the connector, controlling what traffic is diverted through it. Allowed values are ALL\_TRAFFIC and PRIVATE\_RANGES\_ONLY. If unset, this field preserves the previously set value. | `string` | `null` | no |
 
 ## Outputs
 
@@ -131,3 +133,4 @@ information on contributing to this module.
 [project-factory-module-site]: https://github.com/terraform-google-modules/terraform-google-project-factory/
 [terraform-provider-gcp-site]: https://github.com/terraform-providers/terraform-provider-google/
 [terraform-site]: https://www.terraform.io/
+[terraform-0.13-upgrade]: https://www.terraform.io/upgrade-guides/0-13.html
diff --git a/build/int.cloudbuild.yaml b/build/int.cloudbuild.yaml
@@ -38,4 +38,4 @@ tags:
 - 'integration'
 substitutions:
   _DOCKER_IMAGE_DEVELOPER_TOOLS: 'cft/developer-tools'
-  _DOCKER_TAG_VERSION_DEVELOPER_TOOLS: '0.13'
+  _DOCKER_TAG_VERSION_DEVELOPER_TOOLS: '1.0'
diff --git a/build/lint.cloudbuild.yaml b/build/lint.cloudbuild.yaml
@@ -21,4 +21,4 @@ tags:
 - 'lint'
 substitutions:
   _DOCKER_IMAGE_DEVELOPER_TOOLS: 'cft/developer-tools'
-  _DOCKER_TAG_VERSION_DEVELOPER_TOOLS: '0.13'
+  _DOCKER_TAG_VERSION_DEVELOPER_TOOLS: '1.0'
