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
Pass context to StartTestServer to get contextualized logger from it during tests #115298
Pass context to StartTestServer to get contextualized logger from it during tests #115298
Conversation
/release-note-none |
Please title this with the intention of the change, not what it enables |
/title Pass context to StartTestServer to get contextualized logger from it during tests |
Nope, that's test/integration. |
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.
/approve
/remove-sig api-machinery |
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.
Looks good to me.
ea435d6
to
6a06b63
Compare
/assign @mikedanese |
/remove-sig api-machinery |
/test pull-kubernetes-e2e-gce |
/retest-required |
/test pull-kubernetes-e2e-gce |
A bunch of stuff failing on downdetector right now. I guess pull-kubernetes-e2e-gce and pull-kubernetes-node-e2e-containerd fails are related 🤔 I don't see why they would fail consistently with this PR otherwise |
/retest-required |
/retest |
2 similar comments
/retest |
/retest |
@mikedanese can you please have a look? :) |
Looks like this has ok's from @alculquicondor and @pohly, it lokos good to me as well. /approve |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: alculquicondor, dims, knelasevero 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 |
LGTM label has been added. Git tree hash: f529b56484a222f389dbfb600fecd486d5294ecd
|
/triage accepted |
What type of PR is this?
/kind feature
What this PR does / why we need it:
Changes needed before we go forward with #111155
Which issue(s) this PR fixes:
Special notes for your reviewer:
We need to pass context to StartTestServer to get contextualized logger from it during tests (in that other PR: #111155).
Does this PR introduce a user-facing change?
"NONE"