Skip to content

Conversation

dsseng
Copy link
Member

@dsseng dsseng commented Sep 9, 2025

Make sure processes are only launched into freshly-created cgroups
with all limits set when they are restarted.

This also allows processes to restart after cgroup being killed via the
cgroup.kill mechanism.

Fixes #11785

Signed-off-by: Dmitrii Sharshakov [email protected]

@github-project-automation github-project-automation bot moved this to To Do in Planning Sep 9, 2025
@talos-bot talos-bot moved this from To Do to In Review in Planning Sep 9, 2025
@github-project-automation github-project-automation bot moved this from In Review to Approved in Planning Sep 9, 2025
@dsseng dsseng requested a review from smira September 9, 2025 13:42
@dsseng dsseng force-pushed the fix-11785-recreate-cgroups branch from f4c34c7 to 3de69a5 Compare September 9, 2025 14:46
Copy link
Member

@smira smira left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

let's hide all details about cgroups v1/v2 into the new package, make unified simple interface as well

@dsseng dsseng force-pushed the fix-11785-recreate-cgroups branch from 3de69a5 to 562b05f Compare September 11, 2025 17:28
Make sure processes are only launched into freshly-created cgroups
with all limits set when they are restarted.

This also allows processes to restart after cgroup being killed via the
cgroup.kill mechanism.

Fixes siderolabs#11785

Signed-off-by: Dmitrii Sharshakov <[email protected]>
@dsseng dsseng force-pushed the fix-11785-recreate-cgroups branch from 562b05f to 69ab076 Compare September 12, 2025 09:33
@dsseng
Copy link
Member Author

dsseng commented Sep 12, 2025

/m

@talos-bot talos-bot merged commit 69ab076 into siderolabs:main Sep 12, 2025
61 checks passed
@github-project-automation github-project-automation bot moved this from Approved to Done in Planning Sep 12, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Proposed
Status: Done
Development

Successfully merging this pull request may close these issues.

Containerized services have no resource limits after they restart
5 participants