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
enable benchmark by matching test config #95608
Conversation
Change kubernetes/test-infra@fc3e91a from 105 -> 90 causes this test not to be run.
@MHBauer: 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 |
/retest |
/lgtm |
/assign @derekwaynecarr |
/approve |
/lgtm |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: derekwaynecarr, dims, MHBauer 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 |
Change kubernetes/test-infra@fc3e91a from 105 -> 90 causes this test
not to be run.
What type of PR is this?
/kind bug
/kind regression
What this PR does / why we need it:
Enables test that has not been running. Runs both in dockerd and containerd as separate prow jobs
https://testgrid.k8s.io/sig-node-kubelet#node-kubelet-benchmark&sort-by-name=
https://testgrid.k8s.io/sig-node-containerd#node-e2e-benchmark
I'm assuming good faith that someone does want this to run.
It has not been running since June.
I think we do not need it and we should remove these tests/benchmarks.
Which issue(s) this PR fixes:
Found while investigating #95043
Special notes for your reviewer:
Does this PR introduce a user-facing change?:
/cc @dims @bart0sh
/assign @dchen1107