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 backend #11

Merged
merged 2 commits into from
Jan 31, 2025
Merged

Terraform backend #11

merged 2 commits into from
Jan 31, 2025

Conversation

SamuelBeaurepaire
Copy link
Contributor

@SamuelBeaurepaire SamuelBeaurepaire commented Jan 9, 2025

Description

We (RTS Digital) now have and use a single common Terraform backend (S3 bucket + DynamoDB table) for all our terraform deployments.
We suggest to use this new backend for this repo as well.

Changes Made

  • Changed the Terraform backend configuration in all modules except 01-terraform-backend
  • Added the "srg-repository" tag to AWS resources deployed by Terraform

Actions needed

  1. Add this profile to your ~/.aws/config
[profile services-prd]
sso_start_url = https://srgssr.awsapps.com/start
sso_region = eu-central-1
sso_account_id = 404557912613
sso_role_name = Contributor
  1. Perform a terraform init -migrate-state in each module except 01-terraform-backend
  2. Perform a terraform plan in each module to check everything is alright
  3. Perform a terraform destroy in 01-terraform-backend module if its resources are actually unused
  4. Create a new PR to delete the 01-terraform-backend module files and update the project README accordingly

Checklist

  • I have followed the project's style and contribution guidelines.
  • I have performed a self-review of my own changes.
  • I have made corresponding changes to the documentation.
  • I have added tests that prove my fix is effective or that my feature works.

Copy link
Contributor

@jboix jboix left a comment

Choose a reason for hiding this comment

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

Looks good to me! I'll create a separate Pull Request to remove the backend configuration since its no longer needed.

@jboix jboix added this pull request to the merge queue Jan 31, 2025
Merged via the queue into main with commit 9e6e846 Jan 31, 2025
1 check passed
@jboix jboix deleted the terraform-backend branch January 31, 2025 12:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: ✅ Done
Development

Successfully merging this pull request may close these issues.

2 participants