-
Notifications
You must be signed in to change notification settings - Fork 46
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
Add-on resources and data sources now retrieve output values. #822
Conversation
2b3b06c
to
f0dd42e
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Rubber stamping. Please get review from your team before merging.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks like we do the conversion 4 times because of different ways to access it. The code looks pretty much identical, should we refactor it so there is only one place that does the conversion? A simple helper function perhaps?
7200487
to
e915de2
Compare
3068c39
to
6b151c0
Compare
0e75095
to
799de0c
Compare
🛠️ Description
This updates the
hcp_waypoint_add_on
resource and data source to read Terraform output values, when present.In addition, it bumps hcp-sdk-go to v0.96.0
To test: