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
probers: log to file #981
probers: log to file #981
Conversation
clusterloader2/pkg/measurement/common/probes/manifests/dnsLookup/dns-prober-deployment.yaml
Outdated
Show resolved
Hide resolved
6e20c01
to
6f5d72b
Compare
/lgtm This LGTM, but let's wait for the kubernetes/test-infra#15895 to get first so we can verify that everything works in the presubmit /hold |
PTAL |
/lgtm /hold |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: mm4tt, oxddr 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 |
/hold cancel |
/retest |
/test pull-perf-tests-clusterloader2 |
/test pull-perf-tests-clusterloader2-kubemark |
Logs has been successfully collected in the presubmit's test (e.g.
https://storage.googleapis.com/kubernetes-jenkins/pr-logs/pull/perf-tests/981/pull-perf-tests-clusterloader2/1230808617817477120/artifacts/e2e-981-62db2-minion-group-m4pp/cl2-dns-lookup.log
)
|
Nice! |
ref kubernetes/kubernetes#87143
/assign @mm4tt