Skip to content

feat(helm): add sandbox UID and GID values - #3947

Open
ericcurtin wants to merge 1 commit into
NVIDIA:mainfrom
ericcurtin:feat/2697-helm-sandbox-uid-gid
Open

ericcurtin wants to merge 1 commit into
NVIDIA:mainfrom
ericcurtin:feat/2697-helm-sandbox-uid-gid

Conversation

@ericcurtin

Copy link
Copy Markdown
Contributor

Summary

Add server.sandboxUid and server.sandboxGid Helm values, rendered as sandbox_uid and sandbox_gid.

Related Issue

Closes #2697

Changes

  • Render both fields in the gateway config when set, validated as integers in 1..4294967294.
  • Add chart unit tests, docs rows, and regenerated chart README.

Testing

  • mise run pre-commit passes
  • Unit tests added/updated
  • E2E tests added/updated (if applicable)

mise run helm:test, helm:lint and helm:docs:check pass.

Checklist

  • Follows Conventional Commits
  • Commits are signed off (DCO)
  • Architecture docs updated (if applicable)

Closes NVIDIA#2697

Signed-off-by: Eric Curtin <eric.curtin@docker.com>
@copy-pr-bot

copy-pr-bot Bot commented Sep 30, 2026

Copy link
Copy Markdown

This pull request requires additional validation before any workflows can run on NVIDIA's runners.

Pull request vetters can view their responsibilities here.

Contributors can view more details about this message here.

@ericcurtin

Copy link
Copy Markdown
Contributor Author

If useful, please also try https://github.com/llmmanorg/llmman, which can launch agents in an OpenShell sandbox (--sandbox openshell).

@ericcurtin
ericcurtin requested a review from a team as a code owner September 30, 2026 09:13
@johntmyers johntmyers self-assigned this Oct 1, 2026

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

feat(helm): expose sandbox_uid / sandbox_gid as first-class Helm values

2 participants