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

[Backport release-1.27] Fix hanging start-stop-daemon in footloose Alpine #3438

Merged
merged 1 commit into from
Aug 31, 2023

Conversation

k0s-bot
Copy link

@k0s-bot k0s-bot commented Aug 29, 2023

Automated backport to release-1.27, triggered by a label in #3436.

Some distributions package Docker in a way that allows for a vast
number of open file descriptors, often achieved through systemd's
`LimitNOFILE=infinity` configuration. This situation leads to start-
stop-daemon appearing to hang, consequently blocking the OpenRC
init sequence. However, the issue is not an actual hang; rather, it
attempts to close all open file descriptors, resulting in a loop with
approximately one billion iterations of the close() syscall. This
behavior effectively renders the footloose machine unable to boot.

Mitigate this by setting the open file descriptor limit to 1 Mi in
OpenRC's rc.conf.

Signed-off-by: Tom Wieczorek <[email protected]>
Link: moby/moby@8db6109
Link: https://github.com/OpenRC/openrc/blob/eb8831a1416ab2ee8123b3add78421c2aa316b39/src/start-stop-daemon/start-stop-daemon.c#L1104-L1105
(cherry picked from commit ed4e359)
@k0s-bot k0s-bot requested a review from a team as a code owner August 29, 2023 13:19
@twz123 twz123 added chore area/smoke-tests backport/release-1.26 PR that needs to be backported/cherrypicked to release-1.26 branch labels Aug 29, 2023
@twz123 twz123 enabled auto-merge August 29, 2023 14:13
@twz123 twz123 merged commit 36ca638 into release-1.27 Aug 31, 2023
@twz123 twz123 deleted the backport-3436-to-release-1.27 branch August 31, 2023 09:27
@k0s-bot
Copy link
Author

k0s-bot commented Aug 31, 2023

Successfully created backport PR for release-1.26:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/smoke-tests backport/release-1.26 PR that needs to be backported/cherrypicked to release-1.26 branch chore
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants