Skip to content

Commit ebf9169

Browse files
committed
refactor: Improve ulimit configuration formatting in ContainerManager
- Enhanced the readability of ulimit configurations by restructuring the code for `nproc` and `nofile` settings. - Maintained existing functionality while improving code clarity and maintainability.
1 parent 2d602d0 commit ebf9169

1 file changed

Lines changed: 10 additions & 2 deletions

File tree

src/services/container/manager.py

Lines changed: 10 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -254,8 +254,16 @@ def create_container(
254254
"read_only": False,
255255
"tmpfs": {"/tmp": "noexec,nosuid,size=100m"},
256256
"ulimits": [
257-
docker.types.Ulimit(name="nproc", soft=settings.max_processes, hard=settings.max_processes),
258-
docker.types.Ulimit(name="nofile", soft=settings.max_open_files, hard=settings.max_open_files),
257+
docker.types.Ulimit(
258+
name="nproc",
259+
soft=settings.max_processes,
260+
hard=settings.max_processes,
261+
),
262+
docker.types.Ulimit(
263+
name="nofile",
264+
soft=settings.max_open_files,
265+
hard=settings.max_open_files,
266+
),
259267
],
260268
# Note: /proc/kallsyms and /proc/modules masking requires MaskedPaths
261269
# which docker-py doesn't support. Bind mounts to /proc are blocked by runc.

0 commit comments

Comments
 (0)