-
Notifications
You must be signed in to change notification settings - Fork 38.7k
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
E2E: {cpu,topology} manager: improve debuggability #107915
E2E: {cpu,topology} manager: improve debuggability #107915
Conversation
Make sure to log out the cpu capacity and allocatable for the node running the tests, to make the troubleshooting of test failures easier. Signed-off-by: Francesco Romani <fromani@redhat.com>
The existing cpu/topology manager tests correctly check for the node resources and skip if the detected resources are not enough to run the tests, to avoid false negatives. Unfortunately they do the check against the node capacity, while the correct approach is to check the allocatable resources. The existing check is correct only on a narrow set of cases; otherwise can still lead to false negatives. This PR fixes that. Signed-off-by: Francesco Romani <fromani@redhat.com>
@fromanirh: This issue is currently awaiting triage. If a SIG or subproject determines this is a relevant issue, they will accept it by applying the The 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. |
/sig node |
Even though CI machines _usually_ have at least two cpus, let's rather not assume this holds true, and let's actually check the allocatable CPUs, skipping even the simplest tests if the assumption is broken, to avoid false negatives. Signed-off-by: Francesco Romani <fromani@redhat.com>
A cpu/topology manager e2e test wants to require one exclusive CPU and a share of CPU time; let's round up the allocatable CPU requirements (from 1 to 2) to reduce the chances of false negatives. Signed-off-by: Francesco Romani <fromani@redhat.com>
3924d96
to
7004a71
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
just debuggability updates and more skip logic
/lgtm
/approve
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: fromanirh, SergeyKanzhelev The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
/kind cleanup
/kind failing-test
What this PR does / why we need it:
Improve the debuggability of topology manager e2e tests which were failing lately on the crio lane.
xref: #107805
Which issue(s) this PR fixes:
Fixes N/A
Special notes for your reviewer:
Aiming to make the failures clearer in order to deliver the actual fix
Does this PR introduce a user-facing change?