Skip to content

Conversation

@hs26gill
Copy link
Contributor

@hs26gill hs26gill commented Nov 17, 2025

Description

This PR adds BETA project-remote-state attribute to Workspace. This new attribute allows the workspace to share its remote state with other workspaces within the project.

Testing plan

Added unit tests: TestWorkspacesProjectRemoteState

-->

Output from tests

Including output from tests may require access to a TFE instance. Ignore this section if you have no environment to test against.

$ TFE_ADDRESS="https://example" TFE_TOKEN="example" go test ./... -v -run TestWorkspacesProjectRemoteState
Screenshot 2025-11-17 at 7 26 02 PM

@datadog-terraform-cloud-hashicorp
Copy link

datadog-terraform-cloud-hashicorp bot commented Nov 18, 2025

⚠️ Tests

⚠️ Warnings

🧪 2 Tests failed

TestPolicySetsRead from policy_set_integration_test.go (Datadog)
failed test
TestPolicySetsRead/with_policy_set_version from policy_set_integration_test.go (Datadog)

	Error Trace:	/home/runner/work/go-tfe/go-tfe/policy_set_integration_test.go:649
	            				/home/runner/go/pkg/mod/github.com/!data!dog/dd-trace-go/[email protected]/internal/civisibility/integrations/gotesting/instrumentation_orchestrion.go:242
	Error:      	Expected value not to be nil.
	Test:       	TestPolicySetsRead/with_policy_set_version

ℹ️ Info

❄️ No new flaky tests detected

This comment will be updated automatically if new data arrives.
🔗 Commit SHA: e1c2611 | Docs | Was this helpful? Give us feedback!

@hs26gill hs26gill marked this pull request as ready for review November 18, 2025 00:41
@hs26gill hs26gill requested a review from a team as a code owner November 18, 2025 00:41
@hs26gill hs26gill changed the title Add project-remote-state Add project-remote-state attribute to Workspace Nov 18, 2025
Copy link
Contributor

@Maed223 Maed223 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looking good, just need a slight adjustment in the changelog

Copy link
Contributor

@jbonhag jbonhag left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you!! The attribute changes look good, but I think we should drop the "beta" labels since this is available in TFE, and we're not calling it beta there.

@ctrombley
Copy link
Collaborator

@hs26gill would you mind rebasing this PR branch on main?

@hs26gill hs26gill force-pushed the hs26gill/TF-31708-Add-the-ProjectRemoteState branch from 6e830cc to 16e9fa0 Compare November 28, 2025 22:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants