[PR #283] [MERGED] feat: Allow redis service port name to be set by chart. #302

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

📋 Pull Request Information

Original PR: https://github.com/open-webui/helm-charts/pull/283
Author: @swarren-sumglobal
Created: 8/8/2025
Status: Merged
Merged: 8/8/2025
Merged by: @westbrook-ai

Base: mainHead: main


📝 Commits (1)

  • 749f2e1 feat: Allow redis service port name to be set by chart. Istio needs a name like tcp-redis instead of http

📊 Changes

6 files changed (+11 additions, -6 deletions)

View changed files

📝 charts/open-webui/Chart.yaml (+1 -1)
📝 charts/open-webui/README.md (+4 -3)
📝 charts/open-webui/templates/websocket-redis.yaml (+1 -1)
📝 charts/open-webui/values-gke-min.yaml (+2 -0)
📝 charts/open-webui/values.yaml (+2 -0)
📝 charts/pipelines/README.md (+1 -1)

📄 Description

Need to be able to set the redis service name. Istio incorrectly configures its proxy with the default name of 'http'. It needs a name like 'tcp-redis' to correctly discern the configuration for the proxy


🔄 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/283 **Author:** [@swarren-sumglobal](https://github.com/swarren-sumglobal) **Created:** 8/8/2025 **Status:** ✅ Merged **Merged:** 8/8/2025 **Merged by:** [@westbrook-ai](https://github.com/westbrook-ai) **Base:** `main` ← **Head:** `main` --- ### 📝 Commits (1) - [`749f2e1`](https://github.com/open-webui/helm-charts/commit/749f2e11b6b3f06db443f5168db4a067444f895a) feat: Allow redis service port name to be set by chart. Istio needs a name like tcp-redis instead of http ### 📊 Changes **6 files changed** (+11 additions, -6 deletions) <details> <summary>View changed files</summary> 📝 `charts/open-webui/Chart.yaml` (+1 -1) 📝 `charts/open-webui/README.md` (+4 -3) 📝 `charts/open-webui/templates/websocket-redis.yaml` (+1 -1) 📝 `charts/open-webui/values-gke-min.yaml` (+2 -0) 📝 `charts/open-webui/values.yaml` (+2 -0) 📝 `charts/pipelines/README.md` (+1 -1) </details> ### 📄 Description Need to be able to set the redis service name. Istio incorrectly configures its proxy with the default name of 'http'. It needs a name like 'tcp-redis' to correctly discern the configuration for the proxy --- <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:17:17 -05:00
yindo closed this issue 2026-02-15 19:17:17 -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#302