Skip to content

remove deprecated TF#477

Merged
roisindowling merged 1 commit into
mainfrom
TFCleanup
Sep 3, 2025
Merged

remove deprecated TF#477
roisindowling merged 1 commit into
mainfrom
TFCleanup

Conversation

@roisindowling
Copy link
Copy Markdown
Contributor

No description provided.

@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud Bot commented Sep 2, 2025

@vercel
Copy link
Copy Markdown

vercel Bot commented Sep 2, 2025

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

Project Deployment Preview Comments Updated (UTC)
vevote-frontend-dev Ready Ready Preview Comment Sep 2, 2025 2:31pm

Copy link
Copy Markdown

@dougalrea dougalrea left a comment

Choose a reason for hiding this comment

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

Approved, but as mentioned on call just have to raise that managing infrastructure manually via Vercel UI, instead of via IaC in terraform, goes against devops best practices and exposes the project to risks ie infrastructure changes being made outside of the typical git permissioning and code review process, and infra changes not being trackable and revertible via the code versioning system. But as discussed those risks are acceptable for vevote.

I have gone through both dev and prod terraform states and removed all vercel infra from state files to sever the ties between terraform and vercel. All frontend infra is now managed entirely via Vercel UI

@roisindowling roisindowling merged commit bbc0110 into main Sep 3, 2025
13 checks passed
@roisindowling roisindowling deleted the TFCleanup branch September 3, 2025 13:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants