-
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/scheduler/framework/runtime
to use contextual logging
#116842
Migrated pkg/scheduler/framework/runtime
to use contextual logging
#116842
Conversation
Skipping CI for Draft Pull Request. |
/test all |
/test all |
c727f1d
to
220d8ca
Compare
/test all |
1 similar comment
/test all |
/test pull-kubernetes-node-e2e-containerd |
/triage accepted |
Rebased. |
@kerthcet lgtm? |
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.
Sorry for the slow response, I'm not lazy on this but have something to focus on. I will commit to the review work of other PRs as long as I have time. But maybe I should take more advantages of the gap time.
2774b72
to
bb6b58d
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.
General LGTM, except the discussion about https://github.com/kubernetes/kubernetes/pull/116842/files#r1166613985.
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.
I think we should include the changes in schedule_one.go in this PR
Otherwise we might be repeating the pod
key over and over.
bb6b58d
to
86b4e3e
Compare
86b4e3e
to
fe72899
Compare
/lgtm Thanks! |
LGTM label has been added. Git tree hash: d5ec8d133b13954f5f0301ac2300383fe92cb466
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: alculquicondor, mengjiao-liu 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 |
/remove-kind feature |
What type of PR is this?
/kind feature
What this PR does / why we need it:
Migrated
pkg/scheduler/framework/runtime
to use contextual loggingWhich issue(s) this PR fixes:
Part of #91633 (comment)
Special notes for your reviewer:
Does this PR introduce a user-facing change?
Additional documentation e.g., KEPs (Kubernetes Enhancement Proposals), usage docs, etc.:
/sig instrumentation
/wg structured-logging
/area logging
/sig scheduling