From 38d7ecfe2115eade18c1353eb5ebf3281758cf1c Mon Sep 17 00:00:00 2001 From: jyje Date: Fri, 13 Dec 2024 16:41:50 +0900 Subject: [PATCH] Fix CI to verify chart installing on test-namespace Remove ambiguity for `default` and `test-namespace`. It comes from `namespaceOverride` after #122. This commit helps to complete `namespaceOverride`. - create `test-namespace` namespace - install app on `test-namespace` namespace instead of `default` - (Optional) wait until `pod/open-webui-0` is ready --- .github/workflows/helm-test-open-webui.yml | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/.github/workflows/helm-test-open-webui.yml b/.github/workflows/helm-test-open-webui.yml index a830cda..f477314 100644 --- a/.github/workflows/helm-test-open-webui.yml +++ b/.github/workflows/helm-test-open-webui.yml @@ -49,4 +49,6 @@ jobs: - name: Verify open-webui run: | - kubectl apply -f open-webui.yaml \ No newline at end of file + kubectl create namespace test-namespace + kubectl apply --namespace test-namespace -f open-webui.yaml + kubectl wait --namespace test-namespace pod/open-webui-0 --for=condition=Ready --timeout=600s