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

Incubator - Upgrade Terraform to the latest version 1.7.4 #38

Open
4 tasks
gjagnoor opened this issue Oct 31, 2022 · 9 comments
Open
4 tasks

Incubator - Upgrade Terraform to the latest version 1.7.4 #38

gjagnoor opened this issue Oct 31, 2022 · 9 comments

Comments

@gjagnoor
Copy link
Member

gjagnoor commented Oct 31, 2022

Dependency

Overview

Since Terraform helps manage our entire infrastructure, it's important to keep it up to date to make sure we have to worry about the least number of bugs, and security issues. It will also be helpful for new developers joining our team so they can refer to documentation and get accurate info on how to perform different tasks

Action Items

  • Upgrade to the new version of terraform v1.7.4, incrementally
  • Update Incubator repository to work with the new version
  • Confirm that all the projects using the new version are running without trouble

Resources/Instructions

Like any other software tool, library or platform, every new patch or release comes with improvements, bug fixes, and most importantly, security fixes. This also applies to Terraform, and is the main reason to consider upgrading the tool that you use to manage your whole cloud infrastructure with. Upgrading Terraform: Why and how to?

@gjagnoor gjagnoor added the size: 3pt Can be done in 13-18 hours label Oct 31, 2022
@JasonEb
Copy link
Contributor

JasonEb commented Oct 31, 2022

This will also have to reconcile terragrunt versioning. For the year, we had to rely on a specific terragrunt version. I do believe we have always used the latest terragrunt.

@ExperimentsInHonesty
Copy link
Member

This issue needs a tiny bit of a rewrite. Let's talk about it at the next meeting on the 2022-11-09

@JasonEb
Copy link
Contributor

JasonEb commented Nov 17, 2022

After some discussion, we feel this issue would be better realized after Incubator Documentation contains guides about using Docker and Terragrunt.

Will put this it into the Icebox until we can revisit it.

@ExperimentsInHonesty ExperimentsInHonesty transferred this issue from hackforla/ops Nov 22, 2023
@robinglov
Copy link
Member

This issue needs to indicate what the version we're currently using is and point to a website where we can review any security updates to determine the time sensitivity of this issue.

@robinglov robinglov mentioned this issue Feb 14, 2024
@ExperimentsInHonesty ExperimentsInHonesty added this to the 02 - Security milestone Feb 19, 2024
@chelseybeck chelseybeck changed the title Incubator - Upgrade Terraform to the latest version 1.3.3 Incubator - Upgrade Terraform to the latest version 1.7.4 Mar 2, 2024
@chelseybeck
Copy link
Member

@robinglov I think we need a few adjustments here. I'm not seeing in the codebase where we have set a required version of Terraform. We can dig a little more to verify that...but if it is the case then the version is just whatever version someone has installed on their machine when they run commands. There is no "update" here in the traditional sense. With Terraform updates the flow (for me anyway) is typically:

  1. install the latest version on my machine
  2. run commands terraform init and terraform plan. if there are version conflicts, I'll get errors and make adjustments in the codebase as necessary

All of this said, I think that we should enforce versions in our configuration files to prevent future conflicts. This issue should be further refined

@chelseybeck
Copy link
Member

chelseybeck commented Mar 2, 2024

  • Also, I think that we should only do this for terraform projects (not the ones still on terragrunt) and then prioritize migrating remaining projects over to use terraform by taking The Terraform Migration Epic #20 out of the icebox :)

@chelseybeck
Copy link
Member

chelseybeck commented Mar 10, 2024

I checked the state files and there are different versions in different states. We will need to discuss best options

@ExperimentsInHonesty
Copy link
Member

I moved this from the icebox to the new issue review because it has a lot of unresolved comments and no dependency.

@ExperimentsInHonesty
Copy link
Member

We are putting this back into the icebox with a dependency.

current version of terraform is 1.9.8.

When this issue comes back out of the icebox, add the latest version to a comment and ask for a redraft of this issue by CoP leads

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

No branches or pull requests

5 participants