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

Setting cgroup resources #372

Closed
wants to merge 1 commit into from
Closed

Conversation

rajasec
Copy link
Contributor

@rajasec rajasec commented Oct 27, 2015

Using the container interface set to set the cgroup resources when container is running
Usage:
runc set
This will apply the new values to running container as per the runtime.json

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

Usage: "set cgroup resources",
Flags: []cli.Flag{
cli.StringFlag{
Name: "config-file, c",
Copy link
Contributor

Choose a reason for hiding this comment

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

This shouldn't require the path to config.json then. We probably want the bundlePath from the other PR.

@rajasec
Copy link
Contributor Author

rajasec commented Nov 3, 2015

@mrunalp
If you are mentioning about PR #373 , I can wait until it is merged, post that I can rebase and commit.

@rajasec
Copy link
Contributor Author

rajasec commented Nov 18, 2015

@mrunalp
I've re-based ( by having changes PR #373) and added the bundle path changes in this PR.

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

Adding bundle path

Signed-off-by: Rajasekaran <rajasec79@gmail.com>

Updating README file

Signed-off-by: rajasec <rajasec79@gmail.com>
@rajasec
Copy link
Contributor Author

rajasec commented Nov 25, 2015

@mrunalp
Updated README file

@hqhq
Copy link
Contributor

hqhq commented Dec 8, 2015

@rajasec Actually container.Set() does more than just set cgroup resources, it replaces the whole container config. So I think it might dangerous to supply such a commend to users, let them replace the whole config.

Maybe we should start discussing https://github.com/opencontainers/specs/blob/master/ROADMAP.md#live-container-updates , have a conclusion about what can be updated, and we only take care of these fields from config.

@mrunalp
Copy link
Contributor

mrunalp commented Jan 26, 2016

I think we should revisit this later once we have a plan in the spec. Closing for now.

@mrunalp mrunalp closed this Jan 26, 2016
stefanberger pushed a commit to stefanberger/runc that referenced this pull request Sep 8, 2017
Fix the build by getting rid of go get for vet
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.

4 participants