open-webui/helm-charts
1
0
Fork 0
mirror of https://github.com/open-webui/helm-charts.git synced 2026-07-22 09:55:25 -04:00
Code Issues 13 Packages Projects Releases 10 Wiki Activity

[PR #125] [MERGED] Fix CI to verify chart installing on test-namespace #197

New Issue
Closed
opened 2026-02-15 19:16:48 -05:00 by yindo · 0 comments
yindo commented 2026-02-15 19:16:48 -05:00
Owner
Copy Link
Copy Source

📋 Pull Request Information

Original PR: https://github.com/open-webui/helm-charts/pull/125
Author: @jyje
Created: 12/13/2024
Status: ✅ Merged
Merged: 12/16/2024
Merged by: @westbrook-ai

Base: main ← Head: feat/open-webui

📝 Commits (1)

  • 38d7ecf Fix CI to verify chart installing on test-namespace

📊 Changes

1 file changed (+3 additions, -1 deletions)

View changed files

📝 .github/workflows/helm-test-open-webui.yml (+3 -1)

📄 Description

Opinion

Hello, folks. To resolve #124, we should remove the ambiguity between default and test-namespace, which stems from the namespaceOverride introduced in #122. This commit ensures proper completion of namespaceOverride. Proposed changes are followings:

  • Create the test-namespace namespace.
  • Deploy the application in the test-namespace namespace instead of the default namespace.
  • (Optional) Use kubectl wait to ensure pod/open-webui-0 is ready before proceeding.

And local test is following:

local test result for this PR

All resources became ready except open-webui-ollama-test-connection. That pod would be 'Completed' when recreated so we can ignore it.

If you think that using kubectl wait might cause delays or unintended lags, please let me know.

Copilot Summary

This pull request includes changes to the .github/workflows/helm-test-open-webui.yml file to enhance the deployment process for the open-webui. The most important changes include creating a new namespace, applying the configuration within that namespace, and ensuring the pod is ready before proceeding.

Deployment process improvements:

  • .github/workflows/helm-test-open-webui.yml: Added commands to create a test-namespace, apply the open-webui.yaml configuration within this namespace, and wait for the open-webui-0 pod to be ready with a timeout of 600 seconds.

🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/open-webui/helm-charts/pull/125 **Author:** [@jyje](https://github.com/jyje) **Created:** 12/13/2024 **Status:** ✅ Merged **Merged:** 12/16/2024 **Merged by:** [@westbrook-ai](https://github.com/westbrook-ai) **Base:** `main` ← **Head:** `feat/open-webui` --- ### 📝 Commits (1) - [`38d7ecf`](https://github.com/open-webui/helm-charts/commit/38d7ecfe2115eade18c1353eb5ebf3281758cf1c) Fix CI to verify chart installing on test-namespace ### 📊 Changes **1 file changed** (+3 additions, -1 deletions) <details> <summary>View changed files</summary> 📝 `.github/workflows/helm-test-open-webui.yml` (+3 -1) </details> ### 📄 Description ## Opinion Hello, folks. To resolve #124, we should remove the ambiguity between `default` and `test-namespace`, which stems from the `namespaceOverride` introduced in #122. This commit ensures proper completion of `namespaceOverride`. Proposed changes are followings: - Create the `test-namespace` namespace. - Deploy the application in the `test-namespace` namespace instead of the `default` namespace. - (Optional) Use `kubectl wait` to ensure `pod/open-webui-0` is ready before proceeding. And local test is following: ![local test result for this PR](https://github.com/user-attachments/assets/eea0479f-8aa8-44a8-ad47-470cbb77542d) All resources became ready except `open-webui-ollama-test-connection`. That pod would be 'Completed' when recreated so we can ignore it. If you think that using `kubectl wait` might cause delays or unintended lags, please let me know. --- ## Copilot Summary This pull request includes changes to the `.github/workflows/helm-test-open-webui.yml` file to enhance the deployment process for the `open-webui`. The most important changes include creating a new namespace, applying the configuration within that namespace, and ensuring the pod is ready before proceeding. Deployment process improvements: * [`.github/workflows/helm-test-open-webui.yml`](diffhunk://#diff-e56e3a533bceaee1312f7ca86210883c90d7853a9058e3b30706e81006c535c3L52-R54): Added commands to create a `test-namespace`, apply the `open-webui.yaml` configuration within this namespace, and wait for the `open-webui-0` pod to be ready with a timeout of 600 seconds. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
yindo added the pull-request label 2026-02-15 19:16:48 -05:00
yindo closed this issue 2026-02-15 19:16:49 -05:00
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels
awaiting-user-feedback
awaiting-user-feedback
bug
bug
documentation
enhancement
good first issue
help wanted
pull-request
Mirrored from GitHub Pull Request
 question
upstream
No Label pull-request
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
yindo
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: open-webui/helm-charts#197
Delete Branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?