Skip to content

[improvement] Increase RLIMIT_MEMLOCK in containers#860

Merged
eljohnson92 merged 2 commits into
linode:mainfrom
dthorsen:increase_locked_memory
Sep 3, 2025
Merged

[improvement] Increase RLIMIT_MEMLOCK in containers#860
eljohnson92 merged 2 commits into
linode:mainfrom
dthorsen:increase_locked_memory

Conversation

@dthorsen

@dthorsen dthorsen commented Sep 2, 2025

Copy link
Copy Markdown
Contributor

What this PR does / why we need it:
On kernel versions older than 5.16, the default RLIMIT_MEMLOCK in containers is quite low, at 64KB. Newer kernels set the limit to 8MB. This PR increases RLIMIT_MEMLOCK to 8MB in containerd, so that containerized workloads that require more locked memory work on nodes with older linux kernel versions.

TODOs:

  • squashed commits
  • includes documentation
  • adds unit tests
  • adds or updates e2e tests

@codecov

codecov Bot commented Sep 2, 2025

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 64.76%. Comparing base (758c935) to head (7904e5e).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #860   +/-   ##
=======================================
  Coverage   64.76%   64.76%           
=======================================
  Files          71       71           
  Lines        7871     7871           
=======================================
  Hits         5098     5098           
  Misses       2465     2465           
  Partials      308      308           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@dthorsen dthorsen marked this pull request as ready for review September 3, 2025 20:13
@eljohnson92 eljohnson92 merged commit 9013d31 into linode:main Sep 3, 2025
13 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants