Skip to content

Add project-remote-state attribute to Workspace#1248

Merged
hs26gill merged 8 commits intomainfrom
hs26gill/TF-31708-Add-the-ProjectRemoteState
Dec 18, 2025
Merged

Add project-remote-state attribute to Workspace#1248
hs26gill merged 8 commits intomainfrom
hs26gill/TF-31708-Add-the-ProjectRemoteState

Conversation

@hs26gill
Copy link
Copy Markdown
Contributor

@hs26gill hs26gill commented Nov 17, 2025

Description

This PR adds a project-remote-state attribute to Workspaces. This 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
Copy Markdown

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

✅ Tests

🎉 All green!

❄️ No new flaky tests detected
🧪 All tests passed

This comment will be updated automatically if new data arrives.
🔗 Commit SHA: 78271e5 | 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
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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
@hs26gill hs26gill force-pushed the hs26gill/TF-31708-Add-the-ProjectRemoteState branch 3 times, most recently from 5dcf57b to b45e5cc Compare December 8, 2025 20:44
Copy link
Copy Markdown
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.

I don't know how specific we need to get with the CHANGELOG entry, but it's accurate to say that this is only available on Terraform Enterprise (for now).

@jbonhag jbonhag dismissed their stale review December 11, 2025 16:47

Looks good!

jbonhag
jbonhag previously approved these changes Dec 11, 2025
@hs26gill hs26gill force-pushed the hs26gill/TF-31708-Add-the-ProjectRemoteState branch from 2951ef4 to fec52e1 Compare December 17, 2025 19:49
@hs26gill hs26gill force-pushed the hs26gill/TF-31708-Add-the-ProjectRemoteState branch from fec52e1 to d526631 Compare December 18, 2025 21:27
@hs26gill hs26gill force-pushed the hs26gill/TF-31708-Add-the-ProjectRemoteState branch from d526631 to 78271e5 Compare December 18, 2025 21:29
Copy link
Copy Markdown
Collaborator

@ctrombley ctrombley left a comment

Choose a reason for hiding this comment

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

Thanks @hs26gill !

@hs26gill hs26gill merged commit ea440f1 into main Dec 18, 2025
16 checks passed
@hs26gill hs26gill deleted the hs26gill/TF-31708-Add-the-ProjectRemoteState branch December 18, 2025 22:47
@github-actions
Copy link
Copy Markdown

Reminder to the contributor that merged this PR: if your changes have added important functionality or fixed a relevant bug, open a follow-up PR to update CHANGELOG.md with a note on your changes.

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