From 3de7c31cc9cd09dda17aa375de88e483c9b98e9f Mon Sep 17 00:00:00 2001 From: Pei Ran Li Date: Mon, 14 Nov 2022 10:16:52 -0800 Subject: [PATCH] fixed katib installation helm path (#504) **Which issue is resolved by this Pull Request:** N.A **Description of your changes:** helm installation path is incorrect, results in `Chart.yaml` not found error. ``` ==========Installing katib========== Error: Chart.yaml file is missing Traceback (most recent call last): File "utils/kubeflow_installation.py", line 249, in install_kubeflow( ``` **Testing:** - [ N.A] Unit tests pass - [ N.A] e2e tests pass `make deploy-kubeflow INSTALLATION_OPTION=helm DEPLOYMENT_OPTION=s3-only` runs without errors now. ``` Waiting for katib pods to be ready ... running command: kubectl wait --for=condition=ready pod -l 'katib.kubeflow.org/component in (controller, db-manager, ui)' --timeout=240s -n kubeflow pod/katib-controller-75b988dccc-4r5j8 condition met pod/katib-db-manager-5d46869758-4lvvp condition met pod/katib-ui-766d5dc8ff-47md2 condition met All katib pods are running! ``` By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license. --- tests/e2e/resources/installation_config/s3-only.yaml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tests/e2e/resources/installation_config/s3-only.yaml b/tests/e2e/resources/installation_config/s3-only.yaml index ca7e505f5f..d8a2d9af5a 100644 --- a/tests/e2e/resources/installation_config/s3-only.yaml +++ b/tests/e2e/resources/installation_config/s3-only.yaml @@ -294,7 +294,7 @@ katib: paths: - ../../upstream/apps/katib/upstream/installs/katib-with-kubeflow helm: - paths: ../../charts/apps/katib + paths: ../../charts/apps/katib/vanilla validations: pods: namespace: kubeflow