Merge pull request #44 from cerebrotech/ignore-provider-state-err

Ignore provider state err
dominodatalab · Dec 10, 2019 · 9b30a6a · 9b30a6a
2 parents aa65fcf + f359272
commit 9b30a6a
Show file tree

Hide file tree

Showing 2 changed files with 14 additions and 1 deletion.
diff --git a/.circleci/config.yml b/.circleci/config.yml
@@ -7,7 +7,7 @@ jobs:
       CLOUDSDK_CORE_PROJECT: domino-eng-platform-dev
       CLOUDSDK_COMPUTE_ZONE: us-west1-a
       GOOGLE_APPLICATION_CREDENTIALS: /root/.config/gcloud/legacy_credentials/terraform-gke-test@domino-eng-platform-dev.iam.gserviceaccount.com/adc.json
-      TERRAFORM_VERSION: 0.12.12
+      TERRAFORM_VERSION: 0.12.17
 
     steps:
       - checkout

diff --git a/main.tf b/main.tf
@@ -154,6 +154,19 @@ resource "google_container_cluster" "domino_cluster" {
     }
   }
 
+  # This resource's provider has issues with reconciling the remote/local state
+  # of the `issue_client_certificate` field because we use channels to
+  # implicitly set a cluster version.
+  #
+  # We're going to ignore all changes to the `master_auth` block since we set
+  # these values statically. Hopefully, this issue will be resolved in a future
+  # version of the provider. See the following issue for more context.
+  #
+  # https://github.com/terraform-providers/terraform-provider-google/issues/3369#issuecomment-487226330
+  lifecycle {
+    ignore_changes = [master_auth]
+  }
+
   vertical_pod_autoscaling {
     enabled = var.enable_vertical_pod_autoscaling
   }