Skip to content

Commit

Permalink
test: make sure the docker container stopped (ChainSafe#6642)
Browse files Browse the repository at this point in the history
Fix the docker container stopping
  • Loading branch information
nazarhussain authored Apr 5, 2024
1 parent 3226411 commit f2ec0d4
Showing 1 changed file with 3 additions and 2 deletions.
5 changes: 3 additions & 2 deletions packages/cli/test/utils/simulation/runner/DockerRunner.ts
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,6 @@ import {
SpawnChildProcessOptions,
execChildProcess,
spawnChildProcess,
stopChildProcess,
ChildProcessResolve,
} from "@lodestar/test-utils";
import {Job, JobOptions, RunnerEnv, RunnerType} from "../interfaces.js";
Expand Down Expand Up @@ -119,7 +118,9 @@ export class DockerRunner implements RunnerEnv<RunnerType.Docker> {
if (childProcess === undefined) {
return;
}
await stopChildProcess(childProcess);
// TODO: Debug why stopping the process was not killing the container
// await stopChildProcess(childProcess);
await execChildProcess(`docker stop ${jobOption.id} --time 2 || true`, {pipeStdioToParent: true});
},
};
}
Expand Down

0 comments on commit f2ec0d4

Please sign in to comment.