[ARVADOS] updated: 1.3.0-2237-g87c40af4f

Git user git at public.arvados.org
Mon Apr 13 18:57:37 UTC 2020


Summary of changes:
 lib/crunchrun/crunchrun.go | 7 ++++---
 1 file changed, 4 insertions(+), 3 deletions(-)

       via  87c40af4f0f1936ff992d080b23acf4fcb96c10f (commit)
      from  64c991ac1aa7524a3062cc77037e14c2b751a67d (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.


commit 87c40af4f0f1936ff992d080b23acf4fcb96c10f
Author: Lucas Di Pentima <lucas at di-pentima.com.ar>
Date:   Mon Apr 13 15:34:12 2020 -0300

    16313: Updates minimum requestd ram on docker containers from 4 to 16 MB.
    
    Arvados-DCO-1.1-Signed-off-by: Lucas Di Pentima <lucas at di-pentima.com.ar>

diff --git a/lib/crunchrun/crunchrun.go b/lib/crunchrun/crunchrun.go
index b0a4007f7..c8f171ca9 100644
--- a/lib/crunchrun/crunchrun.go
+++ b/lib/crunchrun/crunchrun.go
@@ -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,

-----------------------------------------------------------------------


hooks/post-receive
-- 




More information about the arvados-commits mailing list