langchain-ai/helm
1
0
Fork 0
mirror of https://github.com/langchain-ai/helm.git synced 2026-07-23 15:16:29 -04:00
Code Issues 24 Packages Projects Releases 10 Wiki Activity

[PR #671] [MERGED] fix: self-hosted agent-builder routing #668

New Issue
Closed
opened 2026-06-05 19:17:21 -04:00 by yindo · 0 comments
yindo commented 2026-06-05 19:17:21 -04:00
Owner
Copy Link
Copy Source

📋 Pull Request Information

Original PR: https://github.com/langchain-ai/helm/pull/671
Author: @AaryanPotdar
Created: 4/9/2026
Status: ✅ Merged
Merged: 4/9/2026
Merged by: @AaryanPotdar

Base: main ← Head: aaryan/fix-self-hosted-agent-builder-routing

📝 Commits (1)

  • afbc05e fix self-hosted agent-builder routing

📊 Changes

1 file changed (+18 additions, -0 deletions)

View changed files

📝 charts/langsmith/templates/frontend/config-map.yaml (+18 -0)

📄 Description

Description

  • Self-hosted LangSmith deployments were missing an explicit frontend nginx route for /api/v1/agent-builder/*.
  • Because of that, Fleet access-control requests such as /api/v1/agent-builder/integrations fell through to the generic /api/v1 backend proxy and were sent to the Python backend instead of the Go platform backend. In practice, that caused self-hosted Fleet access controls to fail with 404s.
  • This change adds explicit frontend nginx routing for /api/v1/agent-builder/ in both the basePath and non-basePath chart templates so those requests are rewritten to /v1/agent-builder/* and forwarded to the platform backend.

Test Plan

  • none

🔄 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/671 **Author:** [@AaryanPotdar](https://github.com/AaryanPotdar) **Created:** 4/9/2026 **Status:** ✅ Merged **Merged:** 4/9/2026 **Merged by:** [@AaryanPotdar](https://github.com/AaryanPotdar) **Base:** `main` ← **Head:** `aaryan/fix-self-hosted-agent-builder-routing` --- ### 📝 Commits (1) - [`afbc05e`](https://github.com/langchain-ai/helm/commit/afbc05e969beddb48ec8462104eaa9d7b66e800f) fix self-hosted agent-builder routing ### 📊 Changes **1 file changed** (+18 additions, -0 deletions) <details> <summary>View changed files</summary> 📝 `charts/langsmith/templates/frontend/config-map.yaml` (+18 -0) </details> ### 📄 Description ## Description - Self-hosted LangSmith deployments were missing an explicit frontend nginx route for `/api/v1/agent-builder/*`. - Because of that, Fleet access-control requests such as `/api/v1/agent-builder/integrations` fell through to the generic `/api/v1` backend proxy and were sent to the Python backend instead of the Go platform backend. In practice, that caused self-hosted Fleet access controls to fail with 404s. - This change adds explicit frontend nginx routing for `/api/v1/agent-builder/` in both the basePath and non-basePath chart templates so those requests are rewritten to `/v1/agent-builder/*` and forwarded to the platform backend. ## Test Plan - [x] none --- <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-06-05 19:17:21 -04:00
yindo closed this issue 2026-06-05 19:17:21 -04:00
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels
pull-request
Mirrored from GitHub Pull Request
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: langchain-ai/helm#668
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?