fix: make project_id and region as required input for instance_template

terraform-google-modules · Dec 15, 2024 · 1df13ff · 1df13ff
1 parent 0c9d75b
commit 1df13ff
Show file tree

Hide file tree

Showing 16 changed files with 27 additions and 6 deletions.
diff --git a/examples/instance_template/additional_disks/main.tf b/examples/instance_template/additional_disks/main.tf
@@ -25,6 +25,7 @@ module "instance_template" {
   version = "~> 12.0"
 
   project_id      = var.project_id
+  region          = var.region
   subnetwork      = var.subnetwork
   service_account = var.service_account
   name_prefix     = "additional-disks"

diff --git a/examples/instance_template/alias_ip_range/main.tf b/examples/instance_template/alias_ip_range/main.tf
@@ -29,6 +29,7 @@ module "instance_template" {
   version = "~> 12.0"
 
   project_id      = var.project_id
+  region          = var.region
   subnetwork      = var.subnetwork
   service_account = var.service_account
   name_prefix     = "alias-ip-range"

diff --git a/examples/instance_template/simple/main.tf b/examples/instance_template/simple/main.tf
@@ -36,6 +36,7 @@ module "instance_template" {
   version = "~> 12.0"
 
   project_id                   = var.project_id
+  region                       = var.region
   subnetwork                   = var.subnetwork
   stack_type                   = "IPV4_ONLY"
   service_account              = var.service_account

diff --git a/examples/mig/autoscaler/main.tf b/examples/mig/autoscaler/main.tf
@@ -31,6 +31,7 @@ module "instance_template" {
   version = "~> 12.0"
 
   project_id      = var.project_id
+  region          = var.region
   subnetwork      = var.subnetwork
   service_account = var.service_account
 }

diff --git a/examples/mig/healthcheck/main.tf b/examples/mig/healthcheck/main.tf
@@ -55,6 +55,7 @@ module "instance_template" {
   version = "~> 12.0"
 
   project_id      = var.project_id
+  region          = var.region
   subnetwork      = google_compute_subnetwork.main.name
   service_account = var.service_account
 }

diff --git a/examples/mig/simple/main.tf b/examples/mig/simple/main.tf
@@ -29,6 +29,7 @@ module "instance_template" {
   version = "~> 12.0"
 
   project_id         = var.project_id
+  region             = var.region
   subnetwork         = var.subnetwork
   service_account    = var.service_account
   subnetwork_project = var.project_id

diff --git a/examples/umig/named_ports/main.tf b/examples/umig/named_ports/main.tf
@@ -25,6 +25,7 @@ module "instance_template" {
   version = "~> 12.0"
 
   project_id      = var.project_id
+  region          = var.region
   subnetwork      = var.subnetwork
   service_account = var.service_account
 }

diff --git a/examples/umig/simple/main.tf b/examples/umig/simple/main.tf
@@ -25,6 +25,7 @@ module "instance_template" {
   version = "~> 12.0"
 
   project_id         = var.project_id
+  region             = var.region
   subnetwork         = var.subnetwork
   subnetwork_project = var.project_id
   service_account    = var.service_account

diff --git a/examples/umig/static_ips/main.tf b/examples/umig/static_ips/main.tf
@@ -25,6 +25,7 @@ module "instance_template" {
   version = "~> 12.0"
 
   project_id         = var.project_id
+  region             = var.region
   subnetwork         = var.subnetwork
   subnetwork_project = var.project_id
   service_account    = var.service_account

diff --git a/modules/instance_template/README.md b/modules/instance_template/README.md
@@ -44,8 +44,8 @@ See the [simple](../../examples/instance_template/simple) for a usage example.
 | nic\_type | Valid values are "VIRTIO\_NET", "GVNIC" or set to null to accept API default behavior. | `string` | `null` | no |
 | on\_host\_maintenance | Instance availability Policy | `string` | `"MIGRATE"` | no |
 | preemptible | Allow the instance to be preempted | `bool` | `false` | no |
-| project\_id | The GCP project ID | `string` | `null` | no |
-| region | Region where the instance template should be created. | `string` | `null` | no |
+| project\_id | The GCP project ID | `string` | n/a | yes |
+| region | Region where the instance template should be created. | `string` | n/a | yes |
 | resource\_policies | A list of self\_links of resource policies to attach to the instance. Modifying this list will cause the instance to recreate. Currently a max of 1 resource policy is supported. | `list(string)` | `[]` | no |
 | service\_account | Service account to attach to the instance. See https://www.terraform.io/docs/providers/google/r/compute_instance_template#service_account. | <pre>object({<br>    email  = string<br>    scopes = optional(set(string), ["cloud-platform"])<br>  })</pre> | n/a | yes |
 | shielded\_instance\_config | Not used unless enable\_shielded\_vm is true. Shielded VM configuration for the instance. | <pre>object({<br>    enable_secure_boot          = bool<br>    enable_vtpm                 = bool<br>    enable_integrity_monitoring = bool<br>  })</pre> | <pre>{<br>  "enable_integrity_monitoring": true,<br>  "enable_secure_boot": true,<br>  "enable_vtpm": true<br>}</pre> | no |

diff --git a/modules/instance_template/metadata.yaml b/modules/instance_template/metadata.yaml
@@ -83,9 +83,11 @@ spec:
       - name: project_id
         description: The GCP project ID
         varType: string
+        required: true
       - name: region
         description: Region where the instance template should be created.
         varType: string
+        required: true
       - name: name_prefix
         description: Name prefix for the instance template
         varType: string

diff --git a/modules/instance_template/variables.tf b/modules/instance_template/variables.tf
@@ -17,13 +17,11 @@
 variable "project_id" {
   type        = string
   description = "The GCP project ID"
-  default     = null
 }
 
 variable "region" {
   type        = string
   description = "Region where the instance template should be created."
-  default     = null
 }
 
 variable "name_prefix" {

diff --git a/modules/preemptible_and_regular_instance_templates/README.md b/modules/preemptible_and_regular_instance_templates/README.md
@@ -24,7 +24,8 @@ See the [simple](../../examples/preemptible_and_regular_instance_templates/simpl
 | metadata | Metadata, provided as a map | `map(string)` | `{}` | no |
 | name\_prefix | Name prefix for the instance template | `string` | `"default-it"` | no |
 | network | The name or self\_link of the network to attach this interface to. Use network attribute for Legacy or Auto subnetted networks and subnetwork for custom subnetted networks. | `string` | `""` | no |
-| project\_id | The GCP project ID | `string` | `null` | no |
+| project\_id | The GCP project ID | `string` | n/a | yes |
+| region | Region where the instance template should be created. | `string` | n/a | yes |
 | service\_account | Service account to attach to the instance. See https://www.terraform.io/docs/providers/google/r/compute_instance_template#service_account. | <pre>object({<br>    email  = string<br>    scopes = set(string)<br>  })</pre> | n/a | yes |
 | source\_image | Source disk image. If neither source\_image nor source\_image\_family is specified, defaults to the latest public Rocky Linux 9 optimized for GCP image. | `string` | `""` | no |
 | source\_image\_family | Source image family. If neither source\_image nor source\_image\_family is specified, defaults to the latest public Rocky Linux 9 optimized for GCP image. | `string` | `""` | no |

diff --git a/modules/preemptible_and_regular_instance_templates/main.tf b/modules/preemptible_and_regular_instance_templates/main.tf
@@ -22,6 +22,7 @@ module "preemptible" {
   source               = "../../modules/instance_template"
   name_prefix          = "${var.name_prefix}-preemptible"
   project_id           = var.project_id
+  region               = var.region
   machine_type         = var.machine_type
   labels               = var.labels
   metadata             = var.metadata
@@ -48,6 +49,7 @@ module "regular" {
   source               = "../../modules/instance_template"
   name_prefix          = "${var.name_prefix}-regular"
   project_id           = var.project_id
+  region               = var.region
   machine_type         = var.machine_type
   labels               = var.labels
   metadata             = var.metadata

diff --git a/modules/preemptible_and_regular_instance_templates/metadata.yaml b/modules/preemptible_and_regular_instance_templates/metadata.yaml
@@ -83,6 +83,11 @@ spec:
       - name: project_id
         description: The GCP project ID
         varType: string
+        required: true
+      - name: region
+        description: Region where the instance template should be created.
+        varType: string
+        required: true
       - name: name_prefix
         description: Name prefix for the instance template
         varType: string

diff --git a/modules/preemptible_and_regular_instance_templates/variables.tf b/modules/preemptible_and_regular_instance_templates/variables.tf
@@ -17,7 +17,11 @@
 variable "project_id" {
   type        = string
   description = "The GCP project ID"
-  default     = null
+}
+
+variable "region" {
+  type        = string
+  description = "Region where the instance template should be created."
 }
 
 variable "name_prefix" {