-
Notifications
You must be signed in to change notification settings - Fork 233
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
Determine what protocols are supported #786
Comments
I'm having the same issue, can anyone shed a light on this? |
Hi @reneherrero 👋 Thank you for raising this. Providers written in v2 of this framework are implemented using a protocol version 5.0 server (essentially requiring Terraform CLI 0.12+). Uploads to the public Terraform Registry default to this protocol version, unless a Terraform Registry manifest file is included with a different protocols value. Will certainly keep this open to document this better. |
Cool, thanks for the info @bflad |
As part of #857, newer SDK logging messages implemented via terraform-plugin-log will now include a structured logging key Formal documentation for Terraform plugin protocol versions in https://terraform.io/plugin will be introduced as part of hashicorp/terraform-plugin-mux#30. In particular, the terraform-plugin-sdk/v2 documentation will include this information at the top: Terraform Plugin SDKv2 is an established way to develop Terraform Plugins on Protocol Version 5 (where "Protocol Version 5" is a link to a new section describing what that is). Since I believe both of those should cover this, I'm going to close this out. Thanks again for raising this! |
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. |
SDK version
Use-cases
Provide a way to determine what protocols are supported (ex: 5.1) by the provider referencing this library.
This is a mandatory field introduced by the Provider Registry Protocol: https://www.terraform.io/docs/internals/provider-registry-protocol.html#response-properties-1
Attempted Solutions
Proposal
Not sure what's the best way to expose this info.
Putting the info in the
README.md
would seem like an easy first step.References
N/A
The text was updated successfully, but these errors were encountered: