[PR #548] [CLOSED] v13: KEDA support for LangSmith queues + beacon logging + few issue fixes + deprecate basicauth initialOrgAdminEmail #552

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

📋 Pull Request Information

Original PR: https://github.com/langchain-ai/helm/pull/548
Author: @romain-priour-lc
Created: 1/6/2026
Status: Closed

Base: mainHead: romain/enable-keda-langsmith


📝 Commits (10+)

  • f22dc94 Support KEDA autoscaling in helm. Need to test
  • 477fcf4 Load tested, scale up happened
  • 72d0172 Update autoscaling config example
  • d92f92a Remove deprecated initialOrgAdminEmail from the basicauth section
  • 147b0e9 scale down non-ingest queue replicas to 1 by default
  • 1c0cbcb Added KEDA to all services for symmetry + updated docs and examples. Need to test
  • 75d15cb Tested end to end
  • ec00cb8 Update README
  • a3bd667 Add logs to beacon
  • dbc3d67 update README

📊 Changes

32 files changed (+1279 additions, -244 deletions)

View changed files

📝 charts/langsmith/README.md (+205 -58)
📝 charts/langsmith/examples/autoscaling_config.yaml (+168 -21)
📝 charts/langsmith/templates/ace-backend/deployment.yaml (+1 -1)
📝 charts/langsmith/templates/ace-backend/hpa.yaml (+9 -9)
charts/langsmith/templates/ace-backend/scaled-object.yaml (+56 -0)
📝 charts/langsmith/templates/backend/deployment.yaml (+1 -1)
📝 charts/langsmith/templates/backend/hpa.yaml (+9 -9)
charts/langsmith/templates/backend/scaled-object.yaml (+56 -0)
📝 charts/langsmith/templates/config-map.yaml (+2 -0)
📝 charts/langsmith/templates/frontend/deployment.yaml (+1 -1)
📝 charts/langsmith/templates/frontend/hpa.yaml (+9 -9)
charts/langsmith/templates/frontend/scaled-object.yaml (+56 -0)
📝 charts/langsmith/templates/host-backend/deployment.yaml (+1 -1)
📝 charts/langsmith/templates/host-backend/hpa.yaml (+7 -7)
charts/langsmith/templates/host-backend/scaled-object.yaml (+56 -0)
📝 charts/langsmith/templates/ingest-queue/deployment.yaml (+1 -1)
📝 charts/langsmith/templates/ingest-queue/hpa.yaml (+7 -10)
charts/langsmith/templates/ingest-queue/scaled-object.yaml (+70 -0)
📝 charts/langsmith/templates/listener/deployment.yaml (+1 -1)
📝 charts/langsmith/templates/listener/hpa.yaml (+7 -7)

...and 12 more files

📄 Description

  • Not enabled by default.
  • Gave users ample configuration
  • Added validation and useful error messages if still using old configuration
  • Also fix an issue where OTLP_ENDPOINT sticks around after disabling tracing, which ends up not disabling it.
  • Fixed some comments around blob storage affecting README docs

Also added logs shipping to beacon, opt-out


🔄 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/548 **Author:** [@romain-priour-lc](https://github.com/romain-priour-lc) **Created:** 1/6/2026 **Status:** ❌ Closed **Base:** `main` ← **Head:** `romain/enable-keda-langsmith` --- ### 📝 Commits (10+) - [`f22dc94`](https://github.com/langchain-ai/helm/commit/f22dc946cb96dfabd3875ac1285597c868e82e67) Support KEDA autoscaling in helm. Need to test - [`477fcf4`](https://github.com/langchain-ai/helm/commit/477fcf45fe0a5ccf9a7c448d1c6b46b9e46a956c) Load tested, scale up happened - [`72d0172`](https://github.com/langchain-ai/helm/commit/72d017229063632c64e60c2f8a62adf94dc94143) Update autoscaling config example - [`d92f92a`](https://github.com/langchain-ai/helm/commit/d92f92a5e3f85f972292ae9dcb4003529869ddd7) Remove deprecated initialOrgAdminEmail from the basicauth section - [`147b0e9`](https://github.com/langchain-ai/helm/commit/147b0e9a9b86e6d63c770dbde1035cbe72ebe70b) scale down non-ingest queue replicas to 1 by default - [`1c0cbcb`](https://github.com/langchain-ai/helm/commit/1c0cbcb672067826caa6c8ed18bbf6ed6caa8b63) Added KEDA to all services for symmetry + updated docs and examples. Need to test - [`75d15cb`](https://github.com/langchain-ai/helm/commit/75d15cb1c04c9b97bae2290492b20dd80cec5866) Tested end to end - [`ec00cb8`](https://github.com/langchain-ai/helm/commit/ec00cb8fda581dd4c00a559eaab6b45e6d75a9dc) Update README - [`a3bd667`](https://github.com/langchain-ai/helm/commit/a3bd6679c0bc42231481dac3a572a50a0e010b92) Add logs to beacon - [`dbc3d67`](https://github.com/langchain-ai/helm/commit/dbc3d6736c7a38f12e9bc4d222319a2afe69b613) update README ### 📊 Changes **32 files changed** (+1279 additions, -244 deletions) <details> <summary>View changed files</summary> 📝 `charts/langsmith/README.md` (+205 -58) 📝 `charts/langsmith/examples/autoscaling_config.yaml` (+168 -21) 📝 `charts/langsmith/templates/ace-backend/deployment.yaml` (+1 -1) 📝 `charts/langsmith/templates/ace-backend/hpa.yaml` (+9 -9) ➕ `charts/langsmith/templates/ace-backend/scaled-object.yaml` (+56 -0) 📝 `charts/langsmith/templates/backend/deployment.yaml` (+1 -1) 📝 `charts/langsmith/templates/backend/hpa.yaml` (+9 -9) ➕ `charts/langsmith/templates/backend/scaled-object.yaml` (+56 -0) 📝 `charts/langsmith/templates/config-map.yaml` (+2 -0) 📝 `charts/langsmith/templates/frontend/deployment.yaml` (+1 -1) 📝 `charts/langsmith/templates/frontend/hpa.yaml` (+9 -9) ➕ `charts/langsmith/templates/frontend/scaled-object.yaml` (+56 -0) 📝 `charts/langsmith/templates/host-backend/deployment.yaml` (+1 -1) 📝 `charts/langsmith/templates/host-backend/hpa.yaml` (+7 -7) ➕ `charts/langsmith/templates/host-backend/scaled-object.yaml` (+56 -0) 📝 `charts/langsmith/templates/ingest-queue/deployment.yaml` (+1 -1) 📝 `charts/langsmith/templates/ingest-queue/hpa.yaml` (+7 -10) ➕ `charts/langsmith/templates/ingest-queue/scaled-object.yaml` (+70 -0) 📝 `charts/langsmith/templates/listener/deployment.yaml` (+1 -1) 📝 `charts/langsmith/templates/listener/hpa.yaml` (+7 -7) _...and 12 more files_ </details> ### 📄 Description - Not enabled by default. - Gave users ample configuration - Added validation and useful error messages if still using old configuration - Also fix an issue where OTLP_ENDPOINT sticks around after disabling tracing, which ends up not disabling it. - Fixed some comments around blob storage affecting README docs Also added logs shipping to beacon, opt-out --- <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:38 -05:00
yindo closed this issue 2026-02-16 02:16:38 -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#552