-
Notifications
You must be signed in to change notification settings - Fork 0
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
Conversation
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
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Left 1 comment
There was a problem hiding this 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 theapp_name
,provides
, andrequires
values
There was a problem hiding this 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)
* 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
* 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
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)
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
tox -e tflint
terraform apply -var "channel=latest/edge" -var "model_name=kubeflow" --auto-approve
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.