From 557b299c4b2b6d53f1165f2e199f080663b2bbcc Mon Sep 17 00:00:00 2001 From: Paul Hinze Date: Mon, 29 Jun 2015 12:40:20 -0500 Subject: [PATCH] core: move close transforms after flatten and destroy transforms fixes destroy-related failures introduced by the close transforms --- terraform/graph_builder.go | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/terraform/graph_builder.go b/terraform/graph_builder.go index 174581b45..8a697fee8 100644 --- a/terraform/graph_builder.go +++ b/terraform/graph_builder.go @@ -117,12 +117,10 @@ func (b *BuiltinGraphBuilder) Steps(path []string) []GraphTransformer { &MissingProviderTransformer{Providers: b.Providers}, &ProviderTransformer{}, &DisableProviderTransformer{}, - &CloseProviderTransformer{}, // Provisioner-related transformations &MissingProvisionerTransformer{Provisioners: b.Provisioners}, &ProvisionerTransformer{}, - &CloseProvisionerTransformer{}, // Run our vertex-level transforms &VertexTransformer{ @@ -166,6 +164,10 @@ func (b *BuiltinGraphBuilder) Steps(path []string) []GraphTransformer { Then: &PruneDestroyTransformer{Diff: b.Diff, State: b.State}, }), + // Insert nodes to close opened plugin connections + &CloseProviderTransformer{}, + &CloseProvisionerTransformer{}, + // Make sure we have a single root after the above changes. // This is the 2nd root transformer. In practice this shouldn't // actually matter as the RootTransformer is idempotent.