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

Update code base to a current Terraform version #1265

Open
agold-rh opened this issue Jun 4, 2024 · 4 comments
Open

Update code base to a current Terraform version #1265

agold-rh opened this issue Jun 4, 2024 · 4 comments
Labels
backlog bug Something isn't working

Comments

@agold-rh
Copy link

agold-rh commented Jun 4, 2024

TL;DR

The documents (README) for this code base indicates that Terraform version 1.3.10 should be used to for execution. Current Terraform is 1.8.4 as of this writing, and 1.3.10 is a version no longer supported by Hashicorp. I'm listing this as a bug, because creating new code for an unsupported tf version that's reached EOL is problematic.

Expected behavior

I expect most currently supported Terraform versions to execute correctly.
Screenshot 2024-06-04 at 15 18 57

Observed behavior

Screenshot 2024-06-04 at 15 17 45

Terraform Configuration

None

Terraform Version

None

Additional information

A requirement to use an obsolete version of TF for correct execution is a red flag for any cautious, security conscious software engineer. Since this repo was created to conveniently provide a security conscious landing zone, a TF update seems desirable.

@agold-rh agold-rh added the bug Something isn't working label Jun 4, 2024
@fmichaelobrien
Copy link
Contributor

FYI for cloud build users

An upgrade past 1.3.10 in the DockerFile used in cloud build will benefit from the upgrade - for now we downgrade to 1.3 to avoid a statefile corruption between 0-bootstrap and 1-org
#1151
workaround
GoogleCloudPlatform/pbmm-on-gcp-onboarding#374

@bdashrad
Copy link
Contributor

I've been running my own clone of this on terraform 1.8 on github after modifying the deploy scripts.

@eeaton
Copy link
Collaborator

eeaton commented Jun 20, 2024

We've partially updated the version to 1.5.7 (last version before hashicorp changed the license). However, I acknowledge that this doesn't entirely address this issue, because versions 1.6 and earlier are out of support.

I'm seeking legal confirmation internally whether we can use versions after 1.6 with the BSL license changes from Hashicorp. In the meantime we've added text emphasizing that users of the blueprint can upgrade terraform to the latest version so long as they ensure that the TF version used in manual steps and in build pipeline are the same.

@eeaton eeaton added the backlog label Jun 27, 2024
Copy link

This issue is stale because it has been open 60 days with no activity. Remove stale label or comment or this will be closed in 7 days

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backlog bug Something isn't working
Projects
None yet
Development

No branches or pull requests

5 participants