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

Added conversion for cpu.weight v2 #2213

Merged

Conversation

Zyqsempai
Copy link
Contributor

Fix #2166

Added conversion for cpu.weight v2 from cpu.shares

Signed-off-by: Boris Popovschi [email protected]

@AkihiroSuda AkihiroSuda mentioned this pull request Jan 16, 2020
@Zyqsempai
Copy link
Contributor Author

@crosbymichael @cyphar @AkihiroSuda PTAL

@AkihiroSuda
Copy link
Member

@mrunalp @hqhq @dqminh PTAL (and #2212 #2192 as well as this one)

@mrunalp
Copy link
Contributor

mrunalp commented Jan 30, 2020

LGTM

Approved with PullApprove

@Zyqsempai Zyqsempai force-pushed the 2166-convert-cpu-weight-poperly branch from 2567240 to 1febadd Compare February 11, 2020 10:00
@Zyqsempai
Copy link
Contributor Author

@hqhq Done, PTAL

update.go Outdated Show resolved Hide resolved
@Zyqsempai Zyqsempai force-pushed the 2166-convert-cpu-weight-poperly branch 2 times, most recently from 5d3a4e0 to fdd4fd1 Compare February 11, 2020 12:17
@Zyqsempai
Copy link
Contributor Author

@hqhq PTAL on more time here as well.

Copy link
Member

@giuseppe giuseppe left a comment

Choose a reason for hiding this comment

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

LGTM

libcontainer/utils/utils.go Outdated Show resolved Hide resolved
@Zyqsempai Zyqsempai force-pushed the 2166-convert-cpu-weight-poperly branch from fdd4fd1 to 7c439cc Compare February 12, 2020 09:32
Signed-off-by: Boris Popovschi <[email protected]>
@hqhq
Copy link
Contributor

hqhq commented Feb 12, 2020

LGTM

Approved with PullApprove

@AkihiroSuda
Copy link
Member

merge?

@hqhq
Copy link
Contributor

hqhq commented Feb 17, 2020

Needs re-approve. @mrunalp

@mrunalp
Copy link
Contributor

mrunalp commented Feb 17, 2020

LGTM

Approved with PullApprove

@mrunalp mrunalp merged commit 81ef502 into opencontainers:master Feb 17, 2020
AkihiroSuda added a commit to AkihiroSuda/runc that referenced this pull request Mar 9, 2020
* TestConvertCPUSharesToCgroupV2Value(0) was returning 70369281052672, while the correct value is 0
* ConvertBlkIOToCgroupV2Value(0) was returning 32, while the correct value is 0
* ConvertBlkIOToCgroupV2Value(1000) was returning 4, while the correct value is 10000

Fix opencontainers#2244
Follow-up to opencontainers#2212 opencontainers#2213

Signed-off-by: Akihiro Suda <[email protected]>
AkihiroSuda added a commit to AkihiroSuda/runc that referenced this pull request Mar 12, 2020
* TestConvertCPUSharesToCgroupV2Value(0) was returning 70369281052672, while the correct value is 0
* ConvertBlkIOToCgroupV2Value(0) was returning 32, while the correct value is 0
* ConvertBlkIOToCgroupV2Value(1000) was returning 4, while the correct value is 10000

Fix opencontainers#2244
Follow-up to opencontainers#2212 opencontainers#2213

Signed-off-by: Akihiro Suda <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

cgroup2: cpu.shares is broken
6 participants