Skip to content

SRE-486: Remove Terraform AWS infrastructure modules#8419

Merged
TimDiekmann merged 1 commit intomainfrom
t/sre-486-remove-unused-terraform-modules-and-move-to-labs
Feb 13, 2026
Merged

SRE-486: Remove Terraform AWS infrastructure modules#8419
TimDiekmann merged 1 commit intomainfrom
t/sre-486-remove-unused-terraform-modules-and-move-to-labs

Conversation

@TimDiekmann
Copy link
Member

🌟 What is the purpose of this PR?

Remove the Terraform AWS infrastructure code from the repository as it's no longer being used.

🔍 What does this change?

  • Removes the Terraform modules and configurations for AWS infrastructure
  • Removes the reference to Terraform in the infra/README.md
  • Deletes all AWS-specific infrastructure code including VPC, ECS, RDS, and other service configurations
  • Removes documentation related to AWS deployment

Pre-Merge Checklist 🚀

🚢 Has this modified a publishable library?

This PR:

  • does not modify any publishable blocks or libraries, or modifications do not need publishing

📜 Does this require a change to the docs?

The changes in this PR:

  • are internal and do not require a docs change

🕸️ Does this require a change to the Turbo Graph?

The changes in this PR:

  • do not affect the execution graph

🛡 What tests cover this?

  • No tests needed as this is removing unused code

❓ How to test this?

  1. Verify that the Terraform code has been completely removed
  2. Ensure the application can still be built and run locally

@TimDiekmann TimDiekmann requested a review from a team as a code owner February 13, 2026 16:09
@cursor
Copy link

cursor bot commented Feb 13, 2026

PR Summary

Low Risk
Pure deletion of unused infrastructure code and documentation; main risk is only to anyone still relying on these Terraform modules for deployments.

Overview
Removes the unused AWS Terraform infrastructure from infra/terraform, including modules for networking (VPC/subnets/PrivateLink), bastion/tunneling, ECS/ECR, Redis, Temporal workers, and related deployment/playbook documentation.

Updates infra/README.md to drop the Terraform reference, leaving only the Docker-based infra assets documented.

Written by Cursor Bugbot for commit da892a0. This will update automatically on new commits. Configure here.

@vercel
Copy link

vercel bot commented Feb 13, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
hash Ready Ready Preview, Comment Feb 13, 2026 4:17pm
petrinaut Ready Ready Preview Feb 13, 2026 4:17pm
2 Skipped Deployments
Project Deployment Actions Updated (UTC)
hashdotdesign Ignored Ignored Preview Feb 13, 2026 4:17pm
hashdotdesign-tokens Ignored Ignored Preview Feb 13, 2026 4:17pm

@github-actions github-actions bot added area/infra Relates to version control, CI, CD or IaC (area) area/infra > terraform type/legal Owned by the @legal team labels Feb 13, 2026
Copy link
Member Author

This stack of pull requests is managed by Graphite. Learn more about stacking.

@augmentcode
Copy link

augmentcode bot commented Feb 13, 2026

🤖 Augment PR Summary

Summary: Removes the repository’s AWS Terraform infrastructure, which is no longer used.

Changes:

  • Deletes the entire infra/terraform tree (modules, environment tfvars, and helper scripts)
  • Removes Terraform-related infrastructure documentation and playbooks under infra/terraform
  • Updates infra/README.md to no longer reference Terraform as an infra option
  • Eliminates AWS-specific definitions formerly covering VPC, bastion, Redis, ECR/ECS helpers, and PrivateLink/VPC hub configs

Technical Notes: This is a repository cleanup/removal-only change; remaining infra guidance focuses on Docker-based deployment.

🤖 Was this summary useful? React with 👍 or 👎

Copy link

@augmentcode augmentcode bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Review completed. 1 suggestions posted.

Fix All in Augment

Comment augment review to trigger a new review at any time.

@@ -10,4 +10,3 @@
This folder contains different kinds of infrastructure to deploy HASH.

- `docker` - Contains production Docker assets.
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Now that infra/terraform is removed, there are still repo references implying it exists (e.g. AGENTS.md lists /infra/terraform, and .github/labeler.yml has an area/infra > terraform rule). Consider updating/removing those to avoid stale documentation/labels.

Severity: low

Other Locations
  • AGENTS.md:25
  • .github/labeler.yml:61

Fix This in Augment

🤖 Was this useful? React with 👍 or 👎, or 🚀 if it prevented an incident/outage.

@TimDiekmann TimDiekmann added this pull request to the merge queue Feb 13, 2026
@graphite-app graphite-app bot requested a review from a team February 13, 2026 16:17
Merged via the queue into main with commit b1d013d Feb 13, 2026
48 checks passed
@TimDiekmann TimDiekmann deleted the t/sre-486-remove-unused-terraform-modules-and-move-to-labs branch February 13, 2026 16:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/infra > terraform area/infra Relates to version control, CI, CD or IaC (area) type/legal Owned by the @legal team

Development

Successfully merging this pull request may close these issues.

2 participants