Rename project to project_id

q2w · q2w · commit 4e677e70f6be · 2024-10-08T13:56:26.000Z
diff --git a/README.md b/README.md
@@ -55,7 +55,7 @@ module "memorystore" {
 | memory\_size\_gb | Redis memory size in GiB. Defaulted to 1 GiB | `number` | `1` | no |
 | name | The ID of the instance or a fully qualified identifier for the instance. | `string` | n/a | yes |
 | persistence\_config | The Redis persistence configuration parameters. https://cloud.google.com/memorystore/docs/redis/reference/rest/v1/projects.locations.instances#persistenceconfig | <pre>object({<br>    persistence_mode    = string<br>    rdb_snapshot_period = string<br>  })</pre> | `null` | no |
-| project | The ID of the project in which the resource belongs to. | `string` | n/a | yes |
+| project\_id | The ID of the project in which the resource belongs to. | `string` | n/a | yes |
 | read\_replicas\_mode | Read replicas mode. https://cloud.google.com/memorystore/docs/redis/reference/rest/v1/projects.locations.instances#readreplicasmode | `string` | `"READ_REPLICAS_DISABLED"` | no |
 | redis\_configs | The Redis configuration parameters. See [more details](https://cloud.google.com/memorystore/docs/redis/reference/rest/v1/projects.locations.instances#Instance.FIELDS.redis_configs) | `map(any)` | `{}` | no |
 | redis\_version | The version of Redis software. | `string` | `null` | no |
diff --git a/docs/upgrading_to_v12.0.md b/docs/upgrading_to_v12.0.md
@@ -0,0 +1,23 @@
+# Upgrading to v12.0
+
+The v12.0 release contains backwards-incompatible changes.
+
+## Use `project_id` instead of `project`
+
+This release renames `project` to `project_id`. The required input variable `project_id` will be used to configure the project for the resource creation.
+
+
+```diff
+module "memorystore" {
+  source  = "terraform-google-modules/memorystore/google"
+  version = "~> 11.0"
+
+  name           = "memorystore"
+-  project        = "memorystore"
++  project_id     = "memorystore"
+  memory_size_gb = "1"
+  enable_apis    = "true"
+}
+```
+
+To be backward compatible, user needs to set `project_id` instead of `project` input variable.
diff --git a/examples/basic/memorystore.tf b/examples/basic/memorystore.tf
@@ -19,7 +19,7 @@ module "memorystore" {
   version = "~> 11.0"
 
   name           = "memorystore"
-  project        = "memorystore"
+  project_id     = "memorystore"
   memory_size_gb = "1"
   enable_apis    = "true"
 }
diff --git a/examples/memcache/main.tf b/examples/memcache/main.tf
@@ -29,7 +29,7 @@ module "memcache" {
   version = "~> 11.0"
 
   name               = "example-memcache"
-  project            = var.project_id
+  project_id         = var.project_id
   memory_size_mb     = "1024"
   enable_apis        = true
   cpu_count          = "1"
diff --git a/examples/minimal/main.tf b/examples/minimal/main.tf
@@ -19,7 +19,7 @@ module "memstore" {
   version = "~> 11.0"
 
   name               = "test-minimal"
-  project            = var.project_id
+  project_id         = var.project_id
   region             = "us-east1"
   location_id        = "us-east1-b"
   enable_apis        = true
diff --git a/examples/redis-cluster/main.tf b/examples/redis-cluster/main.tf
@@ -38,7 +38,7 @@ module "redis_cluster" {
   version = "~> 11.0"
 
   name                        = "test-redis-cluster"
-  project                     = var.project_id
+  project_id                  = var.project_id
   region                      = "us-central1"
   network                     = ["projects/${var.project_id}/global/networks/${local.network_name}"]
   node_type                   = "REDIS_STANDARD_SMALL"
diff --git a/examples/redis/main.tf b/examples/redis/main.tf
@@ -20,7 +20,7 @@ module "memstore" {
 
   name = "test-redis"
 
-  project                 = var.project_id
+  project_id              = var.project_id
   region                  = "us-east1"
   location_id             = "us-east1-b"
   alternative_location_id = "us-east1-d"
diff --git a/main.tf b/main.tf
@@ -17,7 +17,7 @@
 resource "google_redis_instance" "default" {
   depends_on = [module.enable_apis]
 
-  project            = var.project
+  project            = var.project_id
   name               = var.name
   tier               = var.tier
   replica_count      = var.tier == "STANDARD_HA" ? var.replica_count : null
@@ -72,7 +72,7 @@ module "enable_apis" {
   source  = "terraform-google-modules/project-factory/google//modules/project_services"
   version = "~> 17.0"
 
-  project_id                  = var.project
+  project_id                  = var.project_id
   enable_apis                 = var.enable_apis
   disable_services_on_destroy = false
   disable_dependent_services  = false
diff --git a/metadata.display.yaml b/metadata.display.yaml
@@ -96,3 +96,9 @@ spec:
         transit_encryption_mode:
           name: transit_encryption_mode
           title: Transit Encryption Mode
+    runtime:
+      outputs:
+        host:
+          visibility: VISIBILITY_ROOT
+        port:
+          visibility: VISIBILITY_ROOT
diff --git a/metadata.yaml b/metadata.yaml
@@ -61,7 +61,7 @@ spec:
         connections:
         - source:
             source: github.com/terraform-google-modules/terraform-google-network//modules/vpc
-            version: v9.1.0
+            version: ">= 9.1.0"
           spec:
             outputExpr: network_name
       - name: connect_mode
diff --git a/modules/memcache/README.md b/modules/memcache/README.md
@@ -19,7 +19,7 @@ A Terraform module for creating a fully functional Google Memorystore (memcache)
 | name | The ID of the instance or a fully qualified identifier for the instance. | `string` | n/a | yes |
 | node\_count | Number of nodes in the memcache instance. | `number` | `1` | no |
 | params | Parameters for the memcache process | `map(string)` | `null` | no |
-| project | The ID of the project in which the resource belongs to. | `string` | n/a | yes |
+| project\_id | The ID of the project in which the resource belongs to. | `string` | n/a | yes |
 | region | The GCP region to use. | `string` | n/a | yes |
 | zones | Zones where memcache nodes should be provisioned. If not provided, all zones will be used. | `list(string)` | `null` | no |
 
diff --git a/modules/memcache/main.tf b/modules/memcache/main.tf
@@ -17,7 +17,7 @@
 resource "google_memcache_instance" "self" {
   depends_on         = [module.enable_apis]
   provider           = google-beta
-  project            = var.project
+  project            = var.project_id
   zones              = var.zones
   name               = var.name
   region             = var.region
@@ -62,7 +62,7 @@ module "enable_apis" {
   source  = "terraform-google-modules/project-factory/google//modules/project_services"
   version = "~> 17.0"
 
-  project_id  = var.project
+  project_id  = var.project_id
   enable_apis = var.enable_apis
 
   disable_services_on_destroy = false
diff --git a/modules/memcache/variables.tf b/modules/memcache/variables.tf
@@ -19,7 +19,7 @@ variable "region" {
   type        = string
 }
 
-variable "project" {
+variable "project_id" {
   description = "The ID of the project in which the resource belongs to."
   type        = string
 }
diff --git a/modules/redis-cluster/README.md b/modules/redis-cluster/README.md
@@ -36,7 +36,7 @@ module "redis_cluster" {
 | name | The ID of the instance or a fully qualified identifier for the instance. must be 1 to 63 characters and use only lowercase letters, numbers, or hyphens. It must start with a lowercase letter and end with a lowercase letter or number | `string` | n/a | yes |
 | network | List of consumer network where the network address of the discovery endpoint will be reserved, in the form of projects/{network\_project\_id\_or\_number}/global/networks/{network\_id}. Currently, only one item is supported | `list(string)` | n/a | yes |
 | node\_type | The nodeType for the Redis cluster. If not provided, REDIS\_HIGHMEM\_MEDIUM will be used as default Possible values are: REDIS\_SHARED\_CORE\_NANO, REDIS\_HIGHMEM\_MEDIUM, REDIS\_HIGHMEM\_XLARGE, REDIS\_STANDARD\_SMALL. | `string` | `null` | no |
-| project | The ID of the project in which the resource belongs to. | `string` | n/a | yes |
+| project\_id | The ID of the project in which the resource belongs to. | `string` | n/a | yes |
 | redis\_configs | Configure Redis Cluster behavior using a subset of native Redis configuration parameters | <pre>object({<br>    maxmemory-clients       = optional(string)<br>    maxmemory               = optional(string)<br>    maxmemory-policy        = optional(string)<br>    notify-keyspace-events  = optional(string)<br>    slowlog-log-slower-than = optional(number)<br>    maxclients              = optional(number)<br>  })</pre> | `null` | no |
 | region | The name of the region of the Redis cluster | `string` | n/a | yes |
 | replica\_count | The number of replica nodes per shard. Each shard can have 0, 1, or 2 replica nodes. Replicas provide high availability and additional read throughput, and are evenly distributed across zones | `number` | `0` | no |
diff --git a/modules/redis-cluster/main.tf b/modules/redis-cluster/main.tf
@@ -15,7 +15,7 @@
  */
 
 resource "google_redis_cluster" "redis_cluster" {
-  project                     = var.project
+  project                     = var.project_id
   name                        = var.name
   shard_count                 = var.shard_count
   region                      = var.region
@@ -68,7 +68,7 @@ module "enable_apis" {
   source  = "terraform-google-modules/project-factory/google//modules/project_services"
   version = "~> 17.0"
 
-  project_id  = var.project
+  project_id  = var.project_id
   enable_apis = var.enable_apis
 
   disable_services_on_destroy = false
diff --git a/modules/redis-cluster/variables.tf b/modules/redis-cluster/variables.tf
@@ -19,7 +19,7 @@ variable "name" {
   type        = string
 }
 
-variable "project" {
+variable "project_id" {
   description = "The ID of the project in which the resource belongs to."
   type        = string
 }
diff --git a/variables.tf b/variables.tf
@@ -20,7 +20,7 @@ variable "region" {
   default     = null
 }
 
-variable "project" {
+variable "project_id" {
   description = "The ID of the project in which the resource belongs to."
   type        = string
 }

Original file line number	Diff line number	Diff line change
`@@ -19,7 +19,7 @@ module "memorystore" {`
`19`	`19`	`version = "~> 11.0"`
`20`	`20`
`21`	`21`	`name = "memorystore"`
`22`		`- project = "memorystore"`
	`22`	`+ project_id = "memorystore"`
`23`	`23`	`memory_size_gb = "1"`
`24`	`24`	`enable_apis = "true"`
`25`	`25`	`}`