16313: Updates minimum requestd ram on docker containers from 4 to 16 MB.
authorLucas Di Pentima <lucas@di-pentima.com.ar>
Mon, 13 Apr 2020 18:34:12 +0000 (15:34 -0300)
committerPeter Amstutz <peter.amstutz@curii.com>
Mon, 13 Apr 2020 18:46:22 +0000 (14:46 -0400)
Arvados-DCO-1.1-Signed-off-by: Lucas Di Pentima <lucas@di-pentima.com.ar>

lib/crunchrun/crunchrun.go

index b0a4007f74f87fe12be18046583ba23cc29fa9ea..c8f171ca9b83f38d1e2870af16913a4175db6490 100644 (file)
@@ -1074,9 +1074,10 @@ func (runner *ContainerRunner) CreateContainer() error {
        runner.ContainerConfig.Volumes = runner.Volumes
 
        maxRAM := int64(runner.Container.RuntimeConstraints.RAM)
-       if maxRAM < 4*1024*1024 {
-               // Docker daemon won't let you set a limit less than 4 MiB
-               maxRAM = 4 * 1024 * 1024
+       minDockerRAM := int64(16)
+       if maxRAM < minDockerRAM*1024*1024 {
+               // Docker daemon won't let you set a limit less than ~10 MiB
+               maxRAM = minDockerRAM * 1024 * 1024
        }
        runner.HostConfig = dockercontainer.HostConfig{
                Binds: runner.Binds,