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 #172] [MERGED] feat: [pipelines] add volumeMounts and volumes #226

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

📋 Pull Request Information

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

Base: main ← Head: issue-150/pipelines-extra-volumes

📝 Commits (3)

  • 3e16c22 feat(pipelines): add volumeMounts and volumes
  • 6d1192b Merge remote-tracking branch 'upstream/main' into issue-150/pipelines-extra-volumes
  • 2a015f0 chore(pipelines): bump chart version from 0.1.0 to 0.2.0

📊 Changes

4 files changed (+28 additions, -2 deletions)

View changed files

📝 charts/pipelines/Chart.yaml (+1 -1)
📝 charts/pipelines/README.md (+3 -1)
📝 charts/pipelines/templates/deployment.yaml (+6 -0)
📝 charts/pipelines/values.yaml (+18 -0)

📄 Description

Opinion

Hello, guys. I suggest applying values for volumeMounts and volumes in the pipelines chart.

  • Adding this will allow us to include extra volumes from NFS storage, Secrets, ConfigMaps.
  • To resolve #150
    • SSH Keys can be mounted as secret volume to the pipeline container

Verification

run followings:

helm template pipelines ./charts/pipelines \
  --namespace test-namespace \
  --set "volumes[0].name=empty" \
  --set "volumes[0].emptyDir=null" \
  --set "volumeMounts[0].name=empty" \
  --set "volumeMounts[0].mountPath=/test-empty-volume" \
  > owui-pipelines-test.yaml

kubectl create ns test-namespace
kubectl apply -f owui-pipelines-test.yaml

PR

  • I changed the chart and checked docs, values
  • I skipped bumping chart version so I leave this PR as a draft
    • After #171 (v0.5.13) merged and PR for v0.5.14, we could continue this PR

🔄 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/172 **Author:** [@jyje](https://github.com/jyje) **Created:** 2/18/2025 **Status:** ✅ Merged **Merged:** 2/18/2025 **Merged by:** [@westbrook-ai](https://github.com/westbrook-ai) **Base:** `main` ← **Head:** `issue-150/pipelines-extra-volumes` --- ### 📝 Commits (3) - [`3e16c22`](https://github.com/open-webui/helm-charts/commit/3e16c22e08b4e655b6a6526bb13807159a7e3a97) feat(pipelines): add volumeMounts and volumes - [`6d1192b`](https://github.com/open-webui/helm-charts/commit/6d1192b3204168ecbb069ff02107536b62830fcc) Merge remote-tracking branch 'upstream/main' into issue-150/pipelines-extra-volumes - [`2a015f0`](https://github.com/open-webui/helm-charts/commit/2a015f01e1885098ccbc29e313e3c6841ca38346) chore(pipelines): bump chart version from 0.1.0 to 0.2.0 ### 📊 Changes **4 files changed** (+28 additions, -2 deletions) <details> <summary>View changed files</summary> 📝 `charts/pipelines/Chart.yaml` (+1 -1) 📝 `charts/pipelines/README.md` (+3 -1) 📝 `charts/pipelines/templates/deployment.yaml` (+6 -0) 📝 `charts/pipelines/values.yaml` (+18 -0) </details> ### 📄 Description ## Opinion Hello, guys. I suggest applying values for `volumeMounts` and `volumes` in the pipelines chart. - Adding this will allow us to include extra volumes from NFS storage, `Secrets`, `ConfigMaps`. - To resolve #150 - SSH Keys can be mounted as **secret volume** to the pipeline container ## Verification run followings: ```sh helm template pipelines ./charts/pipelines \ --namespace test-namespace \ --set "volumes[0].name=empty" \ --set "volumes[0].emptyDir=null" \ --set "volumeMounts[0].name=empty" \ --set "volumeMounts[0].mountPath=/test-empty-volume" \ > owui-pipelines-test.yaml kubectl create ns test-namespace kubectl apply -f owui-pipelines-test.yaml ``` ## PR - I changed the chart and checked docs, values - I skipped bumping chart version so I leave this PR as a draft - After #171 (v0.5.13) merged and PR for v0.5.14, we could continue this PR --- <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:57 -05:00
yindo closed this issue 2026-02-15 19:16:57 -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#226
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?