[PR #526] [MERGED] Support Redis Cluster #530

Closed
opened 2026-02-16 02:16:33 -05:00 by yindo · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/langchain-ai/helm/pull/526
Author: @romain-priour-lc
Created: 11/30/2025
Status: Merged
Merged: 12/1/2025
Merged by: @romain-priour-lc

Base: mainHead: romain/redis-cluster-support-langsmith


📝 Commits (10+)

📊 Changes

7 files changed (+186 additions, -16 deletions)

View changed files

📝 charts/langsmith/Chart.yaml (+2 -2)
📝 charts/langsmith/README.md (+13 -7)
charts/langsmith/examples/redis_cluster.yaml (+123 -0)
📝 charts/langsmith/templates/_helpers.tpl (+19 -0)
📝 charts/langsmith/templates/redis/secrets.yaml (+6 -1)
📝 charts/langsmith/templates/validate.yaml (+7 -0)
📝 charts/langsmith/values.yaml (+16 -6)

📄 Description

  • Adds support for Redis cluster in LangSmith
  • Can either use existing secret, or pass in node URIs and password directly through the values
  • Password defaults to empty string

🔄 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/langchain-ai/helm/pull/526 **Author:** [@romain-priour-lc](https://github.com/romain-priour-lc) **Created:** 11/30/2025 **Status:** ✅ Merged **Merged:** 12/1/2025 **Merged by:** [@romain-priour-lc](https://github.com/romain-priour-lc) **Base:** `main` ← **Head:** `romain/redis-cluster-support-langsmith` --- ### 📝 Commits (10+) - [`f57b111`](https://github.com/langchain-ai/helm/commit/f57b1114dc71a3f09dfdeb7d389883c594c044ff) Add support for redis cluster - [`a16b7a4`](https://github.com/langchain-ai/helm/commit/a16b7a456fa77f931650181780c0bf758b4250e7) Update readme - [`5489861`](https://github.com/langchain-ai/helm/commit/54898610899fe66f11690f1da84102297b979bf3) Add comment about node URI format - [`42b2945`](https://github.com/langchain-ai/helm/commit/42b2945a1ca393601b86bd689066830eb34ad321) Add a redis cluster example file - [`9ac9ca9`](https://github.com/langchain-ai/helm/commit/9ac9ca9fb0e062adbc22e91a0eb4f91ca2c67220) Add ability to customize secret key - [`a4e58c5`](https://github.com/langchain-ai/helm/commit/a4e58c50ac97448299db9c5757bbf25c98d7e5e9) update README - [`14bf67d`](https://github.com/langchain-ai/helm/commit/14bf67d213e90bfc285b2b129b3c119e6bd12893) comment out readme secret keys - [`12f9af3`](https://github.com/langchain-ai/helm/commit/12f9af3fd076abf27b3f547c817fe3b4db91cb70) default redis_node_uris - [`282e668`](https://github.com/langchain-ai/helm/commit/282e668f91d3afa80b998c9e0eca8466ca4fef80) update image version - [`01e8206`](https://github.com/langchain-ai/helm/commit/01e82068b0c0f68961c94e28f7409dd81d268d05) README update ### 📊 Changes **7 files changed** (+186 additions, -16 deletions) <details> <summary>View changed files</summary> 📝 `charts/langsmith/Chart.yaml` (+2 -2) 📝 `charts/langsmith/README.md` (+13 -7) ➕ `charts/langsmith/examples/redis_cluster.yaml` (+123 -0) 📝 `charts/langsmith/templates/_helpers.tpl` (+19 -0) 📝 `charts/langsmith/templates/redis/secrets.yaml` (+6 -1) 📝 `charts/langsmith/templates/validate.yaml` (+7 -0) 📝 `charts/langsmith/values.yaml` (+16 -6) </details> ### 📄 Description - Adds support for Redis cluster in LangSmith - Can either use existing secret, or pass in node URIs and password directly through the values - Password defaults to empty string --- <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-16 02:16:33 -05:00
yindo closed this issue 2026-02-16 02:16:33 -05:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: langchain-ai/helm#530