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

[0.39] Cgroup v2 fixes backport #2879

Merged
merged 5 commits into from
May 31, 2021

Conversation

kolyshkin
Copy link
Contributor

giuseppe added 5 commits May 25, 2021 14:09
read the memory stats from the specified cgroup instead of looking the
values up in the ancestor cgroups.

Signed-off-by: Giuseppe Scrivano <[email protected]>
(cherry picked from commit cf79551)
Signed-off-by: Kir Kolyshkin <[email protected]>
when the "max" value is found handle it as math.MaxUint64.

Signed-off-by: Giuseppe Scrivano <[email protected]>
(cherry picked from commit 7bd9ea7)
Signed-off-by: Kir Kolyshkin <[email protected]>
read quota and limit from cpu.max, and convert cpu.weight to the range
expected with cgroup v1.

Signed-off-by: Giuseppe Scrivano <[email protected]>
(cherry picked from commit 56ced55)
Signed-off-by: Kir Kolyshkin <[email protected]>
Signed-off-by: Giuseppe Scrivano <[email protected]>
(cherry picked from commit 5bffc55)
Signed-off-by: Kir Kolyshkin <[email protected]>
Signed-off-by: Giuseppe Scrivano <[email protected]>
(cherry picked from commit 70b339d)
Signed-off-by: Kir Kolyshkin <[email protected]>
@k8s-ci-robot
Copy link
Collaborator

Hi @kolyshkin. Thanks for your PR.

I'm waiting for a google member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.

Once the patch is verified, the new status will be reflected by the ok-to-test label.

I understand the commands that are listed here.

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@google-cla
Copy link

google-cla bot commented May 25, 2021

All (the pull request submitter and all commit authors) CLAs are signed, but one or more commits were authored or co-authored by someone other than the pull request submitter.

We need to confirm that all authors are ok with their commits being contributed to this project. Please have them confirm that by leaving a comment that contains only @googlebot I consent. in this pull request.

Note to project maintainer: There may be cases where the author cannot leave a comment, or the comment is not properly detected as consent. In those cases, you can manually confirm consent of the commit author(s), and set the cla label to yes (if enabled on your project).

ℹ️ Googlers: Go here for more info.

@google-cla google-cla bot added the cla: no label May 25, 2021
@kolyshkin kolyshkin changed the base branch from master to release-v0.39 May 25, 2021 21:14
@kolyshkin
Copy link
Contributor Author

@googlebot I consent.

@google-cla
Copy link

google-cla bot commented May 25, 2021

All (the pull request submitter and all commit authors) CLAs are signed, but one or more commits were authored or co-authored by someone other than the pull request submitter.

We need to confirm that all authors are ok with their commits being contributed to this project. Please have them confirm that by leaving a comment that contains only @googlebot I consent. in this pull request.

Note to project maintainer: There may be cases where the author cannot leave a comment, or the comment is not properly detected as consent. In those cases, you can manually confirm consent of the commit author(s), and set the cla label to yes (if enabled on your project).

ℹ️ Googlers: Go here for more info.

@Creatone
Copy link
Collaborator

/ok-to-test

@giuseppe
Copy link
Contributor

@googlebot I consent.

@google-cla google-cla bot added cla: yes and removed cla: no labels May 26, 2021
@giuseppe
Copy link
Contributor

all green!

@kolyshkin
Copy link
Contributor Author

@bobbypage PTAL

Copy link
Collaborator

@Creatone Creatone left a comment

Choose a reason for hiding this comment

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

LGTM

@Creatone Creatone merged commit 619b69f into google:release-v0.39 May 31, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Cut the 0.40 (or 0.39.2) release to have cgroup v2 fixes
4 participants