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.

Sign up for GitHub

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

Jump to bottom

Terraform cloud crashes on plan (same configuration worked few weeks ago) #505

Closed
liarco opened this issue Oct 22, 2020 · 2 comments · Fixed by #506
Closed

Terraform cloud crashes on plan (same configuration worked few weeks ago) #505

liarco opened this issue Oct 22, 2020 · 2 comments · Fixed by #506
Assignees
@andrewsomething
Labels
bug regression

Comments

@liarco
Copy link

liarco commented Oct 22, 2020

Versions

Terraform: v0.13.5
DigitalOcean provider: v2.0.0

Affected Resource(s)

The issue doesn't seem to be related to any specific resource.

Terraform Configuration Files & Debug Output

Complete gist: https://gist.github.com/liarco/99f3c59ebc0c3996986b2c654b2b5f09

I'm using Terraform Cloud with remote execution and I can't find a way to get crash.log.

I'm sorry that I can't provide more information, but please let me know if there's anything I can do to help you understanding the issue.
@andrewsomething andrewsomething self-assigned this Oct 22, 2020
andrewsomething added a commit that referenced this issue Oct 22, 2020
@andrewsomething
resourceDigitalOceanCDNRead: return nil on success (Closes: #505).
950a0d4
@andrewsomething andrewsomething mentioned this issue Oct 22, 2020
Merged
andrewsomething added a commit that referenced this issue Oct 22, 2020
@andrewsomething
Fix panics introduced in move from ReadFunc to ReadContextFunc (#506)
295deb7 
* resourceDigitalOceanCDNRead: return nil on success (Closes: #505).

* resourceDigitalOceanAppRead: return nil on success.
@andrewsomething
Copy link
Member

Thank you for flagging this issue @liarco. We've just releases v2.0.1 that should resolve it.

terraform init -upgrade

Apologizes for the disruption.

@tdyas tdyas mentioned this issue Oct 24, 2020
Merged
@tdyas
Copy link
Contributor

tdyas commented Oct 24, 2020

I missed that change in semantic of diag.FromErr in doing the upgrade. :( I've sent a PR to change the SDK to handle nil errors passed to that method. hashicorp/terraform-plugin-sdk#623

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@andrewsomething andrewsomething

Labels
bug regression
Projects
None yet
Milestone
No milestone
3 participants
@andrewsomething @tdyas @liarco