Skip to content

Commit

Permalink
improve formatting of path in error message
Browse files Browse the repository at this point in the history
  • Loading branch information
kmoe committed Feb 17, 2022
1 parent d6cc3d9 commit 9794650
Showing 1 changed file with 9 additions and 2 deletions.
11 changes: 9 additions & 2 deletions internal/terraform/node_resource_validate.go
Expand Up @@ -2,12 +2,12 @@ package terraform

import (
"fmt"
"strings"

"github.com/hashicorp/hcl/v2"
"github.com/hashicorp/terraform/internal/addrs"
"github.com/hashicorp/terraform/internal/configs"
"github.com/hashicorp/terraform/internal/configs/configschema"
"github.com/hashicorp/terraform/internal/configs/hcl2shim"
"github.com/hashicorp/terraform/internal/didyoumean"
"github.com/hashicorp/terraform/internal/providers"
"github.com/hashicorp/terraform/internal/provisioners"
Expand Down Expand Up @@ -395,10 +395,17 @@ func (n *NodeValidatableResource) validateResource(ctx EvalContext) tfdiags.Diag
attrSchema := schema.AttributeByPath(path)

if attrSchema != nil && !attrSchema.Optional && attrSchema.Computed {
// ignore_changes uses absolute traversal syntax in config despite
// using relative traversals, so we strip the leading "." added by
// FormatCtyPath for a better error message.
attrDisplayPath := tfdiags.FormatCtyPath(path)
if strings.HasPrefix(attrDisplayPath, ".") {
attrDisplayPath = attrDisplayPath[1:]
}
diags = diags.Append(&hcl.Diagnostic{
Severity: hcl.DiagWarning,
Summary: "Invalid ignore_changes",
Detail: fmt.Sprintf("ignore_changes cannot be used with non-Optional Computed attributes. Please remove \"%s\" from ignore_changes.", hcl2shim.FlatmapKeyFromPath(path)),
Detail: fmt.Sprintf("ignore_changes cannot be used with non-Optional Computed attributes. Please remove \"%s\" from ignore_changes.", attrDisplayPath),

Subject: &n.Config.TypeRange,
})
Expand Down

0 comments on commit 9794650

Please sign in to comment.