-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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
Can't use full region form in provider block #4133
Comments
When configuring the provider, currently we only accept the name of a region. As per hashicorp/terraform-provider-google#4133, being able to use self_links for regions here would be helpful. I added a utility function to pull the region's name from a self_link, tests for that behavior, and then used it when setting up our Config object. It'll either return what the user passed in, or if it recognizes it as a region self_link, it'll just return the name part of the region. I also left it open to expand on potentially more formats in the future.
This looks like it can be resolved by just accepting either the self link to the region or the region name interchangeably in the provider. I've opened GoogleCloudPlatform/magic-modules#2119 to implement that. |
When configuring the provider, currently we only accept the name of a region. As per hashicorp/terraform-provider-google#4133, being able to use self_links for regions here would be helpful. I added a utility function to pull the region's name from a self_link, tests for that behavior, and then used it when setting up our Config object. It'll either return what the user passed in, or if it recognizes it as a region self_link, it'll just return the name part of the region. I also left it open to expand on potentially more formats in the future.
This has been merged and should be fixed with the next release. :) |
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 feel this issue should be reopened, we encourage creating a new issue linking back to this one for added context. If you feel I made an error 🤖 🙉 , please reach out to my human friends 👉 [email protected]. Thanks! |
Terraform Version
Affected Resource(s)
Terraform Configuration Files
Debug Output
Panic Output
Expected Behavior
Actual Behavior
Steps to Reproduce
terraform apply
Important Factoids
References
The text was updated successfully, but these errors were encountered: