From 96d84da0328c86d18d1c36364d0fe523e2949005 Mon Sep 17 00:00:00 2001
From: Pam Selle <204372+pselle@users.noreply.github.com>
Date: Thu, 5 Sep 2019 15:43:50 -0400
Subject: [PATCH] Do ignore changes processing again after plan, in case
 strange things are done during plan that should not be

---
 terraform/eval_diff.go | 6 ++++++
 1 file changed, 6 insertions(+)

diff --git a/terraform/eval_diff.go b/terraform/eval_diff.go
index 6853f895c..559ab0a3e 100644
--- a/terraform/eval_diff.go
+++ b/terraform/eval_diff.go
@@ -214,6 +214,12 @@ func (n *EvalDiff) Eval(ctx EvalContext) (interface{}, error) {
 	plannedNewVal := resp.PlannedState
 	plannedPrivate := resp.PlannedPrivate
 
+	plannedNewVal, ignoreChangeDiags = n.processIgnoreChanges(priorVal, plannedNewVal)
+	diags = diags.Append(ignoreChangeDiags)
+	if ignoreChangeDiags.HasErrors() {
+		return nil, diags.Err()
+	}
+
 	if plannedNewVal == cty.NilVal {
 		// Should never happen. Since real-world providers return via RPC a nil
 		// is always a bug in the client-side stub. This is more likely caused