Merge pull request #131 from DSC-McMaster-U/rawan/terraform-location

Updating Cluster Locations to be Zones
DSC-McMaster-U · Nov 18, 2023 · 7442d51 · 7442d51
2 parents 4e2e0db + 54ca82c
commit 7442d51
Showing 1 changed file with 4 additions and 4 deletions.
diff --git a/cloud-infra/main.tf b/cloud-infra/main.tf
@@ -1,6 +1,6 @@
 variable "project" { default = "automateml" }
 variable "region" { default = "us-east1" }
-variable "zone" { default = "us-east1-a" }
+variable "zone" { default = "us-east1-b" }
 variable "sa_email" { default = "owner-sa@automateml.iam.gserviceaccount.com" }
 
 provider "google" {
@@ -12,7 +12,7 @@ provider "google" {
 
 resource "google_container_cluster" "automl_cluster" {
   name     = "automl-cluster"
-  location = var.region
+  location = var.zone
   project  = var.project
 
   remove_default_node_pool = true
@@ -22,7 +22,7 @@ resource "google_container_cluster" "automl_cluster" {
 //single node "node pool" for frontend and backend pods
 resource "google_container_node_pool" "febe_node_pool" {
   name       = "frontend-backend-node-pool"
-  location   = var.region
+  location   = var.zone
   cluster    = google_container_cluster.automl_cluster.name
   node_count = 1
 
@@ -45,7 +45,7 @@ resource "google_container_node_pool" "febe_node_pool" {
 //pool for machine learning (allows us to adjust the compute later if needed)
 resource "google_container_node_pool" "ml_node_pool" {
   name       = "machine-learning-node-pool"
-  location   = var.region
+  location   = var.zone
   cluster    = google_container_cluster.automl_cluster.name
   node_count = 1