Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

ci/ciimage: Lower nofile ulimit on containers that test D #13710

Merged
merged 1 commit into from
Oct 2, 2024
Merged

ci/ciimage: Lower nofile ulimit on containers that test D #13710

merged 1 commit into from
Oct 2, 2024

Commits on Sep 24, 2024

  1. ci/ciimage: Lower nofile ulimit on containers that test D 

    The D stdlib function std.process.spawnProcessPosix suffers from a bug
that causes any program that calls it to OOM if the nofile ulimit is
large.

This doesn't affect CI currently but trying to run or build the
containers locally will most likely crash when calling dub as docker
defaults to something like 1073741816 for both the hard limit and soft
limit. Lowering the soft limit is enough to make dub behave.

Signed-off-by: Andrei Horodniceanu <[email protected]>
    @the-horo
    the-horo committed Sep 24, 2024
    Configuration menu
    Copy the full SHA
    5ed2751 View commit details
    Browse the repository at this point in the history