-
-
Notifications
You must be signed in to change notification settings - Fork 57
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
use minimum versions for provider pinning #25
Conversation
Hey @woz5999 ! Thank you for your PR. I think I have the same issue as you do. My problem is with your PR that it would introduce the same problem but in reverse. In case of breaking changes in v4, you might need to pin the version back... |
@scream314 This scenario is anticipated and covered by the official best practice recommended by HashiCorp in the link in the description.
|
/test all |
/test all |
Thanks @woz5999 for linking to that official best practice! We've learned this lesson the hard way. Will incorporate this into our own best practices. |
what
set minimum versions for providers without pinning to a specific major version
why
the current method makes it very difficult to maintain or upgrade consistent provider versions within a project
references
https://www.terraform.io/docs/configuration/provider-requirements.html#best-practices-for-provider-versions