Skip to content

Updating blkio default value in man page#1142

Closed
rajasec wants to merge 1 commit intoopencontainers:mainfrom
rajasec:update-stats
Closed

Updating blkio default value in man page#1142
rajasec wants to merge 1 commit intoopencontainers:mainfrom
rajasec:update-stats

Conversation

@rajasec
Copy link
Copy Markdown
Contributor

@rajasec rajasec commented Oct 23, 2016

0 is invalid value for blkio value and will be rejected by kernel. Internally in runc implementation also we did not explicitly set 0 value. So it will take from parent cgroup.
updated the man page accordingly.

Signed-off-by: rajasec rajasec79@gmail.com

Signed-off-by: rajasec <rajasec79@gmail.com>
Comment thread man/runc-update.8.md
# OPTIONS
--resources value, -r value path to the file containing the resources to update or '-' to read from the standard input
--blkio-weight value Specifies per cgroup weight, range is from 10 to 1000 (default: 0)
--blkio-weight value Specifies per cgroup weight, range is from 10 to 1000 (0 to use system default)
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

This should match the output of runc --help (so you have to change that as well). Also I'd make the default text match for all of the options (the line below has a different format).

@hqhq
Copy link
Copy Markdown
Contributor

hqhq commented Mar 7, 2017

@rajasec Can you update and rebase?

@AkihiroSuda
Copy link
Copy Markdown
Member

needs rebase

@kolyshkin
Copy link
Copy Markdown
Contributor

This text was changed in PR #3032 (commit dfc0f06) to remove these details (now it only says "Set a new io weight"). I believe if someone is using runc update to change io weight for a running container, they know what they do.

@kolyshkin kolyshkin closed this Jun 15, 2024
@rajasec rajasec deleted the update-stats branch August 13, 2024 07:20
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.

6 participants