Merge pull request #6833 from hashicorp/jbardin/fix-interpolation

Interpolate variable during Input and Validate

terraform/context_input_test.go

@@ -569,3 +569,26 @@ func TestContext2Input_varPartiallyComputed(t *testing.T) {
 		t.Fatalf("err: %s", err)
 	}
 }
+
+// Module variables weren't being interpolated during the Input walk.
+// https://github.com/hashicorp/terraform/issues/5322
+func TestContext2Input_interpolateVar(t *testing.T) {
+	input := new(MockUIInput)
+
+	m := testModule(t, "input-interpolate-var")
+	p := testProvider("null")
+	p.ApplyFn = testApplyFn
+	p.DiffFn = testDiffFn
+
+	ctx := testContext2(t, &ContextOpts{
+		Module: m,
+		Providers: map[string]ResourceProviderFactory{
+			"template": testProviderFuncFixed(p),
+		},
+		UIInput: input,
+	})
+
+	if err := ctx.Input(InputModeStd); err != nil {
+		t.Fatalf("err: %s", err)
+	}
+}

terraform/context_validate_test.go

@@ -749,3 +749,57 @@ func TestContext2Validate_varRefFilled(t *testing.T) {
 		t.Fatalf("bad: %#v", value)
 	}
 }
+
+// Module variables weren't being interpolated during Validate phase.
+// related to https://github.com/hashicorp/terraform/issues/5322
+func TestContext2Validate_interpolateVar(t *testing.T) {
+	input := new(MockUIInput)
+
+	m := testModule(t, "input-interpolate-var")
+	p := testProvider("null")
+	p.ApplyFn = testApplyFn
+	p.DiffFn = testDiffFn
+
+	ctx := testContext2(t, &ContextOpts{
+		Module: m,
+		Providers: map[string]ResourceProviderFactory{
+			"template": testProviderFuncFixed(p),
+		},
+		UIInput: input,
+	})
+
+	w, e := ctx.Validate()
+	if w != nil {
+		t.Log("warnings:", w)
+	}
+	if e != nil {
+		t.Fatal("err:", e)
+	}
+}
+
+// When module vars reference something that is actually computed, this
+// shouldn't cause validation to fail.
+func TestContext2Validate_interpolateComputedModuleVarDef(t *testing.T) {
+	input := new(MockUIInput)
+
+	m := testModule(t, "validate-computed-module-var-ref")
+	p := testProvider("aws")
+	p.ApplyFn = testApplyFn
+	p.DiffFn = testDiffFn
+
+	ctx := testContext2(t, &ContextOpts{
+		Module: m,
+		Providers: map[string]ResourceProviderFactory{
+			"aws": testProviderFuncFixed(p),
+		},
+		UIInput: input,
+	})
+
+	w, e := ctx.Validate()
+	if w != nil {
+		t.Log("warnings:", w)
+	}
+	if e != nil {
+		t.Fatal("err:", e)
+	}
+}

terraform/eval_context_builtin.go

@@ -291,6 +291,7 @@ func (ctx *BuiltinEvalContext) Interpolate(
 			Path:     ctx.Path(),
 			Resource: r,
 		}
+
 		vs, err := ctx.Interpolater.Values(scope, cfg.Variables)
 		if err != nil {
 			return nil, err

terraform/eval_interpolate.go

@@ -1,8 +1,6 @@
 package terraform
 
-import (
+import "github.com/hashicorp/terraform/config"
-	"github.com/hashicorp/terraform/config"
-)
 
 // EvalInterpolate is an EvalNode implementation that takes a raw
 // configuration and interpolates it.

terraform/graph_config_node_output.go

@@ -44,7 +44,8 @@ func (n *GraphNodeConfigOutput) DependentOn() []string {
 // GraphNodeEvalable impl.
 func (n *GraphNodeConfigOutput) EvalTree() EvalNode {
 	return &EvalOpFilter{
-		Ops: []walkOperation{walkRefresh, walkPlan, walkApply, walkDestroy},
+		Ops: []walkOperation{walkRefresh, walkPlan, walkApply,
+			walkDestroy, walkInput, walkValidate},
 		Node: &EvalSequence{
 			Nodes: []EvalNode{
 				&EvalWriteOutput{

terraform/interpolate.go

@@ -129,15 +129,6 @@ func (i *Interpolater) valueModuleVar(
 	n string,
 	v *config.ModuleVariable,
 	result map[string]ast.Variable) error {
-	// If we're computing all dynamic fields, then module vars count
-	// and we mark it as computed.
-	if i.Operation == walkValidate {
-		result[n] = ast.Variable{
-			Value: config.UnknownVariableValue,
-			Type:  ast.TypeString,
-		}
-		return nil
-	}
 
 	// Build the path to the child module we want
 	path := make([]string, len(scope.Path), len(scope.Path)+1)

terraform/test-fixtures/input-interpolate-var/child/main.tf

@@ -0,0 +1,5 @@
+variable "list" { }
+resource "template_file" "temp" {
+  count     = "${length(split(",", var.list))}"
+  template  = "foo"
+}

terraform/test-fixtures/input-interpolate-var/main.tf

@@ -0,0 +1,7 @@
+module "source" {
+  source = "./source"
+}
+module "child" {
+  source  = "./child"
+  list = "${module.source.list}"
+}

terraform/test-fixtures/input-interpolate-var/source/main.tf

@@ -0,0 +1 @@
+output "list" { value = "foo,bar,baz" }

terraform/test-fixtures/validate-computed-module-var-ref/dest/main.tf

@@ -0,0 +1,5 @@
+variable "destin" { }
+
+resource "aws_instance" "dest" {
+  attr = "${var.destin}"
+}

terraform/test-fixtures/validate-computed-module-var-ref/main.tf

@@ -0,0 +1,8 @@
+module "source" {
+  source = "./source"
+}
+
+module "dest" {
+  source = "./dest"
+  destin = "${module.source.sourceout}"
+}

terraform/test-fixtures/validate-computed-module-var-ref/source/main.tf

@@ -0,0 +1,3 @@
+resource "aws_instance" "source" { }
+
+output "sourceout" { value = "${aws_instance.source.id}" }