Skip to content

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

Draft
seb4stien wants to merge 5 commits into
mainfrom
charmkeeper/terraform
Draft

chore(terraform): align with current terraform standards (charmkeeper)#47
seb4stien wants to merge 5 commits into
mainfrom
charmkeeper/terraform

Conversation

@seb4stien

Copy link
Copy Markdown
Contributor

This PR fixes the existing Terraform module for mattermost-k8s, which was an uncustomised template copy.

Changes

  • Replace all __charm_name__ placeholders with mattermost-k8s / mattermost_k8s
  • Rename model variable → model_uuid
  • Fix Juju provider version: >= 0.20.0~> 1.0
  • Add required_version = "~> 1.12"
  • Enable terraform_required_version tflint rule in all .tflint.hcl files
  • Replace tests/main.tf with tests/setup/main.tf pattern
  • Update test revision to 18 (current latest/edge)
  • Add .github/workflows/test_terraform_module.yaml CI workflow
  • Update renovate.json: add CharmHub datasource, regex manager, ignorePaths: []

Automated by charmkeeper

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.

3 participants