-
Notifications
You must be signed in to change notification settings - Fork 12
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
Terraform Provider Development Program - Initial Review #2
Comments
Hi Chris, Thanks for the feedback! We can certainly split major provider versions into separate repositories if that's helpful. A quick explanation of our current plans regarding provider versions.- in version X.Y.Z The first two digits X.Y map exactly to the vTM REST API version they support, with the final digit Z being the version of the provider itself. The intent was that each new major version of our REST API would get a new terraform provider, and we'd update existing terraform providers to support new minor versions of our APIs as and when new traffic managers were released. If we need to discuss provider versioning and how it could fit into the wider terraform ecosystem a call might be a great way of covering lots of ground quickly. Regards, Nick |
…fix/VTMTF-91-terraform-import-fails-as-name-field-is-not-passed-to-obj-read to master * commit 'ac4c69b1bcf2eb2126838d19167d1cf9a50ce57c': Fix for ISSUE_1 import failures
We've added a RELEASE_NOTES.txt file, but I see it doesn't qualify as it's not in the required CHANGELOG.md format - I'll think about how to update our internal release notes generator. |
Hey Team, 👋
My name if Chris Griggs, I'm a Partner SE here at HashiCorp and will be working with you to prepare the VTM provider for its official release into the Terraform ecosystem.
It looks like you've but a lot of work into this Terraform VTM provider, which looks awesome! At first glance like you have over 40 resources and ~150 data source. Wow!
This first thing that I want to discuss with you is the two separate sets of provider code, which appears to be for two different API versions. Are you wanted to release the provider with both
4.0.0
and5.2.0
supported APIs? If so, we'll have to have a deeper conversation about the best practices of this implementation, a couple options would be, either multiple repos, or add versions in to the resource names for a single provider, something likevtm_action
andvtm_action_4
or something like that.Other then that I have some basic items that are required for the release of the provider into the Terraform ecosystem:
For a full example of the docs, just take a look and any other the current providers under the terraform-providers/ GH org.
A CHANGELOG.md is necessary for the provider release process. The latest version of the changelog must follow the naming convention of
x.x.x (Unreleased)
as shown here.The README.md should follow the standard readme structure.
Please let me know if you have any questions, we can always hop on a call to discuss the providers review process.
Best,
Chris
The text was updated successfully, but these errors were encountered: