-
Notifications
You must be signed in to change notification settings - Fork 104
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
WIP: NOMERGE: DNM: consume u/s PR 126750 #2088
base: release-4.16
Are you sure you want to change the base?
WIP: NOMERGE: DNM: consume u/s PR 126750 #2088
Conversation
…ncorecache is enabled.
…o take CPUs by uncourecache grouping.
@ffromani: the contents of this pull request could not be automatically validated. The following commits could not be validated and must be approved by a top-level approver:
Comment |
/hold |
adapt to the older codebase Signed-off-by: Francesco Romani <fromani@redhat.com>
9270305
to
9abda8e
Compare
@ffromani: the contents of this pull request could not be automatically validated. The following commits could not be validated and must be approved by a top-level approver:
Comment |
/test k8s-e2e-aws-ovn-serial |
@ffromani: the contents of this pull request could not be automatically validated. The following commits could not be validated and must be approved by a top-level approver:
Comment |
In order to improve the observability of the resource management in kubelet, cpu allocation and NUMA alignment, we add more metrics to report if resource alignment is in effect. The more precise reporting would probably be using pod status, but this would require more invasive and riskier changes, and possibly extra interactions to the APIServer. We start adding metrics to report if containers got their compute resources aligned. If metrics are growing, the assingment is working as expected; If metrics stay consistent, perhaps at zero, no resource alignment is done. Signed-off-by: Francesco Romani <fromani@redhat.com>
wire in the l3 group alignment metrics Signed-off-by: Francesco Romani <fromani@redhat.com>
make sure to log the patched version Signed-off-by: Francesco Romani <fromani@redhat.com>
add more verbose logs by default to make the LLC algorithm debuggable. Signed-off-by: Francesco Romani <fromani@redhat.com>
139d258
to
bb9b972
Compare
@ffromani: the contents of this pull request could not be automatically validated. The following commits could not be validated and must be approved by a top-level approver:
Comment |
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: ffromani The full list of commands accepted by this bot can be found here.
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
@ffromani: all tests passed! Full PR test history. Your PR dashboard. 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-sigs/prow repository. I understand the commands that are listed here. |
WIP DNM