comment update and remove extra Name method

This name method won't be called in the full graph, and remove it to
prevent confusion with the parent node in logs.
This commit is contained in:
James Bardin 2020-03-26 11:52:41 -04:00
parent 4f1692cfaf
commit cec989d660
1 changed files with 3 additions and 8 deletions

View File

@ -9,10 +9,9 @@ import (
) )
// nodeExpandApplyableResource handles the first layer of resource // nodeExpandApplyableResource handles the first layer of resource
// expansion during apply. This is required because EvalTree does not have a // expansion during apply. Even though the resource instances themselves are
// context with which to expand the resource into multiple instances. // already expanded from the plan, we still need to expand the
// This type should be a drop in replacement for NodeApplyableResource, and // NodeApplyableResource nodes into their respective modules.
// needs to mirror any non-evaluation methods exactly.
type nodeExpandApplyableResource struct { type nodeExpandApplyableResource struct {
*NodeAbstractResource *NodeAbstractResource
} }
@ -74,10 +73,6 @@ func (n *NodeApplyableResource) Path() addrs.ModuleInstance {
return n.Addr.Module return n.Addr.Module
} }
func (n *NodeApplyableResource) Name() string {
return n.NodeAbstractResource.Name() + " (prepare state)"
}
func (n *NodeApplyableResource) References() []*addrs.Reference { func (n *NodeApplyableResource) References() []*addrs.Reference {
if n.Config == nil { if n.Config == nil {
log.Printf("[WARN] NodeApplyableResource %q: no configuration, so can't determine References", dag.VertexName(n)) log.Printf("[WARN] NodeApplyableResource %q: no configuration, so can't determine References", dag.VertexName(n))