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

Add gc_policy to Bigtable module, bump provider versions to 4.47 #1072

Merged
merged 6 commits into from
Dec 22, 2022

Conversation

iht
Copy link
Member

@iht iht commented Dec 22, 2022

Column families have now a new property, to specify the garbage collection policy. A new option also allows to set a default policy if none is specified.

This changes the previous syntax for column families, that was necessary since the policy is column-family specific. The new syntax makes it easier to specify a policy per column family.

I had to update the default version of the Google TF provider to 4.47, in order to match the latest options available for google_bigtable_gc_policy resources.

Column families have now a new property, to specify the garbage collection
policy. A new option also allows to set a default policy if none is specified.

This changes the previous syntax for column families, that was helpful since the
policy is column-family specific, the new syntax makes it easier to specify a
policy per column family.
@iht iht enabled auto-merge December 22, 2022 22:53
@ludoo
Copy link
Collaborator

ludoo commented Dec 22, 2022

Israel, we use the same version for all modules and blueprints. Can you bump the provider versions in default-versions.tf in the root of the repo, and then replace each of the version files? I use something like this

find . -name versions.tf -exec cp default-versions.tf "{}" \;

@ludoo ludoo added the incompatible change Pull request that breaks compatibility with previous version label Dec 22, 2022
@iht
Copy link
Member Author

iht commented Dec 22, 2022

find . -name versions.tf -exec cp default-versions.tf "{}" \;

Done. Thanks for the tip, I was not sure about how to bump the version, whether just touching the top level file would "magically" change versions elsewhere or not.

@ludoo ludoo changed the title Add gc_policy to Bigtable module Add gc_policy to Bigtable module, bump provider versions to 4.47 Dec 22, 2022
@iht iht merged commit 53135cd into GoogleCloudPlatform:master Dec 22, 2022
@iht iht deleted the bigtable_revamping branch December 22, 2022 23:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
incompatible change Pull request that breaks compatibility with previous version on:blueprints on:modules
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants