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

virtcontainers: inherit parent's CPU constraint #1530

Conversation

devimc
Copy link

@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 [email protected]

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 <[email protected]>
@devimc devimc force-pushed the topic/virtcontainers/inheritParentCpuCgroup branch from 773318e to 59e3956 Compare April 12, 2019 13:59
@devimc
Copy link
Author

devimc commented Apr 12, 2019

/test

@jcvenegas
Copy link
Member

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

@devimc
Copy link
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

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

@egernst Can you please help with this as well?

@gkhanna79
Copy link

@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