Skip to content
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

access: may-start-trybots and approvers #54499

Closed
panjf2000 opened this issue Aug 17, 2022 · 13 comments
Closed

access: may-start-trybots and approvers #54499

panjf2000 opened this issue Aug 17, 2022 · 13 comments
Assignees
Labels
NeedsFix The path to resolution is known, but the work has not been done.
Milestone

Comments

@panjf2000
Copy link
Member

panjf2000 commented Aug 17, 2022

Hi, I'm applying for "may-start-trybots" and "approvers" access, membership of https://github.com/golang as well, thanks!

Email: panjf2000@gmail.com
Contributions: via gerrit and via github
Group: https://go-review.googlesource.com/#/admin/groups/1030,members, https://go-review.googlesource.com/#/admin/groups/1005,members, https://github.com/golang

/cc @golang/release

See https://go.dev/wiki/GerritAccess

@seankhliao seankhliao added the NeedsDecision Feedback is required from experts, contributors, and/or the community before a change can be made. label Aug 17, 2022
@seankhliao
Copy link
Member

cc @golang/release

@panjf2000 panjf2000 changed the title access: may-start-trybots access: may-start-trybots and approvers Aug 17, 2022
@seankhliao seankhliao added this to the Unreleased milestone Aug 20, 2022
@panjf2000
Copy link
Member Author

Hi, I've been working on several CLs recently, it would be a great help if this process could be expedited, thanks~

@heschi heschi added this to In Progress in Go Release Team Aug 30, 2022
@heschi
Copy link
Contributor

heschi commented Sep 2, 2022

Hi Andy,

Thank you for your contributions! After some discussion, we decided that it's too early for Approvers access -- we appreciate all the cleanup work, but it doesn't form a large enough history for us to evaluate. However, we're very happy to give you TryBot access. I've added you to the may-start-trybots group, so you should be able to do that now.

As you've likely read at https://golang.org/wiki/GerritAccess#trybot-access-may-start-trybots, please be sure to at least skim CLs before starting a Trybot run.

Let us know if you have any questions or issues with it, and thanks again!

@heschi heschi closed this as completed Sep 2, 2022
Go Release Team automation moved this from In Progress to Done Sep 2, 2022
@panjf2000
Copy link
Member Author

panjf2000 commented Sep 3, 2022

Hi @heschi , sorry to hear that, I believe I used to do more than just trivial cleanup work, but also feature development, performance optimization and bug fixing, so I wonder what specific prerequisites are needed before I can officially apply for go-approver access? I request go-approvers access to merge my own CLs after they are given +2 vote by at least two Google staff (according to the latest policy of Go team, which I will strictly abide by), sometimes the approvers are just too busy to merge CLs which is ready to be merged after they give +2 votes, and I hate to bother them over and over again until my CLs get merged.

Besides, I was also requesting to be in https://github.com/orgs/golang/teams/go-approvers for https://go.dev/wiki/Gardening and https://github.com/golang/go/wiki/HandlingIssues, as a member of https://github.com/orgs/golang, helping handle some Github issues, please also take a look at this request, thank you!

@ianlancetaylor
Copy link
Contributor

@panjf2000 An approvers grant is normally reserved for people whose CLs are routinely approved with no comments. I don't think your CLs are at that point yet. Sorry.

Issue gardening doesn't require approvers access, as the issues are managed in GitHub, not Gerrit. It's a bit confusing, as the GitHub name is "go-approvers" (not "approvers") for some reason.. I sent you an invitation to the GitHub group.

@golang golang locked and limited conversation to collaborators Oct 20, 2023
@panjf2000
Copy link
Member Author

panjf2000 commented Jan 29, 2024

Hi all,
I'd like to reapply for the Code-Review+2 permission at this point.

I've had a few pending submissions lately, along with a series of pending CLs locally (haven't uploaded yet). It would really help me expedite my work on Gerrit if I have permission to submit CLs that have gained sufficient votes. Thanks!

cc @golang/release

@panjf2000 panjf2000 reopened this Jan 29, 2024
@golang golang unlocked this conversation Jan 29, 2024
@cagedmantis cagedmantis self-assigned this Feb 26, 2024
@gopherbot
Copy link

Change https://go.dev/cl/560255 mentions this issue: devapp/owners: add @panjf2000 as owner for the net and net/http

@panjf2000
Copy link
Member Author

Ping @golang/release

@cagedmantis cagedmantis assigned dmitshur and unassigned cagedmantis Apr 3, 2024
@panjf2000
Copy link
Member Author

Hi @golang/release, I have a few new ready-to-submit CLs recently and my request for +2 permission has been around for quite a while, I wonder what its current status is?

@dmitshur dmitshur added NeedsFix The path to resolution is known, but the work has not been done. and removed NeedsDecision Feedback is required from experts, contributors, and/or the community before a change can be made. labels Apr 12, 2024
@dmitshur
Copy link
Contributor

dmitshur commented Apr 12, 2024

Hi @panjf2000, thanks for contributing to the Go project and asking for approvers access again.

We considered this request. As documented at https://go.dev/wiki/GerritAccess, Gerrit "approvers" access permits reviewing and submitting CLs across the entire Go project, and I know you've expressed in helping with net and net/http package maintenance in particular. It's a high bar to clear to get access to leave +2 reviews, and your contribution history is not currently at a point where we can confidently grant it. We'll decline this request for now, but please feel free to reapply after some time.

For CLs that you've authored that have a +2 Code-Review vote and are ready to be submitted from your side, please don't hesitate to ask the person who left the +2 vote to submit your CL (or to leave an Auto-Submit+1 vote if you don't anticipate needing to push additional patch sets, e.g., if it's just waiting for the last trybot run). Thanks again.

@dmitshur
Copy link
Contributor

dmitshur commented Apr 12, 2024

@panjf2000 The Auto-Submit+1 vote is documented at https://go.dev/wiki/GerritAccess#auto-submit. I meant you can ask the person who left the Code-Review+2 vote to either submit or leave Auto-Submit+1 vote—but it needs to be done by someone who is an approver.

@panjf2000
Copy link
Member Author

It's a high bar to clear to get access to leave +2 reviews, and your contribution history is not currently at a point where we can confidently grant it.

To be frank, I can understand setting the "high bar" for +2 permission, it's just that the criteria for that threshold have always seemed unclear to me. I've been constantly contributing over 120 commits to the Go project across net, net/http, runtime, os, internal/poll, log/slog, encoding, etc. and participating in resolving GitHub issues for almost five years. If that still can't suffice to qualify me, I believe I must haven't been doing things right. Please show me what I have done wrong and point me in the right direction so that I can be on the track of earning a +2 permission afterward.

@panjf2000
Copy link
Member Author

I guess what I'm trying to figure out is how far I am to the +2 permission, because at this moment I really can't tell what more I have to do before I should reapply again, or when to do it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
NeedsFix The path to resolution is known, but the work has not been done.
Projects
Status: Done
Development

No branches or pull requests

7 participants