-
Notifications
You must be signed in to change notification settings - Fork 38.6k
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
controller: add approvers for rc/rs #41876
controller: add approvers for rc/rs #41876
Conversation
lgtm. We need an approver listed in pkg/controller/OWNERS to approve the change. @derekwaynecarr could you help? |
cc @grodrigues3 the merge-bot's message has a bug, it should suggest us to find an approver for this PR from pkg/controller/OWNERS |
@k8s-bot tell me a joke |
@janetkuo ptal |
/approve |
[APPROVALNOTIFIER] This PR is APPROVED The following people have approved this PR: janetkuo, kargakis Needs approval from an approver in each of these OWNERS Files:
You can indicate your approval by writing |
Automatic merge from submit-queue |
No description provided.