Skip to content

Commit 3aa1607

Browse files
authored
[iris] Remove 8 GB Docker memory cap on build containers (#4625)
uv sync on a large workspace OOMs against the fixed 8 GB _BUILD_MEMORY_LIMIT_BYTES ceiling. Confirmed via dmesg on the worker: CONSTRAINT_MEMCG killed uv at exactly 8 GB while the host had 1.4 TB free. Remove the artificial limit so build containers are bounded only by host memory.
1 parent 1ad2cfa commit 3aa1607

1 file changed

Lines changed: 3 additions & 2 deletions

File tree

lib/iris/src/iris/cluster/runtime/docker.py

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -348,7 +348,8 @@ def build(self, on_logs: Callable[[list[LogLine]], None] | None = None) -> list[
348348
setup_script = self._generate_setup_script()
349349
self._write_setup_script(setup_script)
350350

351-
# Build containers get max(8 GB, task request) memory
351+
# Build containers get max(32 GB, task request) memory — uv sync on a large
352+
# workspace OOMed at the old 8 GB ceiling on a host with 1.4 TB free.
352353
task_memory_bytes = self.config.resources.memory_bytes if self.config.resources else 0
353354
build_memory_bytes = (
354355
max(self._BUILD_MEMORY_LIMIT_BYTES, task_memory_bytes)
@@ -602,7 +603,7 @@ def cleanup(self) -> None:
602603
# Docker CLI helpers
603604
# -------------------------------------------------------------------------
604605

605-
_BUILD_MEMORY_LIMIT_BYTES = 8 * 1024**3
606+
_BUILD_MEMORY_LIMIT_BYTES = 32 * 1024**3
606607

607608
def _docker_create(
608609
self,

0 commit comments

Comments
 (0)