fix!: make project_id and region as required input for instance_template (#455)

Author: q2w

docs/upgrading_to_mig_v13.0.md

@@ -1,11 +1,20 @@
-# Upgrading to MIG v13.0
+# Upgrading to v13.0
 
-The v13.0 release of MIG is a backwards incompatible release. Now the `project_id` is required input variable. Earlier it was set to `null` by default. But the `terraform plan` would fail because `google_compute_zones` data block requires a non-null `project_id`.
+The v13.0 release of MIG and instance_template is a backwards incompatible release.
+
+## `modules/mig` and `modules/mig_with_percent`
+
+Now the `project_id` is required input variable. Earlier it was set to `null` by default. But the `terraform plan` would fail because `google_compute_zones` data block requires a non-null `project_id`.
 Users might be bypassing this by setting project at provider level and not providing `project_id` for mig module.
 
+## `modules/instance_template` and `modules/preemptible_and_regular_instance_templates`
+
+* Now the `project_id` and `region` is required input variable. Earlier it was set to `null` by default.
+
+
 ## Migration Instructions
 
-### Explicitly provide project_id as input to the MIG module
+### Explicitly provide project_id as input to the MIG modules
 
 Users would need to provide project_id explicitly while using v13.0 of mig/mig_with_percent.
 
@@ -14,7 +23,7 @@ For mig,
 ```diff
 module "mig" {
   source  = "terraform-google-modules/vm/google//modules/mig"
-  version = "~> 12.0"
+  version = "~> 13.0"
 
 + project_id        = var.project_id
   region            = var.region
@@ -29,7 +38,7 @@ For mig_with_percent,
 ```diff
 module "mig_with_percent" {
   source  = "terraform-google-modules/vm/google//modules/mig_with_percent"
-  version = "~> 12.0"
+  version = "~> 13.0"
 
 + project_id                        = var.project_id
   region                            = var.region
@@ -40,3 +49,43 @@ module "mig_with_percent" {
   next_version_percent              = 50
 }
 ```
+
+### Explicitly provide project_id/region as input to the instance_template modules
+
+For instance_template,
+
+```diff
+module "instance_template" {
+  source  = "terraform-google-modules/vm/google//modules/instance_template"
+  version = "~> 13.0"
+
++ project_id                   = var.project_id
++ region                       = var.region
+  subnetwork                   = var.subnetwork
+  stack_type                   = "IPV4_ONLY"
+  service_account              = var.service_account
+  name_prefix                  = "simple"
+  tags                         = var.tags
+  labels                       = var.labels
+  access_config                = [local.access_config]
+  enable_nested_virtualization = var.enable_nested_virtualization
+  threads_per_core             = var.threads_per_core
+}
+```
+
+For preemptible_and_regular_instance_templates
+
+```diff
+module "preemptible_and_regular_instance_templates" {
+  source  = "terraform-google-modules/vm/google//modules/preemptible_and_regular_instance_templates"
+  version = "~> 13.0"
+
+  subnetwork      = var.subnetwork
++ project_id      = var.project_id
++ region          = var.region
+  service_account = var.service_account
+  name_prefix     = "pvm-and-regular-simple"
+  tags            = var.tags
+  labels          = var.labels
+}
+```

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"

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"

examples/instance_template/encrypted_disks/main.tf

@@ -40,6 +40,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"

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

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
 }

examples/mig/full/main.tf

@@ -32,6 +32,7 @@ module "instance_template" {
 
   name_prefix     = "${var.hostname}-instance-template"
   project_id      = var.project_id
+  region          = var.region
   machine_type    = var.machine_type
   tags            = var.tags
   labels          = var.labels

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
 }

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

examples/mig_with_percent/simple/main.tf

@@ -30,6 +30,8 @@ module "preemptible_and_regular_instance_templates" {
   source  = "terraform-google-modules/vm/google//modules/preemptible_and_regular_instance_templates"
   version = "~> 12.0"
 
+  project_id      = var.project_id
+  region          = var.region
   subnetwork      = var.subnetwork
   service_account = var.service_account
 }