Skip to content
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

[TF-18071] Handle unified id for unified resources: project, workspace, org & teams #924

Merged
merged 1 commit into from
Jun 27, 2024

Conversation

roncodingenthusiast
Copy link
Contributor

@roncodingenthusiast roncodingenthusiast commented Jun 26, 2024

Description

  • Resources managed by HCP (synced to terraform) do not use the same external ID style as traditional Terraform resources (these resources are known as unified resources). This PR relaxes validation on external ID
  • This PR adds support for unified resources support

@roncodingenthusiast roncodingenthusiast marked this pull request as ready for review June 26, 2024 16:51
@roncodingenthusiast roncodingenthusiast requested a review from a team as a code owner June 26, 2024 16:51
@roncodingenthusiast roncodingenthusiast requested a review from a team June 26, 2024 16:52
@roncodingenthusiast roncodingenthusiast force-pushed the ronald/TF-18071/relax-valid-id-fn branch from a2e83f3 to 263886a Compare June 26, 2024 17:59
joekarl
joekarl previously approved these changes Jun 26, 2024
@roncodingenthusiast roncodingenthusiast force-pushed the ronald/TF-18071/relax-valid-id-fn branch from 3e51bb1 to 8cd45e4 Compare June 26, 2024 18:47
@roncodingenthusiast roncodingenthusiast merged commit 4596f1a into main Jun 27, 2024
7 checks passed
@roncodingenthusiast roncodingenthusiast deleted the ronald/TF-18071/relax-valid-id-fn branch June 27, 2024 20:59
Copy link

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.

3 participants