-
Notifications
You must be signed in to change notification settings - Fork 9.7k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Templates not working with latest master branch #2495
Comments
Thanks for the report. It could be related to #2406 - will take a look. |
Can you show us your configuration? Or a debug log? That'll help debug this. |
Yes, was just getting that - the debug log is so large it's crashing my browser on gists so looking for another way to post it. |
|
following up on this, looks like the issue first appeared in 3815122 but not sure if this is intended or not. |
I also ran into this problem when I upgraded to terraform master this morning. I was able to do a git bisect on terraform against our configuration and found that 0b1dbf3 was the commit that broke it. We have 5 different providers in our configuration and it doesn't seem to be selective or predictable in which provider it has connection problems with. Sometimes it's multiple. I've reverted to the commit before 0b1dbf3, and can verify that things work great as usual. |
I'm working on reproing this now. The config from @mikeyhill has a bunch of dependencies I'm trying to meet - @rubbish if you happen to have a simpler repro case let me know. |
@phinze So it's happening when I'm doing a targeted plan against a certain module. I'll gist up an example. One sec. |
@phinze Here's an example in
and in
When I run
But sometimes it works. Sometimes it comes back with:
|
Also, when doing a full terraform plan, I've not been able to get the connection errors. |
@rubbish perfect thank you - on it now |
When targeting prunes out all the resource nodes between a provider and its close node, there was no dependency to ensure the close happened after the configure. Needed to add an explicit dependency from the close to the provider. fixes #2495
@phinze - thanks! apologies on the useless config sample above, I'll keep that in mind that you need a working example rather than the actual configs - I'm still a bit green with this. |
When targeting prunes out all the resource nodes between a provider and its close node, there was no dependency to ensure the close happened after the configure. Needed to add an explicit dependency from the close to the provider. This tweak highlighted the fact that CloseProviderTransformer needed to happen after DisableProviderTransformer, since DisableProviderTransformer inspects up-edges to decide what to disable, and CloseProviderTransformer adds an up-edge. fixes #2495
Not sure if this test has value /cc @mitchellh (who requested one be added) to see what I might be missing here. refs #2495
I'm going to lock this issue because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active issues. If you have found a problem that seems similar to this, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further. |
Hey, I just updated TF to pull in some changes and noticed that all of our templates stopped working. If I move back to a previous release all is well. It's been quite a while (a week?) since I updated TF so I don't have any info about which commit this stops working at but for now have reverted back.
Occasionally I'll get multiple lines the same as above. Is this an issue with TF or have the templates changed? I don't see any differences in the documentation.
version: Terraform v0.6.0-dev (c1bb9116ad1899209bc87eae6122b478aee4d839)
The text was updated successfully, but these errors were encountered: