-
Notifications
You must be signed in to change notification settings - Fork 2
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
TERRAFORM CRASH: err="connection is shut down" #30
Comments
Thanks for sending. Sorry it crashed, that's not ideal. I see that some alert conditions are being set to to have a threshold of "0" which we figured out would fail due to a bug in the Go client but was fixed in here: appoptics/appoptics-api-go#70 I think we should probably update the client dependency to the latest version and perhaps re-run. If you wanted to give it a shot, you can run |
Hi again! I actually don't know why it looks like the threshold is 0. It is not, and has never been. The alert has been created by Terraform in the first place, so it should never have been able to be created with that value. Anyways, Terraform has been crashing several times now, and It's actually causing so many issues that I'm unable to work with it anymore. 1.I just tried to change the alert condition for 24 alerts. They where all setup correctly, and a 2.On the first run, it successfully changed the condition of 11 alerts before crashing. 3.After applying the above change, terraform crashed almost instantly. And now the apply fails for every alert:
It is failing because all of the alerts are already exists in AO. So in some way the alerts has been removed from the state. |
Hey guys I'm just now seeing this thread (was at Tech Summit in Brno all last week and basically didn't check on anything else). We've landed the PR mentioned above. I'm going to be doing dev on this thing all week this week, so I may reach out with some questions. |
@sandnabba FYI in Go, you can't have un-initialized memory (for safety reasons), so every type (and therefore every member of a struct) has a "zero value". The zero value of any Int type is, well, |
Today, when applying some changes, Terraform crashed with this message:
Efter running apply again, it now crashes with:
I think that the crashes in because it's trying to create an alert that already exists.
I'll try to clean up my state to see if I can reproduce this error.
The text was updated successfully, but these errors were encountered: