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

Terraform cleanup #26379

Open
wants to merge 7 commits into
base: main
Choose a base branch
from
Open

Terraform cleanup #26379

wants to merge 7 commits into from

Conversation

noahtalerman
Copy link
Member

@noahtalerman noahtalerman commented Feb 17, 2025

@noahtalerman: I was chatting with @edwardsb and learned that we moved the best practice Terraform (for AWS) to a separate repo here.

Now that the best practice Terraform lives there, can we remove this dogfood/aws/ folder.

Changes:

  • Remove dogfood/aws/ folder
  • Update links in docs and guides

@noahtalerman
Copy link
Member Author

noahtalerman commented Feb 17, 2025

@noahtalerman: I was chatting with @edwardsb and learned that we moved the best practice Terraform (for AWS) to a separate repo here.

Now that the best practice Terraform lives there, can we remove this dogfood/aws repo? One less things to maintain. If yes, I can update the links in various spots in the repo/docs.

@rfairburn what do you think?

cc @lukeheath

@rfairburn
Copy link
Contributor

We don't use/maintain this at all and we haven't recommended it at all since the terraform modules were initially created. I know we kept it around for a while because some customers had existing infrastructure that used the legacy dogfood terraform, but eventually this should go away. I think now is probably a good time for that to happen

@noahtalerman
Copy link
Member Author

I know we kept it around for a while because some customers had existing infrastructure that used the legacy dogfood terraform, but eventually this should go away. I think now is probably a good time for that to happen

Sounds good 👍

@rfairburn do we maintain the GCP Terraform? Wondering if we can remove that too.

@lukeheath
Copy link
Member

Fine with me 👍

@noahtalerman
Copy link
Member Author

FYI failing website test is unrelated to these changes:

Screenshot 2025-02-19 at 9 11 11 AM

rfairburn
rfairburn previously approved these changes Feb 19, 2025
@rfairburn
Copy link
Contributor

@rfairburn do we maintain the GCP Terraform? Wondering if we can remove that too.

The GCP Terraform is at least minimally maintained. It's a good starting point for anyone using GCP and we have no alternative there. I wouldn't remove it at this point, personally.

rachaelshaw
rachaelshaw previously approved these changes Feb 20, 2025
@noahtalerman
Copy link
Member Author

@rfairburn @rachaelshaw can you please take another look? I had to resolve some merge conflicts.

If everything looks good please feel free to merge to avoid future conflicts :)

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.

6 participants