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.

Sign up for GitHub

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

Jump to bottom

Create a terraform module for mlflow-server charm #266

Closed
DnPlas opened this issue Sep 9, 2024 · 1 comment · Fixed by #267
Closed

Create a terraform module for mlflow-server charm #266

DnPlas opened this issue Sep 9, 2024 · 1 comment · Fixed by #267
Labels
enhancement New feature or request

Comments

@DnPlas
Copy link
Contributor

DnPlas commented Sep 9, 2024

Context

According to the terraform modules organization spec, this is the first step to provide a terraform solution module. We need to create a terraform module for each charm.

What needs to get done

Create a terraform module for this charm that complies with the spec.

Definition of Done

Charm has a terraform module
@DnPlas DnPlas added the enhancement New feature or request label Sep 9, 2024
@syncronize-issues-to-jira Syncronize issues to Jira
Copy link

Thank you for reporting us your feedback!

The internal ticket has been created: https://warthogs.atlassian.net/browse/KF-6242.

This message was autogenerated

DnPlas added a commit that referenced this issue Sep 18, 2024
@DnPlas
feat: add Terraform module for mlflow-server
17eb6e7 
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 #266
@DnPlas DnPlas mentioned this issue Sep 18, 2024
Merged
DnPlas added a commit that referenced this issue Sep 18, 2024
@DnPlas
feat: add Terraform module for mlflow-server
6d3ae20 
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 #266
DnPlas added a commit that referenced this issue Sep 20, 2024
@DnPlas
feat: add Terraform module for mlflow-server (#267)
d7a2442 
* feat: add Terraform module for mlflow-server

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 #266
DnPlas added a commit that referenced this issue Sep 24, 2024
@DnPlas
ci: remove the channel from terraform checks
be3da88 
Since the default for deploying the charm is now set to latest/edge in
the re-usable workflow, this is not required anymore.

Part of #266
DnPlas added a commit that referenced this issue Sep 24, 2024
@DnPlas
docs: remove compatibility note
0781ae4 
The latest standard for the terraform/ README.md is that the compatibility
note can be obviated since the module at the branch is compatible with the
charm in the same branch.

Part of #266
@DnPlas DnPlas mentioned this issue Sep 24, 2024
Merged
DnPlas added a commit that referenced this issue Sep 24, 2024
@DnPlas
ci, docs: update to match standards for terraform (#273)
7b21655 
* ci: remove the channel from terraform checks

Since the default for deploying the charm is now set to latest/edge in
the re-usable workflow, this is not required anymore.

Part of #266

* docs: remove compatibility note

The latest standard for the terraform/ README.md is that the compatibility
note can be obviated since the module at the branch is compatible with the
charm in the same branch.

Part of #266
DnPlas added a commit that referenced this issue Sep 24, 2024
@DnPlas
feat: add Terraform module for mlflow-server (#267)
e630cdd 
* feat: add Terraform module for mlflow-server

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 #266
DnPlas added a commit that referenced this issue Sep 24, 2024
@DnPlas
ci, docs: update to match standards for terraform (#273)
a1596ce 
* ci: remove the channel from terraform checks

Since the default for deploying the charm is now set to latest/edge in
the re-usable workflow, this is not required anymore.

Part of #266

* docs: remove compatibility note

The latest standard for the terraform/ README.md is that the compatibility
note can be obviated since the module at the branch is compatible with the
charm in the same branch.

Part of #266
@DnPlas DnPlas mentioned this issue Sep 24, 2024
Merged
DnPlas added a commit that referenced this issue Sep 25, 2024
@DnPlas
feat: add Terraform module for mlflow-server (#267)
bbb435a 
* feat: add Terraform module for mlflow-server

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 #266
DnPlas added a commit that referenced this issue Sep 25, 2024
@DnPlas
ci, docs: update to match standards for terraform (#273)
252d18c 
* ci: remove the channel from terraform checks

Since the default for deploying the charm is now set to latest/edge in
the re-usable workflow, this is not required anymore.

Part of #266

* docs: remove compatibility note

The latest standard for the terraform/ README.md is that the compatibility
note can be obviated since the module at the branch is compatible with the
charm in the same branch.

Part of #266
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

feat: add Terraform module for mlflow-server canonical/mlflow-operator
1 participant
@DnPlas