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

Adding CPU core utilization to stats module #4

Conversation

shivramsrivastava
Copy link
Contributor

Code Changes adding CPU core utilization from heapster.
Adding a new branch for this experimental changes.

@coveralls
Copy link

Coverage Status

Coverage increased (+0.09%) to 28.193% when pulling e0335b8 on shivramsrivastava:per_core_cpu into 1865629 on camsas:master.

@dhilipkumars
Copy link
Contributor

@shivramsrivastava Please resolve conflicts
/assign

@k8s-ci-robot
Copy link
Contributor

Thanks for your pull request. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

📝 Please follow instructions at https://git.k8s.io/community/CLA.md#the-contributor-license-agreement to sign the CLA.

It may take a couple minutes for the CLA signature to be fully registered; after that, please reply here with a new comment and we'll verify. Thanks.


Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@k8s-ci-robot k8s-ci-robot added the cncf-cla: no Indicates the PR's author has not signed the CNCF CLA. label Mar 21, 2018
@shivramsrivastava
Copy link
Contributor Author

I will close this PR for now.
This was a feature which was not properly tested.
The per-core utilization will make more sense if we have the capability to run pods on a specific core,
which is not the case currently with kubernetes.
This will be addressed later if required.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
cncf-cla: no Indicates the PR's author has not signed the CNCF CLA.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants