Skip to content

Conversation

MichaelThamm
Copy link
Contributor

@MichaelThamm MichaelThamm commented Jul 17, 2025

Fixes canonical/observability-stack#39

Sample implementation:

TODO:

Warning

If merged, this will make its way into the v2 track tag, meaning that all charms repo PRs which have .tf file changes will need to conform to the Terraform CI standard. This includes having:

  1. justfile
  2. .tfdocs-config.yml

Without this, PRs will need to be merged on failing TF CI, which is not blocking.

This PR also introduces a linting check which ensures that the endpoints in the outputs.tf file is up-to-date with the charmcraft.yaml. E.g. In the alertmanager-k8s repo you can see we are already diverging:

❯ just lint-tf-endpoints
service-mesh is missing from endpoints
require-cmr-mesh is missing from endpoints
provide-cmr-mesh is missing from endpoints
error: Recipe `lint-tf-endpoints` failed with exit code 1

@lucabello lucabello changed the title feat: Central CI for TF feat!: Central CI for TF Aug 4, 2025
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.

Move Terraform lint, format, and validate Just recipes to central workflows in the observability repo

1 participant