Skip to content
This repository has been archived by the owner on Oct 9, 2024. It is now read-only.

Commit

Permalink
Do not allocate pseudo tty when running containers (#623)
Browse files Browse the repository at this point in the history
  • Loading branch information
ssbarnea authored Jan 24, 2024
1 parent 8a81b9d commit 651f53e
Showing 1 changed file with 3 additions and 1 deletion.
4 changes: 3 additions & 1 deletion src/services/executionEnvironment.ts
Original file line number Diff line number Diff line change
Expand Up @@ -432,7 +432,9 @@ export class ExecutionEnvironment {
this.cleanUpContainer(containerName);

try {
let command = `${this._container_engine} run --rm -it -d `;
// Do not add '-t' option when running the containers as this causes stderr noise, such:
// The input device is not a TTY. The --tty and--interactive flags might not work properly
let command = `${this._container_engine} run -i --rm -d `;
if (this.settingsVolumeMounts && this.settingsVolumeMounts.length > 0) {
command += this.settingsVolumeMounts.join(" ");
}
Expand Down

0 comments on commit 651f53e

Please sign in to comment.