[PR #137] [MERGED] Add redis for websocket #205

Closed
opened 2026-02-15 19:16:51 -05:00 by yindo · 0 comments
Owner

📋 Pull Request Information

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

Base: mainHead: feat/websocket-with-redis


📝 Commits (10+)

  • 72a3242 ci(test): enable chart release for test
  • 1c072ce feat(websocket): add envs and pilot of redis
  • added76 feat(websocket): fix type of '_helpers.tpl'
  • 9b13a7e feat(websocket): fix labels and port and remove security contexts
  • f06ddd9 feat(websocket): change redis name and set container resources
  • 5359859 feat(websocket): fix labels and annotations
  • 51aab38 chore(websocket): revert test lines for release workflows and refactor readme and values
  • 3b1bfc2 chore(websocket): fix enabled flag
  • 0ccce3c feat(websocket): add redis-cluster from bitnami chart
  • 1e17c56 chore(websocket): fix chart dependency

📊 Changes

9 files changed (+187 additions, -4 deletions)

View changed files

📝 .github/workflows/helm-release.yml (+0 -1)
📝 charts/open-webui/Chart.lock (+3 -0)
📝 charts/open-webui/Chart.yaml (+6 -1)
📝 charts/open-webui/README.md (+23 -1)
charts/open-webui/charts/redis-20.6.2.tgz (+0 -0)
📝 charts/open-webui/templates/_helpers.tpl (+17 -1)
charts/open-webui/templates/websocket-redis.yaml (+71 -0)
📝 charts/open-webui/templates/workload-manager.yaml (+8 -0)
📝 charts/open-webui/values.yaml (+59 -0)

📄 Description

Opinion

Important

I tested this feature in local k8s and personal on-premise k8s. To verify clearly, I hope you guys test and improve with me

Hello folks.

To resolve #135, I added redis deployments for websocket supports without PV+PVC or HA settings. This is small installation scenario in here. After merging this, I will propose the installation of HA redis cluster from bitnami chart step-by-step. Please give me any advise.

Thank you


Copilot Summary

This pull request introduces websocket support to the Open WebUI Helm chart, along with several related enhancements and configurations. The most important changes include adding websocket configuration options, defining Redis deployment and service templates, and updating the chart version.

Websocket support and configuration:

  • charts/open-webui/README.md: Added websocket configuration options, including enabling websocket support, specifying the websocket manager, and configuring Redis settings.
  • charts/open-webui/values.yaml: Introduced websocket configuration parameters such as enabling websocket support, specifying the websocket manager, and defining Redis settings.

Redis deployment and service templates:

Chart version update:


🔄 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/137 **Author:** [@jyje](https://github.com/jyje) **Created:** 1/11/2025 **Status:** ✅ Merged **Merged:** 1/16/2025 **Merged by:** [@westbrook-ai](https://github.com/westbrook-ai) **Base:** `main` ← **Head:** `feat/websocket-with-redis` --- ### 📝 Commits (10+) - [`72a3242`](https://github.com/open-webui/helm-charts/commit/72a324248af067ec7b3098f3e339feefb7445266) ci(test): enable chart release for test - [`1c072ce`](https://github.com/open-webui/helm-charts/commit/1c072ce5bf7cf62f90b02e478d9cbdd897d714a4) feat(websocket): add envs and pilot of redis - [`added76`](https://github.com/open-webui/helm-charts/commit/added76a4cc2b07d312be57942e4ca2c940a6718) feat(websocket): fix type of '_helpers.tpl' - [`9b13a7e`](https://github.com/open-webui/helm-charts/commit/9b13a7e2af080701656c243e2284587bb654504d) feat(websocket): fix labels and port and remove security contexts - [`f06ddd9`](https://github.com/open-webui/helm-charts/commit/f06ddd9f2f8dcdd330a26f177db7188925fac7ff) feat(websocket): change redis name and set container resources - [`5359859`](https://github.com/open-webui/helm-charts/commit/535985967d8994519f192203af1b47158a6f7529) feat(websocket): fix labels and annotations - [`51aab38`](https://github.com/open-webui/helm-charts/commit/51aab3802986fac54decd2728929f619baedb445) chore(websocket): revert test lines for release workflows and refactor readme and values - [`3b1bfc2`](https://github.com/open-webui/helm-charts/commit/3b1bfc20d3b1800e8c5a396b6950d924207aaf42) chore(websocket): fix enabled flag - [`0ccce3c`](https://github.com/open-webui/helm-charts/commit/0ccce3cf5f456f59cccae9b42c7275dce03f698d) feat(websocket): add redis-cluster from bitnami chart - [`1e17c56`](https://github.com/open-webui/helm-charts/commit/1e17c5651dbcc06d54fcdb101419845ada1374d1) chore(websocket): fix chart dependency ### 📊 Changes **9 files changed** (+187 additions, -4 deletions) <details> <summary>View changed files</summary> 📝 `.github/workflows/helm-release.yml` (+0 -1) 📝 `charts/open-webui/Chart.lock` (+3 -0) 📝 `charts/open-webui/Chart.yaml` (+6 -1) 📝 `charts/open-webui/README.md` (+23 -1) ➕ `charts/open-webui/charts/redis-20.6.2.tgz` (+0 -0) 📝 `charts/open-webui/templates/_helpers.tpl` (+17 -1) ➕ `charts/open-webui/templates/websocket-redis.yaml` (+71 -0) 📝 `charts/open-webui/templates/workload-manager.yaml` (+8 -0) 📝 `charts/open-webui/values.yaml` (+59 -0) </details> ### 📄 Description ## Opinion > [!IMPORTANT] > I tested this feature in local k8s and personal on-premise k8s. To verify clearly, I hope you guys test and improve with me Hello folks. To resolve #135, I added redis deployments for websocket supports without PV+PVC or HA settings. This is small installation scenario in [here](https://github.com/open-webui/helm-charts/issues/135#issuecomment-2575213899). After merging this, I will propose the installation of HA redis cluster from bitnami chart step-by-step. Please give me any advise. Thank you --- ## Copilot Summary This pull request introduces websocket support to the Open WebUI Helm chart, along with several related enhancements and configurations. The most important changes include adding websocket configuration options, defining Redis deployment and service templates, and updating the chart version. Websocket support and configuration: * [`charts/open-webui/README.md`](diffhunk://#diff-f923dfcf40b056769f08b1c67dd4b43fde108d4ddbea3c27e725070d28e9003dL95-R113): Added websocket configuration options, including enabling websocket support, specifying the websocket manager, and configuring Redis settings. * [`charts/open-webui/values.yaml`](diffhunk://#diff-bbcc9d78d9f50476a0b1619e8a1e9a2377f05aeae7aed666433186247f00f30dR35-R77): Introduced websocket configuration parameters such as enabling websocket support, specifying the websocket manager, and defining Redis settings. Redis deployment and service templates: * [`charts/open-webui/templates/websocket-redis.yaml`](diffhunk://#diff-ac7508dc8b017037907968179b1fe554b72bc83b4e45b305731024e1ca8ef8f6R1-R71): Added a new template for Redis deployment and service, which includes labels, annotations, and container specifications. * [`charts/open-webui/templates/_helpers.tpl`](diffhunk://#diff-30b266d1118433d0bdc6920f21293cd7eebbec14be663c84f0523ea55b8a6d56R156-R171): Defined helper functions for creating labels and selector labels for websocket resources. Chart version update: * [`charts/open-webui/Chart.yaml`](diffhunk://#diff-fd1c91305ae5ec8685061c0996edf27e8fdf180534d5c02219ed03418765f624L3-R3): Updated the chart version from 5.1.1 to 5.2.0. --- <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:51 -05:00
yindo closed this issue 2026-02-15 19:16:51 -05:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: open-webui/helm-charts#205