Skip to content
This repository was archived by the owner on Jul 18, 2025. It is now read-only.

Comments

Adding support for real-time cgroup values#262

Closed
erikstmartin wants to merge 1 commit intodocker-archive-public:masterfrom
erikstmartin:realtime-threads
Closed

Adding support for real-time cgroup values#262
erikstmartin wants to merge 1 commit intodocker-archive-public:masterfrom
erikstmartin:realtime-threads

Conversation

@erikstmartin
Copy link

This is required for an incoming PR I am submitting to docker/docker, so that users may supply --cpu-rt-period and --cpu-rt-runtime values so processes inside a container can set their realtime priority when the kernel option CONFIG_RT_GROUP_SCHED is configured.

This is discussed a bit towards the end of this issue moby/moby#22380

CPUPeriod int64 `json:"CpuPeriod"` // CPU CFS (Completely Fair Scheduler) period
CPUQuota int64 `json:"CpuQuota"` // CPU CFS (Completely Fair Scheduler) quota
CPURtPeriod int64 `json:"CpuRtPeriod"` // CPU real-time period
CPURtRuntime int64 `json:"CpuRtRuntime"` // CPU real-time period
Copy link
Contributor

Choose a reason for hiding this comment

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

copy/paste error on your comment.

@calavera
Copy link
Contributor

calavera commented Jun 8, 2016

Thanks @erikstmartin. It's better if you open the PR in docker/docker at the same time. That way, maintainers can discuss about the feature in general while you work on it.

@erikstmartin
Copy link
Author

I haven't submitted the PR to docker/docker yet, still chasing down some failing integration tests, but for anyone who wants to . I can if there is anyone interested in discussing it in the meantime.

This is the current state of the patch moby/moby@master...erikstmartin:realtime-threads

@crosbymichael
Copy link
Contributor

Thanks for this PR.

We have moved the code from this repo back into the main docker/docker repository; if you still wish to make this change you will need to reopen this PR on the main repo.

You can find the code for the client and types here:

https://github.com/docker/docker/tree/master/api/types
https://github.com/docker/docker/tree/master/client

Sorry for the inconvenience but making this change helps people better contribute and not have the vendor mess going forward when changing the types or client for Docker.

Sorry about that and feel free to ask if you have any questions.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants