diff --git a/terraform/context_apply_test.go b/terraform/context_apply_test.go index 203bd015d6..e4f9aafee6 100644 --- a/terraform/context_apply_test.go +++ b/terraform/context_apply_test.go @@ -2079,7 +2079,7 @@ func TestContext2Apply_provisionerDestroyForEach(t *testing.T) { if diags == nil { t.Fatal("should error") } - if !strings.Contains(diags.Err().Error(), `Reference to "each" in context without for_each`) { + if !strings.Contains(diags.Err().Error(), "each.value is unknown and cannot be used in this context") { t.Fatal("unexpected error:", diags.Err()) } } diff --git a/terraform/evaluate.go b/terraform/evaluate.go index 6681f8ddcb..9e71d3efa8 100644 --- a/terraform/evaluate.go +++ b/terraform/evaluate.go @@ -185,6 +185,16 @@ func (d *evaluationStateData) GetForEachAttr(addr addrs.ForEachAttr, rng tfdiags returnVal = d.InstanceKeyData.EachKey case "value": returnVal = d.InstanceKeyData.EachValue + + if returnVal == cty.NilVal { + diags = diags.Append(&hcl.Diagnostic{ + Severity: hcl.DiagError, + Summary: `each.value cannot be used in this context`, + Detail: fmt.Sprintf(`A reference to "each.value" has been used in a context in which it unavailable, such as when the configuration no longer contains the value in its "for_each" expression. Remove this reference to each.value in your configuration to work around this error.`), + Subject: rng.ToHCL().Ptr(), + }) + return cty.UnknownVal(cty.DynamicPseudoType), diags + } default: diags = diags.Append(&hcl.Diagnostic{ Severity: hcl.DiagError,