-
Notifications
You must be signed in to change notification settings - Fork 5.1k
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 org member requirements #7380
Conversation
/lgtm |
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
👋 I think we have many Emeritus people that contributed to Kubernetes for a long time and even gave the shape that we are today, but have stepped down due to personal reasons, a change of work scope, or even retirement. We know who are those people, we even add them as EMERITUS on our OWNER and OWNER_ALIASES :) I have the feeling that removing their membership org seems not right, and instead, we should have mapped that Emeritus, per their contributions on past will have their status kept :) |
I would agree with @rikatz |
FWIW one of our long-standing (although low priority) requests to GitHub is for a badge of some kind. A way for us to recognize that they were a contributor and show up on their profile that doesn't map to an org. I do understand the desire to let them stay as org members and have that 'badge', but there have been a few issues with letting people stay around forever. First big thing org membership comes with elevated permissions and responsibilities that REALLY should be reserved for people that are actively contributing to the project. Often them getting removed from the org is the signal to clean them up from OWNERS (thats a related, but separate problem -_-). I also don't want to call out individuals, but something else we have seen is someone that has largely been afk for ages come back for a random lgtm or approve for a coworker. I have personally dm'ed people about this. If a person is an emeritus approver and removed from the org but becomes active again they can point to the owners file with them as an emeritus approver to fast track becoming an org member again. Aging out should be considered a part of the 'contributor lifecycle', recognition is good - but the overall health of the project should be the priority. |
Ah it totally makes sense Bob! Thanks for clarifying it :) |
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.
Just one suggestion.
LGTM otherwise, thank you @mrbobbytables! ❤️
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
With ContribEx hat on.
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 think these clarifications are very helpful
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
/approve
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
(with the GitHub admin + ContribEx hat)
/lgtm
(with the steering hat)
Steering Approvals: |
@mrbobbytables -- since you proposed the changes, I am guessing this is an explicit approval from your side. @cpanato -- Can you please confirm again if you lgtm'ed with your steering hat on? |
Yup, I definitely approve 👍 |
with steering hat: |
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 (Steering)
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: cpanato, justaugustus, MadhavJivrajani, mrbobbytables, nikhita, palnabarun, pnbrown, txn55 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 |
/lgtm |
/lgtm |
We have acks from all of steering, releasing the hold. Thanks all! /hold cancel |
This PR does 2 things:
/committee steering
/hold