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

[PR #747] [MERGED] docs(langsmith): mirror readonly example with CI (agent apps) #746

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

📋 Pull Request Information

Original PR: https://github.com/langchain-ai/helm/pull/747
Author: @langchain-infra
Created: 6/1/2026
Status: ✅ Merged
Merged: 6/1/2026
Merged by: @langchain-infra

Base: main ← Head: docs/readonly-example-agent-apps

📝 Commits (1)

  • dec4a32 docs(langsmith): mirror readonly example with CI (agent apps)

📊 Changes

2 files changed (+360 additions, -149 deletions)

View changed files

📝 charts/langsmith/ci/readonly-config-values.yaml (+54 -0)
📝 charts/langsmith/examples/read_only_config.yaml (+306 -149)

📄 Description

Summary

Brings examples/read_only_config.yaml in line with ci/readonly-config-values.yaml for the agent applications, which are now enabled by default and would otherwise fail to start under a read-only root filesystem.

  • Add agent apps to the example: fleet, insights, polly, fleetToolServer, fleetTriggerServer — each with read-only securityContext/podSecurityContext and the required tmp (and postgres) volume mounts. Uses placeholder encryptionKey: "YOUR_ENCRYPTION_KEY" to match the example's YOUR_* style. CI-only bits (resources, hostname, ingress, replicas) are intentionally not added, to keep the example consistent with its existing style.
  • Remove obsolete listener.templates (inline db/redis) from the example — no longer needed.
  • Add operator.templates deployment block to the CI values so CI exercises it (the example already had it).

No chart version bump — consistent with how prior ci//examples/-only changes were merged in this repo (e.g. #664, #558). helm lint and helm template pass against both files.

🔄 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/747 **Author:** [@langchain-infra](https://github.com/langchain-infra) **Created:** 6/1/2026 **Status:** ✅ Merged **Merged:** 6/1/2026 **Merged by:** [@langchain-infra](https://github.com/langchain-infra) **Base:** `main` ← **Head:** `docs/readonly-example-agent-apps` --- ### 📝 Commits (1) - [`dec4a32`](https://github.com/langchain-ai/helm/commit/dec4a32c72d7abed8e4eff179a7fef6cbc9bbd3f) docs(langsmith): mirror readonly example with CI (agent apps) ### 📊 Changes **2 files changed** (+360 additions, -149 deletions) <details> <summary>View changed files</summary> 📝 `charts/langsmith/ci/readonly-config-values.yaml` (+54 -0) 📝 `charts/langsmith/examples/read_only_config.yaml` (+306 -149) </details> ### 📄 Description ## Summary Brings `examples/read_only_config.yaml` in line with `ci/readonly-config-values.yaml` for the agent applications, which are now enabled by default and would otherwise fail to start under a read-only root filesystem. - **Add agent apps to the example:** `fleet`, `insights`, `polly`, `fleetToolServer`, `fleetTriggerServer` — each with read-only `securityContext`/`podSecurityContext` and the required `tmp` (and `postgres`) volume mounts. Uses placeholder `encryptionKey: "YOUR_ENCRYPTION_KEY"` to match the example's `YOUR_*` style. CI-only bits (resources, hostname, ingress, replicas) are intentionally **not** added, to keep the example consistent with its existing style. - **Remove obsolete `listener.templates`** (inline db/redis) from the example — no longer needed. - **Add `operator.templates` deployment block to the CI values** so CI exercises it (the example already had it). No chart version bump — consistent with how prior `ci/`/`examples/`-only changes were merged in this repo (e.g. #664, #558). `helm lint` and `helm template` pass against both files. --- <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:37 -04:00
yindo closed this issue 2026-06-05 19:17:37 -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#746
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?