From 7b216554396a9a3b71d4fbf5dd9079aef5f5c0fe Mon Sep 17 00:00:00 2001 From: Daniela Plascencia Date: Tue, 24 Sep 2024 15:05:57 +0200 Subject: [PATCH] ci, docs: update to match standards for terraform (#273) * 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 --- .github/workflows/integrate.yaml | 1 - terraform/README.md | 3 --- 2 files changed, 4 deletions(-) diff --git a/.github/workflows/integrate.yaml b/.github/workflows/integrate.yaml index e9fe6fb3..24232940 100644 --- a/.github/workflows/integrate.yaml +++ b/.github/workflows/integrate.yaml @@ -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 diff --git a/terraform/README.md b/terraform/README.md index 2de68820..dec3fa40 100644 --- a/terraform/README.md +++ b/terraform/README.md @@ -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.