search

clearcontainers / packaging

Packaging data for Clear Containers
11 stars 11 forks source link

kernel: enable CPU bandwidth provisioning #247

Closed devimc closed 6 years ago

devimc commented 6 years ago

this patch is to enable CPU period and quota in cgroups

Fixes #251 Fixes https://github.com/clearcontainers/agent/issues/195

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

devimc commented 6 years ago

/cc @jcvenegas @jodh-intel

jcvenegas commented 6 years ago

@devimc this is for cgruops support inside the VM ? Is this a mandatory kernel configuration?, if it is the case you may want to document somewhere that this will be used by the agent(?), if case is not mandatory document the expected behavior of not being enabled.

jcvenegas commented 6 years ago

Travis is not happy, it wants a issue and a Fixes

jodh-intel commented 6 years ago

Once you've added a Fixes #XXX,

lgtm

/cc @grahamwhaley.

devimc commented 6 years ago

@jcvenegas yes, we need it to apply cgroups inside the POD

@jodh-intel @jcvenegas @grahamwhaley changes applied, thanks

jodh-intel commented 6 years ago

lgtm

devimc commented 6 years ago

@jcvenegas done

jcvenegas commented 6 years ago

new kernel with PR changes: https://github.com/clearcontainers/linux/releases/download/v4.9.60-84.container/v4.9.60-84.container-binaries.tar.gz