-
Notifications
You must be signed in to change notification settings - Fork 687
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
Update SIG Release teams for 1.21 Release #2405
Conversation
@palnabarun: The label(s) In response to this:
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. |
- palnabarun # 1.20 RT Lead Shadow | ||
- savitharaghunathan # 1.20 RT Lead Shadow | ||
- bai # 1.21 RT Lead Shadow | ||
- oikiki # 1.21 Emeritus Adviser |
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.
Hi Nabarun, Can you replace 1.21 Emeritus Adviser with 1.21 RT Lead Shadow?
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.
Also, I think Kirsten's GH id is @kikisdeliveryservice
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.
Done! 3AM commits are harmful. xD
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.
I got a promotion!! Thank you for your confidence!!!! 😉
ea08fc7
to
58b9f0c
Compare
- palnabarun # 1.20 RT Lead Shadow | ||
- savitharaghunathan # 1.20 RT Lead Shadow | ||
- bai # 1.21 RT Lead Shadow | ||
- kikisdeliveryservice # 1.21 RT Lead Shadow |
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.
lol you refreshed before i could comment :) :)
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.
3AM commits are harmful. xD
58b9f0c
to
4b75820
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
/hold if folks want to verify their comments were 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.
/hold
I might suggest making this PR strictly additive, unless you're positive the contributor in question:
- Is definitely not returning to the Release Team
- Will not hold another role that grants them milestone maintainer privileges
With 1
, you may have shadows returning and you'll end up having to add them back.
I'd wait until the entire team is locked in to do deletions.
Signed-off-by: Nabarun Pal <pal.nabarun95@gmail.com>
4b75820
to
0576d3c
Compare
@justaugustus -- the suggestion makes sense. I've made this changeset strictly additive. |
Here we go! /lgtm |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: hasheddan, justaugustus, palnabarun 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 |
milestone-maintainers
release-team
release-team-leads
ci-signal
ref: kubernetes/sig-release#1404
/assign
/sig release
/area release-team
/priority critical-urgent
/assign @justaugustus @saschagrunert @jeremyrickard @hasheddan @alejandrox1
/cc @kikisdeliveryservice @savitharaghunathan @bai