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: add Terraform module for namespace-node-affinity #48

Merged
merged 7 commits into from
Sep 24, 2024

Conversation

DnPlas
Copy link
Contributor

@DnPlas DnPlas commented Sep 18, 2024

This commit adds the terraform/ directory to the root of the repository to host the Terraform module of this charm. This follows the standard set in CC006. For more information please also refer to canonical/argo-operators/pull/198.

Fixes #46

Testing instructions

  1. Check the correctness of the module with tox -e tflint
  2. Apply with terraform apply -var "channel=latest/edge" -var "model_name=kubeflow" --auto-approve
  3. Wait for the charm to be deployed.

For reviewers

This module must expose all the information in the metadata.yaml of this charm, make sure you review it by comparing that file to the Tf files in this PR.

This commit adds the terraform/ directory to the root of the repository to host
the Terraform module of this charm. This follows the standard set in CC006.
For more information please also refer to canonical/argo-operators/pull/198.

Fixes #46
@DnPlas DnPlas requested a review from a team as a code owner September 18, 2024 22:06
Copy link
Contributor

@orfeas-k orfeas-k left a comment

Choose a reason for hiding this comment

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

Left 1 comment

terraform/README.md Outdated Show resolved Hide resolved
Copy link

@mvlassis mvlassis left a comment

Choose a reason for hiding this comment

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

What I think is missing:

  • A .gitignore file with the terraform state files *.tfstate and *.terraform.
  • outputs.tf that has the app_name, provides, and requires values

Copy link
Contributor

@orfeas-k orfeas-k left a comment

Choose a reason for hiding this comment

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

Almost ready, what about adding the gitignore file that @mvlassis commented? Also, let's not forget to backport once this is merged (updating the channel in the ci too)

@DnPlas DnPlas merged commit df0bb92 into main Sep 24, 2024
9 checks passed
@DnPlas DnPlas deleted the KF-6244-namespace-nodeaffinity-terraform-module branch September 24, 2024 22:11
DnPlas added a commit that referenced this pull request Sep 24, 2024
* feat: add Terraform module for namespace-node-affinity

This commit adds the terraform/ directory to the root of the repository to host
the Terraform module of this charm. This follows the standard set in CC006.
For more information please also refer to canonical/argo-operators/pull/198.

Part of #46
DnPlas added a commit that referenced this pull request Sep 27, 2024
* feat: add Terraform module for namespace-node-affinity

This commit adds the terraform/ directory to the root of the repository to host
the Terraform module of this charm. This follows the standard set in CC006.
For more information please also refer to canonical/argo-operators/pull/198.

Part of #46
DnPlas added a commit that referenced this pull request Sep 27, 2024
This PR backports Terraform changes from `main` into `track/2.2`:

* ci, chore: use version 2.2/stable of this charm for terraform modules to match branch

* feat: add Terraform module for namespace-node-affinity (#48)
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.

Create a terraform module for operator charm
3 participants