fix(ci): require Docker only for routed workloads

This commit is contained in:
Vincent Koc
2026-08-03 08:55:18 -07:00
parent 7f75b3f494
commit fff20ca647
2 changed files with 5 additions and 2 deletions

View File

@@ -271,7 +271,9 @@ jobs:
set -euo pipefail
docker_required=false
if [ "${CRABBOX_JOB:-hydrate}" != "hydrate" ]; then
# crabbox_job is also the readiness marker. Only the routed Docker
# workload contract requires an engine; other marker names do not.
if [ "${CRABBOX_JOB:-hydrate}" = "hydrate-docker" ]; then
docker_required=true
fi

View File

@@ -1203,7 +1203,8 @@ describe("package acceptance workflow", () => {
CRABBOX_JOB: "${{ inputs.crabbox_job }}",
});
expect(ensureDocker.run).toContain("docker_required=false");
expect(ensureDocker.run).toContain('if [ "${CRABBOX_JOB:-hydrate}" != "hydrate" ]; then');
expect(ensureDocker.run).toContain('if [ "${CRABBOX_JOB:-hydrate}" = "hydrate-docker" ]; then');
expect(ensureDocker.run).toContain("other marker names do not");
expect(ensureDocker.run).toContain('if [ "$docker_required" = true ]; then');
expect(ensureDocker.run).toContain(
"Docker is unavailable for ${CRABBOX_JOB:-hydrate}; route this workload to a Docker-capable provider",