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

Panic if no account credentials are available in ARM #4697

Closed
jen20 opened this issue Jan 16, 2016 · 2 comments
Closed

Panic if no account credentials are available in ARM #4697

jen20 opened this issue Jan 16, 2016 · 2 comments

Comments

@jen20
Copy link
Contributor

jen20 commented Jan 16, 2016

Registering the provider types needs to happen after the credentials check: if no credentials are available via environment variables or directly, a panic such as this occurs:

$ terraform plan
Refreshing Terraform state prior to plan...

Error refreshing state: 1 error(s) occurred:

* unexpected EOF
panic: runtime error: invalid memory address or nil pointer dereference
2016/01/16 08:36:14 [DEBUG] terraform-provider-azurerm: [signal 0xb code=0x1 addr=0x10 pc=0x1ba5c7]
2016/01/16 08:36:14 [DEBUG] terraform-provider-azurerm:
2016/01/16 08:36:14 [DEBUG] terraform-provider-azurerm: goroutine 58 [running]:
2016/01/16 08:36:14 [DEBUG] terraform-provider-azurerm: github.com/Azure/azure-sdk-for-go/arm/resources/resources.ProvidersClient.Register(0x10d2270, 0xc820210240, 0x10d2298, 0xc8201dfe40, 0x0, 0x0, 0x9442e0, 0x13, 0x0, 0xd18c2e2800, ...)
2016/01/16 08:36:14 [DEBUG] terraform-provider-azurerm:     /Users/James/Code/go/src/github.com/Azure/azure-sdk-for-go/arm/resources/resources/providers.go:206 +0x217
2016/01/16 08:36:14 [DEBUG] terraform-provider-azurerm: github.com/hashicorp/terraform/builtin/providers/azurerm.registerAzureResourceProvidersWithSubscription(0xc820204280, 0xc820258000, 0x0, 0x0)
2016/01/16 08:36:14 [DEBUG] terraform-provider-azurerm:     /Users/James/Code/go/src/github.com/hashicorp/terraform/builtin/providers/azurerm/provider.go:101 +0x152
@catsby
Copy link
Contributor

catsby commented Feb 17, 2016

#4902 was merged, think I'll close this...

@catsby catsby closed this as completed Feb 17, 2016
@ghost
Copy link

ghost commented Apr 28, 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 Apr 28, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

2 participants