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
Self nominate Kevin Hannon for reviewer for job controller #121782
Conversation
I have been lead the PodReplacementPolicy KEP for alpha and I helped review/fix some issues in beta. https://github.com/kubernetes/kubernetes/pulls?q=+is%3Apr+reviewed-by%3Akannon92+label%3Asig%2Fapps+ I have also been an active reviewer and helped GA job tracking last release. I hope to continue reviewing Job related code.
This issue is currently awaiting triage. If a SIG or subproject determines this is a relevant issue, they will accept it by applying the The Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
/kind feature ? |
/retest |
/retest |
SGTM, could you please convert the reviewer requirements as a checkbox list here. |
/remove-kind feature |
+1 for the nomination |
+1, although you need a sponsor from a different company :) Thank you for your invaluable contributions! |
i am happy to sponsor! +1 from me. |
/approve |
@dims do you want to give this a milestone 1.29 since its not a code change? |
/approve (this is just editing an OWNERS file, so we should be safe and not subject to code freeze) |
LGTM label has been added. Git tree hash: 5145d9901400bdc214420a62a9233c6d520535c9
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: alculquicondor, dims, kannon92 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 |
Thank you @alculquicondor, @mimowo and @dims ! |
/retest |
I have lead the PodReplacementPolicy KEP for alpha and I helped review/fix some issues in beta.
https://github.com/kubernetes/kubernetes/pulls?q=+is%3Apr+reviewed-by%3Akannon92+label%3Asig%2Fapps+
I have also been an active reviewer and helped GA job tracking last release. I hope to continue reviewing Job related code.
What type of PR is this?
What this PR does / why we need it:
Add kannon92 as a reviewer for the job controller code.
Which issue(s) this PR fixes:
Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?
Additional documentation e.g., KEPs (Kubernetes Enhancement Proposals), usage docs, etc.: