Merge pull request #23450 from hashicorp/jbarin/err-diags

fix diagnostics handling

backend/local/backend_apply.go

@@ -65,9 +65,9 @@ func (b *Local) opApply(
 		// If we're refreshing before apply, perform that
 		if op.PlanRefresh {
 			log.Printf("[INFO] backend/local: apply calling Refresh")
-			_, err := tfCtx.Refresh()
+			_, refreshDiags := tfCtx.Refresh()
-			if err != nil {
+			diags = diags.Append(refreshDiags)
-				diags = diags.Append(err)
+			if diags.HasErrors() {
 				runningOp.Result = backend.OperationFailure
 				b.ShowDiagnostics(diags)
 				return

backend/local/backend_plan.go

@@ -86,9 +86,9 @@ func (b *Local) opPlan(
 			b.CLI.Output(b.Colorize().Color(strings.TrimSpace(planRefreshing) + "\n"))
 		}
 
-		refreshedState, err := tfCtx.Refresh()
+		refreshedState, refreshDiags := tfCtx.Refresh()
-		if err != nil {
+		diags = diags.Append(refreshDiags)
-			diags = diags.Append(err)
+		if diags.HasErrors() {
 			b.ReportResult(runningOp, diags)
 			return
 		}

configs/parser_config_dir.go

@@ -154,9 +154,9 @@ func IsEmptyDir(path string) (bool, error) {
 	}
 
 	p := NewParser(nil)
-	fs, os, err := p.dirFiles(path)
+	fs, os, diags := p.dirFiles(path)
-	if err != nil {
+	if diags.HasErrors() {
-		return false, err
+		return false, diags
 	}
 
 	return len(fs) == 0 && len(os) == 0, nil

terraform/node_resource_abstract.go

@@ -170,12 +170,12 @@ func (n *NodeAbstractResource) References() []*addrs.Reference {
 		var result []*addrs.Reference
 
 		for _, traversal := range c.DependsOn {
-			ref, err := addrs.ParseRef(traversal)
+			ref, diags := addrs.ParseRef(traversal)
-			if err != nil {
+			if diags.HasErrors() {
 				// We ignore this here, because this isn't a suitable place to return
 				// errors. This situation should be caught and rejected during
 				// validation.
-				log.Printf("[ERROR] Can't parse %#v from depends_on as reference: %s", traversal, err)
+				log.Printf("[ERROR] Can't parse %#v from depends_on as reference: %s", traversal, diags.Err())
 				continue
 			}
 

tfdiags/contextual.go

@@ -308,8 +308,8 @@ func hclRangeFromIndexStepAndAttribute(idxStep cty.IndexStep, attr *hcl.Attribut
 		}
 		stepKey := idxStep.Key.AsString()
 		for _, kvPair := range pairs {
-			key, err := kvPair.Key.Value(nil)
+			key, diags := kvPair.Key.Value(nil)
-			if err != nil {
+			if diags.HasErrors() {
 				return attr.Expr.Range()
 			}
 			if key.AsString() == stepKey {