-
Notifications
You must be signed in to change notification settings - Fork 151
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 Workspace remote state consumer #292
Conversation
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.
Lookin' good, just a few more bits to add and polish up.
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.
Getting closer! I'll pull down this build and give it a spin now while we wrestle with tests and polish up the last little bits.
github.com/hashicorp/go-retryablehttp v0.6.8 // indirect | ||
github.com/hashicorp/go-slug v0.7.0 // indirect | ||
github.com/hashicorp/go-tfe v0.14.0 | ||
github.com/hashicorp/go-version v1.2.1 | ||
github.com/hashicorp/hcl v0.0.0-20180404174102-ef8a98b0bbce | ||
github.com/hashicorp/terraform-plugin-sdk/v2 v2.3.0 | ||
github.com/hashicorp/terraform-plugin-sdk/v2 v2.6.0 |
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.
? Did you intend for an SDK bump in these changes?
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.
Yup, its to get the acceptance tests to work: https://github.com/hashicorp/terraform-plugin-sdk/blob/main/CHANGELOG.md#243-february-10-2021
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.
Ah excellent!
00ebadb
to
d452f63
Compare
bfd41ad
to
f84dd98
Compare
Description
This adds
global_remote_state
andremote_state_consumer_ids
to workspaces. Read more about remote state sharing hereExternal Links
Output from acceptance tests