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.

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

image: lower file limit for containerd #2815

Merged
merged 1 commit into from
Jan 11, 2024
Merged

Conversation

3u13r
Copy link
Member

@3u13r 3u13r commented Jan 11, 2024

Context

Until containerd/containerd#8924 lands in our
containerd version, we need to decrease the default ulimit that
is set in the upstream containerd service of infinity which newer
systemd versions resolve to 2**30 which is way too large and
results in various inspecific errors such as excessive resource (e.g., memory)
usage.

Proposed change(s)

  • set LimitNOFILE=524288 for containerd

debug image: https://github.com/edgelesssys/constellation/actions/runs/7482948530

Additional info

How to test:

  • create a Constellation with this image
  • execute ulimit -a and verify that the file limit is LimitNOFILE=524288

Checklist

  • Add labels (e.g., for changelog category)
  • Link to Milestone

Copy link

netlify bot commented Jan 11, 2024

Deploy Preview for constellation-docs canceled.

Name Link
🔨 Latest commit bfcfa27
🔍 Latest deploy log https://app.netlify.com/sites/constellation-docs/deploys/659f42715cd945000878a6da

@3u13r 3u13r added this to the v2.15.0 milestone Jan 11, 2024
@3u13r 3u13r added the no changelog Change won't be listed in release changelog label Jan 11, 2024
@3u13r 3u13r marked this pull request as ready for review January 11, 2024 11:47
@3u13r 3u13r merged commit 120ae9d into main Jan 11, 2024
32 checks passed
@3u13r 3u13r deleted the fix/image/containerd-file-limit branch January 11, 2024 11:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
no changelog Change won't be listed in release changelog
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants