Skip to content

fix(ci): keep snap canary sandbox name within limit - #3751

Merged
drew merged 1 commit into
mainfrom
codex/fix-release-canary-snap-name
Sep 27, 2026
Merged

drew merged 1 commit into
mainfrom
codex/fix-release-canary-snap-name

Conversation

@drew

@drew drew commented Sep 27, 2026

Copy link
Copy Markdown
Collaborator

Summary

Fix the Release Canary failure in the Ubuntu Snap with system Docker lane. The old sandbox name exceeded the gateway's 19-character limit, so sandbox creation returned InvalidArgument before exercising the runtime.

Related Issue

No issue required: this is an obvious localized CI workflow bug. The failure is visible in Release Canary run 36290090668.

Changes

  • Use rc-${GITHUB_RUN_ID} for the Snap lane's sandbox name, consistent with the other canary lanes and within the gateway limit.

Testing

  • mise run pre-commit passes.
  • Release Canary branch run 36340894904 passes all six jobs, including Snap sandbox create, exec, and delete.
  • mise run ci passes locally. It fails in unrelated Go gateway tests because this workstation's system gateway configuration adds an unexpected default gateway to their fixtures.
  • Unit tests added/updated (not applicable to this one-line workflow fix).
  • E2E tests added/updated (covered by the branch canary).

Checklist

  • Follows Conventional Commits.
  • Commit is signed off (DCO).
  • Architecture docs updated (not applicable).

Signed-off-by: Drew Newberry <anewberry@nvidia.com>
@drew
drew added this pull request to the merge queue Sep 27, 2026
Merged via the queue into main with commit c9da461 Sep 27, 2026
79 checks passed
@drew
drew deleted the codex/fix-release-canary-snap-name branch September 27, 2026 19:29
@drew drew added this to the OpenShell 0.1.2 milestone Sep 28, 2026
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.

2 participants