Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: Adding Terraform module #577

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

Gmerold
Copy link

@Gmerold Gmerold commented Feb 13, 2024

Context:
Telco team is trying to build Terraform modules for the Charmed 5G, using the Juju Terraform provider. Main reason for having Terraform modules is that, unlike Juju bundles, they support day 2 operations.

This PR adds a Terraform module for the Prometheus charm.
The module uses Terraform Juju provider and allows deploying Alertmanager using Terraform.

All details regarding module's architecture are available in terraform/README.md.

In the next step, this base module will be used by the cos-lite Terraform module (prototype here).

@simskij
Copy link
Member

simskij commented Feb 13, 2024

So, we had a brief discussion about this yesterday, and we haven't really agreed on whether we want the modules scattered across the repos, or if we want all of them in the cos-lite-bundle repo. Let's park this PR until we've had a chance to decide.

@Gmerold
Copy link
Author

Gmerold commented Feb 13, 2024

So, we had a brief discussion about this yesterday, and we haven't really agreed on whether we want the modules scattered across the repos, or if we want all of them in the cos-lite-bundle repo. Let's park this PR until we've had a chance to decide.

Sure. In case you wanted something to compare against, an example of individual modules co-located with the bundle is in https://github.com/Gmerold/terraform-juju-cos-lite-k8s/tree/cos-lite-module

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.

None yet

2 participants