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

check in tpg into tfv repository #5641

Merged

Conversation

ScottSuarez
Copy link
Contributor

@ScottSuarez ScottSuarez commented Jan 24, 2022

resolves GoogleCloudPlatform/terraform-validator#323

clone tpg head branch into tfv in generation pipeline.. eventually will remove the dependency of tpgb through gomod file.

Release Note Template for Downstream PRs (will be copied)


@ScottSuarez ScottSuarez requested a review from melinath January 24, 2022 13:18
Copy link
Member

@melinath melinath left a comment

Choose a reason for hiding this comment

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

TFV has historically relied on TPG rather than TPGB (and I would prefer to stick with that, at least for now.) I am tentatively on board with the idea of generating a full copy of the Terraform provider into TFV rather than having it as a dependency since that would give us the same functionality without the associated delays - but it would be great to have a short write-up of the pros & cons of this vs other approaches, since this is a significant architectural change. I'm not sure we should block the linked PR on this since we already have a workaround (temporarily disabling the tests) and the discussion of what to do here might take a few days.

@ScottSuarez
Copy link
Contributor Author

@melinath do you have a reference for the workaround? or perhaps we could sync

@melinath
Copy link
Member

@ScottSuarez ScottSuarez changed the title check in tpgb into tfv repository check in tpg into tfv repository Jan 24, 2022
@ScottSuarez ScottSuarez requested a review from melinath January 24, 2022 23:43
@melinath
Copy link
Member

marked as safe - /gcbrun

Copy link
Member

@melinath melinath left a comment

Choose a reason for hiding this comment

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

could you make an innocuous change to a random TFV file to trigger downstream generation?

Copy link
Member

@melinath melinath left a comment

Choose a reason for hiding this comment

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

lgtm - not easily testable without merging.

@ScottSuarez ScottSuarez merged commit faac762 into GoogleCloudPlatform:master Jan 26, 2022
@ScottSuarez ScottSuarez deleted the checkin-tpgb-into-tfv-repo branch January 26, 2022 16:41
tsamaras pushed a commit to wayfair-contribs/magic-modules that referenced this pull request Feb 4, 2022
lcaggio pushed a commit to lcaggio/magic-modules that referenced this pull request Mar 16, 2022
lcaggio pushed a commit to lcaggio/magic-modules that referenced this pull request Mar 17, 2022
lcaggio pushed a commit to lcaggio/magic-modules that referenced this pull request Mar 18, 2022
betsy-lichtenberg pushed a commit to betsy-lichtenberg/magic-modules that referenced this pull request Apr 25, 2022
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.

Remove dependency on terraform-provider-google
2 participants