Skip to content

chore(terraform): align with current terraform standards (charmkeeper)#1146

Open
seb4stien wants to merge 2 commits into2/mainfrom
charmkeeper/terraform
Open

chore(terraform): align with current terraform standards (charmkeeper)#1146
seb4stien wants to merge 2 commits into2/mainfrom
charmkeeper/terraform

Conversation

@seb4stien
Copy link
Contributor

This PR fixes all Terraform modules in synapse-operator (root + 8 sub-modules + product) to align with current standards.

Changes

  • Fix Juju provider version to ~> 1.0 across all 11 modules
  • Add/fix required_version = "~> 1.12" across all modules
  • Enable terraform_required_version tflint rule in all 10 .tflint.hcl files
  • Rename modelmodel_uuid in product module (variables.tf + locals.tf)
  • Replace tests/main.tf with tests/setup/main.tf pattern
  • Update test to use model_uuid, latest/edge rev 523
  • Add missing README.md in modules/postgresql-k8s/ and modules/saml-integrator/
  • Add .github/workflows/test_terraform_module.yaml CI workflow (k8s-controller: true)
  • Update renovate.json: add ignorePaths: [], remove forbidden terraform-provider rule, fix regex manager

Automated by charmkeeper

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@seb4stien seb4stien marked this pull request as ready for review March 12, 2026 17:10
@seb4stien seb4stien requested review from a team and Thanhphan1147 as code owners March 12, 2026 17:10
@seb4stien seb4stien requested review from DeeKay3, nrobinaubertin and srbouffard and removed request for a team March 12, 2026 17:10
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