clarify lack of function support

johnstcn · johnstcn · commit 9214bd48a52a · 2024-11-08T10:44:00.000Z
diff --git a/provisioner/terraform/tfparse/tfextract.go b/provisioner/terraform/tfparse/tfextract.go
@@ -341,6 +341,11 @@ func buildEvalContext(varDefaults map[string]string, paramDefaults map[string]st
 
 	evalCtx := &hcl.EvalContext{
 		Variables: map[string]cty.Value{},
+		// NOTE: we do not currently support function execution here.
+		// The default function map for Terraform is not exposed, so we would essentially
+		// have to re-implement or copy the entire map or a subset thereof.
+		// ref: https://github.com/hashicorp/terraform/blob/e044e569c5bc81f82e9a4d7891f37c6fbb0a8a10/internal/lang/functions.go#L54
+		Functions: nil,
 	}
 	if len(varDefaultsM) != 0 {
 		evalCtx.Variables["var"] = cty.MapVal(varDefaultsM)
diff --git a/provisioner/terraform/tfparse/tfparse_test.go b/provisioner/terraform/tfparse/tfparse_test.go
@@ -325,8 +325,8 @@ func Test_WorkspaceTagDefaultsFromFile(t *testing.T) {
 						}
 					}`,
 			},
-			expectTags:  map[string]string{"platform": "kubernetes", "cluster": "developers", "region": "us", "az": "a"},
-			expectError: ``,
+			expectTags:  nil,
+			expectError: `Function calls not allowed; Functions may not be called here.`,
 		},
 	} {
 		tc := tc
