Skip to content

Commit

Permalink
comment update and remove extra Name method
Browse files Browse the repository at this point in the history
This name method won't be called in the full graph, and remove it to
prevent confusion with the parent node in logs.
  • Loading branch information
jbardin committed Mar 26, 2020
1 parent 4f1692c commit cec989d
Showing 1 changed file with 3 additions and 8 deletions.
11 changes: 3 additions & 8 deletions terraform/node_resource_apply.go
Original file line number Diff line number Diff line change
Expand Up @@ -9,10 +9,9 @@ import (
)

// nodeExpandApplyableResource handles the first layer of resource
// expansion during apply. This is required because EvalTree does not have a
// context with which to expand the resource into multiple instances.
// This type should be a drop in replacement for NodeApplyableResource, and
// needs to mirror any non-evaluation methods exactly.
// expansion during apply. Even though the resource instances themselves are
// already expanded from the plan, we still need to expand the
// NodeApplyableResource nodes into their respective modules.
type nodeExpandApplyableResource struct {
*NodeAbstractResource
}
Expand Down Expand Up @@ -74,10 +73,6 @@ func (n *NodeApplyableResource) Path() addrs.ModuleInstance {
return n.Addr.Module
}

func (n *NodeApplyableResource) Name() string {
return n.NodeAbstractResource.Name() + " (prepare state)"
}

func (n *NodeApplyableResource) References() []*addrs.Reference {
if n.Config == nil {
log.Printf("[WARN] NodeApplyableResource %q: no configuration, so can't determine References", dag.VertexName(n))
Expand Down

0 comments on commit cec989d

Please sign in to comment.