-
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
Migrated pkg/controller/endpoint
to contextual logging
#116755
Conversation
54ce67d
to
fe9ec84
Compare
pkg/controller/endpoint
to contextual logging
/test pull-kubernetes-e2e-kind-ipv6 |
/assign @pohly |
fe9ec84
to
0b83249
Compare
/test pull-kubernetes-conformance-kind-ga-only-parallel |
/test pull-kubernetes-e2e-gce |
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.
Almost done 😄
Can you also remove the exception from
Line 45 in 5f23f83
-contextual k8s.io/kubernetes/pkg/controller/endpoint/.* |
0b83249
to
a4090b4
Compare
Updated, thanks for your review |
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.
/lgtm
/assign @soltysh
For approval.
Signed-off-by: xin.li <[email protected]>
a4090b4
to
325205e
Compare
/sig instrumentation |
ping @soltysh |
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.
/lgtm
LGTM label has been added. Git tree hash: 92d8e3fb559558b7af893c10c83f35b99e76ada4
|
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
/triage accepted |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: my-git9, pohly, soltysh 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 |
What type of PR is this?
/kind feature
What this PR does / why we need it:
Which issue(s) this PR fixes:
Ref kubernetes/enhancements#3077
Special notes for your reviewer:
Does this PR introduce a user-facing change?
Additional documentation e.g., KEPs (Kubernetes Enhancement Proposals), usage docs, etc.: