Skip to content

unix: call ulimit -n to avoid overheads #468

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

Merged
merged 1 commit into from
Jan 8, 2025
Merged

unix: call ulimit -n to avoid overheads #468

merged 1 commit into from
Jan 8, 2025

Conversation

indygreg
Copy link
Collaborator

@indygreg indygreg commented Jan 2, 2025

See inline comment in base.Dockerfile for why this hack is necessary.

When upgrading my machine recently, the rlim_max value increased to ~1B, making apt-get invocations take multiple minutes to complete. https://bugs.launchpad.net/ubuntu/+source/apt/+bug/1332440 seems to describe this issue.

The cross building images have a sufficiently new apt that doesn't seem impacted.

See inline comment in `base.Dockerfile` for why this hack is necessary.

When upgrading my machine recently, the `rlim_max` value increased to
~1B, making `apt-get` invocations take multiple minutes to complete.
https://bugs.launchpad.net/ubuntu/+source/apt/+bug/1332440 seems to
describe this issue.

The cross building images have a sufficiently new apt that doesn't seem
impacted.
@zanieb
Copy link
Member

zanieb commented Jan 3, 2025

Oh, I wonder if this solves #238

@zanieb zanieb added the internal A refactor or improvement that is not user-facing label Jan 3, 2025
@indygreg
Copy link
Collaborator Author

indygreg commented Jan 3, 2025

I suspect this was the root cause of #238.

@zanieb zanieb merged commit 61e7c99 into main Jan 8, 2025
300 checks passed
@zanieb zanieb deleted the gps/apt-ulimit branch January 8, 2025 05:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
internal A refactor or improvement that is not user-facing
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants