Skip to content

fix(ci): override ENTRYPOINT on toolbox smoke-test docker run#68

Merged
thinmintdev merged 1 commit into
mainfrom
fix/ci-docker-entrypoint-2026-05-21
May 21, 2026
Merged

fix(ci): override ENTRYPOINT on toolbox smoke-test docker run#68
thinmintdev merged 1 commit into
mainfrom
fix/ci-docker-entrypoint-2026-05-21

Conversation

@thinmintdev
Copy link
Copy Markdown
Contributor

Summary

The hal0-toolbox-vulkan image's ENTRYPOINT is llama-server itself (packaging/toolbox/vulkan.Dockerfile:154). The Integration (tier β) smoke-test step was running:

docker run --rm hal0-toolbox-vulkan:ci /bin/bash -c '...'

…which passes /bin/bash as the first arg to llama-server, producing:

error: invalid argument: /bin/bash

every single run. This has been red on main for a while and every Wave 1 polish PR inherits the failure.

Fix

--entrypoint /bin/bash override on the docker run line.

Test plan

  • CI on this branch runs Integration (tier β) green
  • Slot integration (Vulkan-CPU baseline) step passes

🤖 Generated with Claude Code

The hal0-toolbox-vulkan image's ENTRYPOINT is llama-server itself
(packaging/toolbox/vulkan.Dockerfile:154). The smoke-test job was
running `docker run image /bin/bash -c '...'`, which passes /bin/bash
as the first arg to llama-server, producing:

    error: invalid argument: /bin/bash

Override the entrypoint to /bin/bash for the proof-of-life check.

Closes the slot-integration CI failure that was inherited by every
PR off main (#64, #67).

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@thinmintdev thinmintdev force-pushed the main branch 2 times, most recently from 0c4ce34 to 5458a0d Compare May 21, 2026 04:16
@thinmintdev thinmintdev merged commit 83041ee into main May 21, 2026
2 of 6 checks passed
@thinmintdev thinmintdev deleted the fix/ci-docker-entrypoint-2026-05-21 branch May 21, 2026 04:21
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