From b910cb47efbc048426d1e23c19410d5ccf37eab9 Mon Sep 17 00:00:00 2001 From: Miguel Serrano Date: Fri, 29 Aug 2025 11:26:39 +0200 Subject: [PATCH] Merge pull request #28138 from overleaf/msm-remove-volumes-dockerode [clsi] Remove `Volumes` from container options GitOrigin-RevId: 53a60f69e9689ee777d9b300127885de7b88c1fb --- services/clsi/app/js/DockerRunner.js | 4 ---- 1 file changed, 4 deletions(-) diff --git a/services/clsi/app/js/DockerRunner.js b/services/clsi/app/js/DockerRunner.js index def02eaf5b..039cb5acd6 100644 --- a/services/clsi/app/js/DockerRunner.js +++ b/services/clsi/app/js/DockerRunner.js @@ -213,11 +213,8 @@ const DockerRunner = { ) { const timeoutInSeconds = timeout / 1000 - const dockerVolumes = {} for (const hostVol in volumes) { const dockerVol = volumes[hostVol] - dockerVolumes[dockerVol] = {} - if (volumes[hostVol].slice(-3).indexOf(':r') === -1) { volumes[hostVol] = `${dockerVol}:rw` } @@ -238,7 +235,6 @@ const DockerRunner = { const options = { Cmd: command, Image: image, - Volumes: dockerVolumes, WorkingDir: '/compile', NetworkDisabled: true, Memory: 1024 * 1024 * 1024 * 1024, // 1 Gb