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
Migrate /pkg/controller/disruption to structured and contextual logging #119147
Migrate /pkg/controller/disruption to structured and contextual logging #119147
Conversation
Skipping CI for Draft Pull Request. |
/test all |
0e6e166
to
434a98b
Compare
/test pull-kubernetes-unit |
@mengjiao-liu : there are a few more issues from #116021 that need to be fixed (mostly broken imports, but also missing parameters). You can find them by running |
434a98b
to
0267a1d
Compare
/test all |
Ok, there was a delay the day before yesterday, and the relevant errors have been fixed. |
/test pull-kubernetes-verify-strict-lint |
8aff1a7
to
28e3d68
Compare
/retest |
@mengjiao-liu: needs another rebase because of logcheck.conf. This PR is next, I promise! |
dca7221
to
818d58d
Compare
No problem. Done. |
k8s.io/kubernetes/test/integration/disruption is still failing. I'm going to be offline for the next 5 or so hours, but can have a look after that. |
818d58d
to
79d80ad
Compare
79d80ad
to
1986947
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.
/lgtm
/assign @soltysh
For approval.
LGTM label has been added. Git tree hash: 64045a786b141fa5e4d23fb17cc58583e94ad7ff
|
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
/priority important-longterm |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: mengjiao-liu, 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 cleanup
What this PR does / why we need it:
Migrate /pkg/controller/disruption to structured and contextual logging
Continue the work of to #116021
Which issue(s) this PR fixes:
Part of 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.:
/wg structured-logging
/area logging
/sig instrumentation