Skip to content

fix(docker): use supervisor image path directly#1259

Merged
drew merged 1 commit intomainfrom
fix-docker-supervisor-binary/an
May 8, 2026
Merged

fix(docker): use supervisor image path directly#1259
drew merged 1 commit intomainfrom
fix-docker-supervisor-binary/an

Conversation

@drew
Copy link
Copy Markdown
Collaborator

@drew drew commented May 7, 2026

Summary

Use the supervisor image binary path at /openshell-sandbox for Docker supervisor extraction and validate that path in Docker image CI. Gateway Docker image builds and Debian release gateway binaries bake the matching supervisor image tag into the binary, so the Docker driver does not depend on :latest by default.

Related Issue

None.

Changes

  • Use the shared /openshell-sandbox path constant when creating the Docker extractor container.
  • Run Docker image smoke checks for pushed images as well as local builds, with the supervisor check explicitly invoking /openshell-sandbox.
  • Bake the supervisor image tag into Docker image gateway builds and Debian release gateway binaries.
  • Update runtime architecture docs and troubleshooting notes for supervisor image extraction.

Testing

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

Checklist

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

@drew drew requested review from a team, derekwaynecarr, maxamillion and mrunalp as code owners May 7, 2026 23:34
@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 7, 2026

@drew drew force-pushed the fix-docker-supervisor-binary/an branch from 3e819f2 to fd844d2 Compare May 7, 2026 23:42
@drew drew changed the title fix(packaging): include supervisor binary in deb package fix(docker): use supervisor image entrypoint path May 7, 2026
@drew drew force-pushed the fix-docker-supervisor-binary/an branch from fd844d2 to 53996f7 Compare May 7, 2026 23:53
Signed-off-by: Drew Newberry <anewberry@nvidia.com>
@drew drew force-pushed the fix-docker-supervisor-binary/an branch from 53996f7 to c4afa87 Compare May 8, 2026 00:05
@drew drew changed the title fix(docker): use supervisor image entrypoint path fix(docker): use supervisor image path directly May 8, 2026
@drew drew merged commit 62619ee into main May 8, 2026
23 checks passed
@drew drew deleted the fix-docker-supervisor-binary/an branch May 8, 2026 00:39
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