Skip to content

Commit

Permalink
ci, docs: update to match standards for terraform (#273)
Browse files Browse the repository at this point in the history
* 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
  • Loading branch information
DnPlas authored Sep 24, 2024
1 parent d7a2442 commit 7b21655
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 4 deletions.
1 change: 0 additions & 1 deletion .github/workflows/integrate.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -55,7 +55,6 @@ jobs:
uses: canonical/charmed-kubeflow-workflows/.github/workflows/terraform-checks.yaml@main
with:
charm-path: .
channel: latest/edge
# Skipping the Terraform apply check as the mlflow-server goes to Waiting status
# instead of the expected Blocked or Active. This is currently a limitation of the
# Terraform re-usable workflows in canonical/charmed-kubeflow-workflows
Expand Down
3 changes: 0 additions & 3 deletions terraform/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -2,9 +2,6 @@

This is a Terraform module facilitating the deployment of the mlflow-server charm, using the [Terraform juju provider](https://github.com/juju/terraform-provider-juju/). For more information, refer to the provider [documentation](https://registry.terraform.io/providers/juju/juju/latest/docs).

## Compatibility
This terraform module is compatible with the mlflow-server charm 2.15/stable.

## Requirements
This module requires a `juju` model to be available. Refer to the [usage section](#usage) below for more details.

Expand Down

0 comments on commit 7b21655

Please sign in to comment.