-
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 schedular to structured logging #106305
migrated schedular to structured logging #106305
Conversation
/wg structured-logging |
/remove-sig scheduling |
/assign @serathius |
/retest |
Got |
e5758d5
to
89a566d
Compare
89a566d
to
b60e954
Compare
/triage accepted |
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
/retest
/lgtm |
/milestone v1.23 |
pinging @dims for approval |
/approve |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: dims, pohly, shivanshu1333 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 cleanup
What this PR does / why we need it:
Marking files of
pkg/scheduler
andcmd/kube-scheduler
as migrated to structured loggingWhich issue(s) this PR fixes:
Reference #105841
Special notes for your reviewer:
This is not complete migration of
pkg/scheduler
andcmd/kube-scheduler
, the complete migration is blocked by #106262 and #106276after providing appropriate fixes for the mentioned issues we need to update the packages here as
pkg/scheduler/...
andcmd/kube-scheduler/...
Excluded files to achieve the complete migration of scheduler are
cmd/kube-scheduler/app/options/configfile.go
line-96,and
pkg/scheduler/internal/cache/debugger/dumper.go
line-49, line-60Does this PR introduce a user-facing change?
Additional documentation e.g., KEPs (Kubernetes Enhancement Proposals), usage docs, etc.: