Skip to content
This repository was archived by the owner on May 12, 2021. It is now read-only.

virtcontainers: inherit parent's CPU constraint#1530

Merged
teawater merged 1 commit intokata-containers:masterfrom
devimc:topic/virtcontainers/inheritParentCpuCgroup
Apr 13, 2019
Merged

virtcontainers: inherit parent's CPU constraint#1530
teawater merged 1 commit intokata-containers:masterfrom
devimc:topic/virtcontainers/inheritParentCpuCgroup

Conversation

@devimc
Copy link
Copy Markdown

@devimc devimc commented Apr 12, 2019

Don't set a default CPU constraint if period and quota are not specified, that
way the container will inherit the CPU constraint from its parent. Container
creation won't fail if the parent CPU constraint is smaller than the default
number of vCPUs.

fixes #1521

Signed-off-by: Julio Montes julio.montes@intel.com

Don't set a default CPU constraint if period and quota are not specified, that
way the container will inherit the CPU constraint from its parent. Container
creation won't fail if the parent CPU constraint is smaller than the default
number of vCPUs.

fixes kata-containers#1521

Signed-off-by: Julio Montes <julio.montes@intel.com>
@devimc devimc force-pushed the topic/virtcontainers/inheritParentCpuCgroup branch from 773318e to 59e3956 Compare April 12, 2019 13:59
@devimc
Copy link
Copy Markdown
Author

devimc commented Apr 12, 2019

/test

@jcvenegas
Copy link
Copy Markdown
Member

@egernst @sboeuf @bergwolf please take a look ready to merge

@devimc
Copy link
Copy Markdown
Author

devimc commented Apr 12, 2019

cc @mocolic

@egernst egernst added this to the cgoup-sprint milestone Apr 12, 2019
@teawater teawater merged commit 6e59578 into kata-containers:master Apr 13, 2019
@egernst egernst mentioned this pull request Apr 16, 2019
@gkhanna79
Copy link
Copy Markdown

gkhanna79 commented Apr 19, 2019

@teawater Can you please also backport this to 1.6 stable since it is required for Service Fabric resource governance to work correctly with Kata (similar to requiring #1518)?

CC @mocolic

@gkhanna79
Copy link
Copy Markdown

@egernst Can you please help with this as well?

@gkhanna79
Copy link
Copy Markdown

@egernst @teawater Can you please help with port of this fix to stable 1.6? Service Fabric resource governance with Kata is broken without this change.

This was referenced May 1, 2019
@devimc devimc deleted the topic/virtcontainers/inheritParentCpuCgroup branch October 9, 2019 19:34
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Resource governance doesn't work with kata 1.6.0

5 participants