Fix csi-driver, logging and monitoring default values when autopilot … (

#883) * Fix csi-driver, logging and monitoring default values when autopilot is used Co-authored-by: Ludovico Magnocavallo <ludomagno@google.com>
GoogleCloudPlatform · Oct 14, 2022 · 49d5e21 · 49d5e21
1 parent 1c24e92
commit 49d5e21
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 3 deletions.
diff --git a/default-versions.tf b/default-versions.tf
@@ -23,6 +23,10 @@ terraform {
       source  = "hashicorp/google-beta"
       version = ">= 4.36.0" # tftest
     }
+    # used in modules
+    archive = {
+      source = "hashicorp/archive"
+    }
     # used in fast
     local = {
       source = "hashicorp/local"

diff --git a/modules/gke-cluster/main.tf b/modules/gke-cluster/main.tf
@@ -79,7 +79,11 @@ resource "google_container_cluster" "cluster" {
       )
     }
     gce_persistent_disk_csi_driver_config {
-      enabled = var.enable_addons.gce_persistent_disk_csi_driver
+      enabled = (
+        var.enable_features.autopilot
+        ? true
+        : var.enable_addons.gce_persistent_disk_csi_driver
+      )
     }
     dynamic "gcp_filestore_csi_driver_config" {
       for_each = !var.enable_features.autopilot ? [""] : []
@@ -169,7 +173,7 @@ resource "google_container_cluster" "cluster" {
   }
 
   dynamic "logging_config" {
-    for_each = var.logging_config != null ? [""] : []
+    for_each = var.logging_config != null && !var.enable_features.autopilot ? [""] : []
     content {
       enable_components = var.logging_config
     }
@@ -234,7 +238,7 @@ resource "google_container_cluster" "cluster" {
   }
 
   dynamic "monitoring_config" {
-    for_each = var.monitoring_config != null ? [""] : []
+    for_each = var.monitoring_config != null && !var.enable_features.autopilot ? [""] : []
     content {
       enable_components = var.monitoring_config
     }