fix(scripts): pin Docker preflight platform

vincentkoc · vincentkoc · commit 6aa83374d90d · 2026-06-16T08:06:29.000+02:00
diff --git a/scripts/test-docker-all.mjs b/scripts/test-docker-all.mjs
@@ -542,6 +542,15 @@ export function dockerPreflightContainerNames(raw) {
     );
 }
 
+export function resolveDockerPreflightPlatform(arch = process.arch) {
+  return arch === "arm64" ? "linux/arm64" : "linux/amd64";
+}
+
+export function dockerPreflightSmokeCommand(arch = process.arch) {
+  const platform = resolveDockerPreflightPlatform(arch);
+  return `docker run --rm --platform ${shellQuote(platform)} alpine:3.20 true`;
+}
+
 export function runShellCommand({ command, env, label, logFile, timeoutMs, noOutputTimeoutMs }) {
   return new Promise((resolve) => {
     const pipeOutput = Boolean(logFile || noOutputTimeoutMs > 0);
@@ -873,15 +882,15 @@ async function runDockerPreflight(baseEnv, options) {
 
   const startedAt = Date.now();
   const run = await runShellCommand({
-    command: "docker run --rm alpine:3.20 true",
+    command: dockerPreflightSmokeCommand(),
     env: baseEnv,
     label: "docker-run-smoke",
     timeoutMs: options.runTimeoutMs,
   });
   const elapsedSeconds = Math.round((Date.now() - startedAt) / 1000);
   if (run.status !== 0) {
     throw new Error(
-      `Docker preflight failed: docker run alpine:3.20 true status=${run.status} elapsed=${elapsedSeconds}s`,
+      `Docker preflight failed: ${dockerPreflightSmokeCommand()} status=${run.status} elapsed=${elapsedSeconds}s`,
     );
   }
   console.log(`==> Docker preflight run: ${elapsedSeconds}s`);
diff --git a/test/scripts/docker-all-scheduler.test.ts b/test/scripts/docker-all-scheduler.test.ts
@@ -11,8 +11,10 @@ import {
   canStartSchedulerLane,
   describeDockerSchedulerLimits,
   dockerPreflightContainerNames,
+  dockerPreflightSmokeCommand,
   LOG_TAIL_MAX_BYTES,
   parseDockerAllCliArgs,
+  resolveDockerPreflightPlatform,
   runShellCommand,
   SHELL_CAPTURE_MAX_CHARS,
   tailFile,
@@ -417,6 +419,17 @@ postgres Created
     ]);
   });
 
+  it("pins Docker preflight smoke to the native platform", () => {
+    expect(resolveDockerPreflightPlatform("x64")).toBe("linux/amd64");
+    expect(resolveDockerPreflightPlatform("arm64")).toBe("linux/arm64");
+    expect(dockerPreflightSmokeCommand("x64")).toBe(
+      "docker run --rm --platform 'linux/amd64' alpine:3.20 true",
+    );
+    expect(dockerPreflightSmokeCommand("arm64")).toBe(
+      "docker run --rm --platform 'linux/arm64' alpine:3.20 true",
+    );
+  });
+
   it("bounds captured preflight command output while keeping the newest tail", () => {
     const first = appendBoundedShellCapture("abc", "def", 8);
     expect(first).toEqual({ text: "abcdef", truncated: false });
