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 of cgroups: refactor v2 kill path to use cgroups.kill interface file into release/1.3.x #14396

Merged
merged 1 commit into from
Aug 30, 2022

Conversation

hc-github-team-nomad-core
Copy link
Contributor

Backport

This PR is auto-generated from #14385 to be assessed for backporting due to the inclusion of the label backport/1.3.x.

WARNING automatic cherry-pick of commits failed. Commits will require human attention.

The below text is copied from the body of the original PR.


This PR refactors the cgroups v2 group kill code path to use the
cgroups.kill interface file for destroying the cgroup. Previously
we copied the freeze + sigkill + unfreeze pattern from the v1 code,
but v2 provides a more efficient and more race-free way to handle
this.

Closes #14371

Backport only to 1.3.x (when v2 support was added)

@hashicorp-cla
Copy link

hashicorp-cla commented Aug 30, 2022

CLA assistant check
All committers have signed the CLA.

This PR refactors the cgroups v2 group kill code path to use the
cgroups.kill interface file for destroying the cgroup. Previously
we copied the freeze + sigkill + unfreeze pattern from the v1 code,
but v2 provides a more efficient and more race-free way to handle
this.

Closes #14371
@shoenig shoenig force-pushed the backport/f-cg-use-kill/admittedly-fast-pup branch from d922e5d to a9447e2 Compare August 30, 2022 14:25
@shoenig shoenig merged commit aa4c5d5 into release/1.3.x Aug 30, 2022
@shoenig shoenig deleted the backport/f-cg-use-kill/admittedly-fast-pup branch August 30, 2022 14:39
@github-actions
Copy link

I'm going to lock this pull request because it has been closed for 120 days ⏳. This helps our maintainers find and focus on the active contributions.
If you have found a problem that seems related to this change, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Dec 29, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants