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

[PR #134] [MERGED] Fix 'Pipelines' CI to verify chart installing on test-namespace #203

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

📋 Pull Request Information

Original PR: https://github.com/open-webui/helm-charts/pull/134
Author: @jyje
Created: 1/6/2025
Status: ✅ Merged
Merged: 1/6/2025
Merged by: @westbrook-ai

Base: main ← Head: fix/helm-test-pipelines

📝 Commits (3)

  • 911da24 fix(ci): verify 'pipeline' chart installing on test-namespace
  • 2a1dfd7 fix(ci): add dummy change on the readme file
  • 8d6a474 fix(ci): revert changes for test

📊 Changes

1 file changed (+4 additions, -2 deletions)

View changed files

📝 .github/workflows/helm-test-pipelines.yml (+4 -2)

📄 Description

Opinion

Hello, folks again. To resolve #133, I change CI files remove the ambiguity between default and test-namespace (very similar to #125). Proposed changes are followings:

  • Create the test-namespace namespace.
  • Deploy the application in the test-namespace namespace instead of the default namespace.
  • Use kubectl wait to ensure pod/pipelines-##########-##### is ready before proceeding.

I tested proposed workflow in forked repository. Please check my commits. thx

Copilot Summary

This pull request includes a change to the Helm test pipelines workflow to improve the verification process by adding a namespace creation step and ensuring the pipeline pod is ready before proceeding.

Key changes:

  • .github/workflows/helm-test-pipelines.yml:
    • Added a command to create a new namespace test-namespace before applying the pipelines.yaml file.
    • Modified the kubectl apply command to use the newly created namespace.
    • Introduced commands to get the pipeline pod name and wait for the pod to be ready before continuing.

🔄 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/134 **Author:** [@jyje](https://github.com/jyje) **Created:** 1/6/2025 **Status:** ✅ Merged **Merged:** 1/6/2025 **Merged by:** [@westbrook-ai](https://github.com/westbrook-ai) **Base:** `main` ← **Head:** `fix/helm-test-pipelines` --- ### 📝 Commits (3) - [`911da24`](https://github.com/open-webui/helm-charts/commit/911da2497028d14711f1ca730893dc3bc1d959f5) fix(ci): verify 'pipeline' chart installing on test-namespace - [`2a1dfd7`](https://github.com/open-webui/helm-charts/commit/2a1dfd7c79ac3f0b196cfa41b531dfda3b314797) fix(ci): add dummy change on the readme file - [`8d6a474`](https://github.com/open-webui/helm-charts/commit/8d6a4745ca8015eb997b8ea83c2045102e017039) fix(ci): revert changes for test ### 📊 Changes **1 file changed** (+4 additions, -2 deletions) <details> <summary>View changed files</summary> 📝 `.github/workflows/helm-test-pipelines.yml` (+4 -2) </details> ### 📄 Description ## Opinion Hello, folks again. To resolve #133, I change CI files remove the ambiguity between `default` and `test-namespace` (very similar to #125). Proposed changes are followings: - Create the `test-namespace` namespace. - Deploy the application in the `test-namespace` namespace instead of the `default` namespace. - Use `kubectl wait` to ensure `pod/pipelines-##########-#####` is ready before proceeding. I tested proposed workflow in forked repository. Please check my commits. thx --- ## Copilot Summary This pull request includes a change to the Helm test pipelines workflow to improve the verification process by adding a namespace creation step and ensuring the pipeline pod is ready before proceeding. Key changes: * `.github/workflows/helm-test-pipelines.yml`: - Added a command to create a new namespace `test-namespace` before applying the `pipelines.yaml` file. - Modified the `kubectl apply` command to use the newly created namespace. - Introduced commands to get the pipeline pod name and wait for the pod to be ready before continuing. --- <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:50 -05:00
yindo closed this issue 2026-02-15 19:16:50 -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#203
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?