Skip to content

fix(helm): preflight Agent Sandbox APIs - #2867

Merged
elezar merged 4 commits into
mainfrom
codex/helm-agent-sandbox-preflight
Aug 21, 2026
Merged

fix(helm): preflight Agent Sandbox APIs#2867
elezar merged 4 commits into
mainfrom
codex/helm-agent-sandbox-preflight

Conversation

@elezar

@elezarelezar commented Aug 21, 2026

Copy link
Copy Markdown
Member

Summary

Fail Helm installation early when a cluster lacks a supported Agent Sandbox API, and make the release canary install the same controller dependency used by Kubernetes E2E. This addresses the failed Kubernetes Helm (kind) canary job.

Related Issue

No issue required: localized Helm and release-canary regression fix. Related context: #2013 and #2009.

Changes

  • Add an enabled-by-default Helm API preflight with an offline-rendering override.
  • Provision Agent Sandbox in the kind release canary before chart installation.
  • Share the controller installation and readiness sequence between Kubernetes E2E and canary.
  • Document the cluster-scoped prerequisite and local canary reproduction.

Testing

  • mise run pre-commit passes
  • Helm unit tests, lint, and generated-docs check
  • Shell syntax, workflow YAML, and mocked Agent Sandbox helper command-sequence checks
  • Full Kubernetes E2E run

Checklist

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

Signed-off-by: Evan Lezar <elezar@nvidia.com>
Signed-off-by: Evan Lezar <elezar@nvidia.com>
@github-actions

Copy link
Copy Markdown

@elezar
elezar enabled auto-merge August 21, 2026 10:16
@elezarelezar added the test:e2e Requires end-to-end coverage label Aug 21, 2026
@github-actions

Copy link
Copy Markdown

Label test:e2e applied for 22e9812. Open the existing run and click Re-run all jobs to execute with the label set. The run will execute the standard E2E suite after building the required gateway and supervisor images once. The matching required CI gate status on this PR will flip green automatically once the run finishes.

@elezar

Copy link
Copy Markdown
MemberAuthor

/ok-to-test 22e9812

Signed-off-by: Evan Lezar <elezar@nvidia.com>
Signed-off-by: Evan Lezar <elezar@nvidia.com>
@elezar
elezar added this pull request to the merge queueAug 21, 2026
Merged via the queue into main with commit 3be2cd8Aug 21, 2026
58 of 60 checks passed
@elezar
elezar deleted the codex/helm-agent-sandbox-preflight branch August 21, 2026 14:45
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

test:e2eRequires end-to-end coverage

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

@elezar@drew