From a95151a3a334c544eb993e91d1d90a9ac2bb6b1d Mon Sep 17 00:00:00 2001 From: Harald Musum Date: Mon, 15 Jul 2019 14:58:54 +0200 Subject: Revert "Remove VESPA_TOTAL_MEMORY_MB" --- .../vespa/hosted/node/admin/docker/DockerOperationsImpl.java | 9 +++++++++ 1 file changed, 9 insertions(+) (limited to 'node-admin') diff --git a/node-admin/src/main/java/com/yahoo/vespa/hosted/node/admin/docker/DockerOperationsImpl.java b/node-admin/src/main/java/com/yahoo/vespa/hosted/node/admin/docker/DockerOperationsImpl.java index d7e0605de10..aa537d4f69a 100644 --- a/node-admin/src/main/java/com/yahoo/vespa/hosted/node/admin/docker/DockerOperationsImpl.java +++ b/node-admin/src/main/java/com/yahoo/vespa/hosted/node/admin/docker/DockerOperationsImpl.java @@ -109,6 +109,15 @@ public class DockerOperationsImpl implements DockerOperations { addMounts(context, command); + // TODO: Enforce disk constraints + long minMainMemoryAvailableMb = (long) (context.node().memoryGb() * 1024); + if (minMainMemoryAvailableMb > 0) { + // VESPA_TOTAL_MEMORY_MB is used to make any jdisc container think the machine + // only has this much physical memory (overrides total memory reported by `free -m`). + // TODO: Remove after all tenants are running > 7.67 + command.withEnvironment("VESPA_TOTAL_MEMORY_MB", Long.toString(minMainMemoryAvailableMb)); + } + logger.info("Creating new container with args: " + command); command.create(); } -- cgit v1.2.3