Skip to content

Optionally start/stop the CI k8s sandbox next to the sandbox label - #36

Merged
asheth-eightfold merged 1 commit into
masterfrom
asheth/sandbox-k8s-trigger
Oct 22, 2026
Merged

asheth-eightfold merged 1 commit into
masterfrom
asheth/sandbox-k8s-trigger

Conversation

@asheth-eightfold

Copy link
Copy Markdown
Contributor

Summary

Optionally start and stop the CI k8s sandbox next to the plain sandbox label, the same way CI_K8S_TRIGGER does for needs_ci (#35).

With SANDBOX_K8S_TRIGGER=true (passed in by the calling workflow):

  • needs_sandbox / e2e_needs_sandbox, when this action adds sandbox: it also dispatches sandbox.yml with action=start.
  • stop_sandbox, when it removes sandbox: it also dispatches sandbox.yml with action=stop.

A label change made with GITHUB_TOKEN starts no workflow, so the action dispatches it directly. Regional and alternate-Python sandbox labels are unchanged and never dispatch.

Behaviour change: none unless the caller sets the variable. A failed dispatch never fails the action.

Caller side: EightfoldAI/vscode#122623 (ENG-221034).

Test

entrypoint.sh run locally with curl stubbed:

comment sandbox already on? trigger calls made
needs_sandbox no true adds sandbox, then dispatches start
needs_sandbox no false adds sandbox only (as today)
needs_sandbox:eu no true adds sandbox :eu: only
needs_sandbox yes true none
stop_sandbox yes true removes sandbox, then dispatches stop
stop_sandbox yes false removes sandbox only (as today)

🤖 Generated with Claude Code

https://claude.ai/code/session_01A2bggq7D1jK3hGMp87ohdc

With SANDBOX_K8S_TRIGGER=true (passed by the calling workflow), adding the plain
`sandbox` label on needs_sandbox also dispatches sandbox.yml (start), and removing it on
stop_sandbox dispatches it with stop: a GITHUB_TOKEN label change starts no workflow.

Co-Authored-By: Claude Opus 5.5 <[email protected]>
Claude-Session: https://claude.ai/code/session_01A2bggq7D1jK3hGMp87ohdc
@github-actions

Copy link
Copy Markdown

Dependency Review

✅ No vulnerabilities or license issues found.

Scanned Files

None

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.

1 participant