Skip to content

WIP: Switch to S3 terraform backend#501

Draft
Alex-Welsh wants to merge 3 commits intomainfrom
s3-tf-backend
Draft

WIP: Switch to S3 terraform backend#501
Alex-Welsh wants to merge 3 commits intomainfrom
s3-tf-backend

Conversation

@Alex-Welsh
Copy link
Copy Markdown
Member

This change is a work in progress. Please do not merge it before it's ready. We could end up with a horrible mess if we have competing state files.

Recent changes to the free tier of terraform cloud mean we can't use it anymore. Switching to an S3-compatible object store instead.

TODO:

  • Ensure locking works
  • Ensure old statefile is removed so we atomically switch
  • Apply same change to other repos (e.g. terraform-bastion)
  • Switch to tofu

Copy link
Copy Markdown

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request updates the GitHub provider to version 6.11.1 and migrates the Terraform state backend from Terraform Cloud to a Cloudflare R2 S3-compatible storage. Review feedback suggests correcting the indentation of the backend configuration and adding a required_version constraint to ensure compatibility with the use_lockfile feature, which is only available in recent versions of Terraform and OpenTofu.

@stackhpc-ci
Copy link
Copy Markdown
Contributor

stackhpc-ci commented Apr 8, 2026

Terraform Format and Style 🖌success

Terraform Initialization ⚙️success

Terraform Validation 🤖success

Validation Output


Warning: Deprecated attribute

  on repositories.tf line 20, in resource "github_repository" "repositories":
  20:       has_downloads,

The attribute "has_downloads" is deprecated. Refer to the provider
documentation for details.
Success! The configuration is valid, but there were some validation warnings
as shown above.


Terraform Plan 📖success

Show Plan

undefined

Pusher: @Alex-Welsh, Action: pull_request, Working Directory: ``, Workflow: Terraform GitHub

@Alex-Welsh Alex-Welsh force-pushed the s3-tf-backend branch 3 times, most recently from c0a4fb3 to 86ed633 Compare April 8, 2026 14:55
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.

2 participants