[remote-state] bugfix: Handle case where defaults is incompatible type (

#49)
cloudposse · Jun 2, 2022 · 325d23c · 325d23c
1 parent d7b98c5
commit 325d23c
Show file tree

Hide file tree

Showing 3 changed files with 12 additions and 3 deletions.
diff --git a/.github/renovate.json b/.github/renovate.json
@@ -4,9 +4,9 @@
     ":preserveSemverRanges"
   ],
   "labels": ["auto-update"],
+  "dependencyDashboardAutoclose": true,
   "enabledManagers": ["terraform"],
   "terraform": {
     "ignorePaths": ["**/context.tf", "examples/**"]
   }
 }
-
diff --git a/examples/remote-state/main.tf b/examples/remote-state/main.tf
@@ -4,6 +4,13 @@ module "remote_state_using_stack" {
   component = "test/test-component-override"
   stack     = "tenant1-ue2-dev"
 
+  # Verify that a default output not matching the real output does not cause a Terraform error
+  defaults = {
+    val1 = ["default-list"]
+    val2 = "default-value"
+  }
+
+
   context = module.this.context
 }
 

diff --git a/modules/remote-state/main.tf b/modules/remote-state/main.tf
@@ -36,6 +36,8 @@ locals {
     static = [{ outputs = local.backend }]
   }
 
-  remote_state_backend_key = var.bypass ? "bypass" : local.backend_type
-  outputs                  = try(length(local.remote_state_backend_key), 0) > 0 ? local.remote_states[local.remote_state_backend_key][0].outputs : var.defaults
+  remote_state_backend_key          = var.bypass ? "bypass" : local.backend_type
+  computed_remote_state_backend_key = try(length(local.remote_states[local.remote_state_backend_key]), 0) > 0 ? local.remote_state_backend_key : "bypass"
+
+  outputs = local.remote_states[local.computed_remote_state_backend_key][0].outputs
 }