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.

Sign up for GitHub

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

Jump to bottom

clean up terraform for post-install modules #814

Merged

bobcallaway merged 2 commits into sigstore:main from bobcallaway:terraform-post-install-upgrade

Nov 5, 2023

Member

bobcallaway commented Nov 4, 2023

A few changes:

Remove providers that are not referenced in the module(s)
Relax constraints to allow upgrades
Upgrades hashicorp/helm provder to latest (kubectl provider is already at latest)
Remove label from trillian_namespace, as the helm-chart we use does not specify this and argocd reverts it immediately after terraform applies the change, and then we constantly get an "update in-place" message while running terraform plan


          clean up terraform for post-install modules

958698b

Signed-off-by: Bob Callaway <bcallaway@google.com>

bobcallaway requested review from cpanato and hectorj2f

November 4, 2023 11:47


          pin releases instead of ranges

421a10a

Signed-off-by: Bob Callaway <bcallaway@google.com>

cpanato approved these changes

View reviewed changes

Member

cpanato left a comment

cool

bobcallaway merged commit 1e0d22c into sigstore:main

9 checks passed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment