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/preemption & defaultpreemption
to use contextual logging
#116835
Migrated pkg/scheduler/framework/preemption & defaultpreemption
to use contextual logging
#116835
Conversation
Skipping CI for Draft Pull Request. |
/test all |
pkg/scheduler/framework/preemption
to use contextual logging
12ce38d
to
dca53ba
Compare
/test all |
/triage accepted |
0f95c63
to
fcc9758
Compare
Moved PR #116633 to this PR based on this comment https://github.com/kubernetes/kubernetes/pull/116633/files#r1152703840. |
/retest |
pkg/scheduler/framework/preemption
to use contextual loggingpkg/scheduler/framework/preemption & defaultpreemption
to use contextual logging
/lgtm |
LGTM label has been added. Git tree hash: a892f2c85b5e2386e3abc9768776ca2d5d559d85
|
/cc @alculquicondor |
Also /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.
Some performance concern and general housekeeping based on https://github.com/kubernetes/community/blob/master/contributors/devel/sig-instrumentation/logging.md#message-style-guidelines
pkg/scheduler/framework/plugins/defaultpreemption/default_preemption.go
Outdated
Show resolved
Hide resolved
fcc9758
to
7d14345
Compare
…emption.go to use contextual logging
7d14345
to
37a9260
Compare
All comments addressed. |
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
/lgtm |
LGTM label has been added. Git tree hash: 06e91435954c3a1a0ced94134b19a87461ec711f
|
[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 |
What type of PR is this?
/kind feature
What this PR does / why we need it:
Migrated the preemption scheduler plugin to use contextual logging
Which issue(s) this PR fixes:
Part of #91633 (comment)
Special notes for your reviewer:
the test file
pkg/scheduler/framework/preemption/preemption_test.go
is updated in PR #116842 https://github.com/kubernetes/kubernetes/pull/116842/files#diff-33df49b5fb3da50ca769dc3b9fc05cc3f55a0f7cd8877eba9047fa1a504f52deDoes 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