Skip to content
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

graph refactor causes regression in plan #1047

Closed
pmoust opened this issue Feb 25, 2015 · 9 comments · Fixed by #1086
Closed

graph refactor causes regression in plan #1047

pmoust opened this issue Feb 25, 2015 · 9 comments · Fixed by #1086

Comments

@pmoust
Copy link
Contributor

pmoust commented Feb 25, 2015

After #1010 with a state built with 0.3.7 some nodes on terraform plan are being removed from state and added back in the execution plan, even though the state before plan is fine and matches the actual cloud state.
Reverting our terraform build pre #1010 .

I tried to narrow it down to a minified reproducible case, but no luck yet.

What would be optimal way to trace down the graph plan pre and after #1010 TF builds in order to compare?
Setting DEBUG display would suffice?

@catsby
Copy link
Contributor

catsby commented Feb 25, 2015

Probably related to / fixed by #1046

@pmoust
Copy link
Contributor Author

pmoust commented Feb 25, 2015

Indeed #1046 did it. Thanks guys.

@pmoust pmoust closed this as completed Feb 25, 2015
@catsby
Copy link
Contributor

catsby commented Feb 25, 2015

Great to hear, thank you for reporting the issue!

@pmoust
Copy link
Contributor Author

pmoust commented Feb 26, 2015

Hmm, seems we are experiencing this again even after #1046

@pmoust
Copy link
Contributor Author

pmoust commented Feb 26, 2015

Mentioned in PeoplePerHour#11 this is almost the same output as when first opened the ticket after #1010 . Now also an edge node (that is not dependant on the other changes shown in plan) is marked for record change.
And again I fail in creating a small reproducible case. The issue does exist though,and is not present in our current head.

@mitchellh
Copy link
Contributor

Let me know if you can get more luck in a repro case. I'll keep thinking about it as well.

@pmoust
Copy link
Contributor Author

pmoust commented Feb 28, 2015

TF_LOG @ PeoplePerHour#11 (comment)

@mitchellh
Copy link
Contributor

@pmoust Found it, and made a PR to fix here #1086. Note that this will cause your diff to delete two resources. If you look, these resources always existed in the state, but were always ignored. I imagine these are just orphan resources in your infrastructure that you forgot were running since TF never "saw" them, so the diff should be correct.

Whew, a doozy.

@ghost
Copy link

ghost commented May 4, 2020

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.

@ghost ghost locked and limited conversation to collaborators May 4, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants