Hi Justine,

Thanks for the response. Non-committers don't have Apache accounts; are you
suggesting that there wasn't a need to sign in earlier and a change in this
policy is restricting collaborators from triggering Jenkins builds?

Thanks,
Yash

On Wed, May 24, 2023 at 9:30 PM Justine Olshan <jols...@confluent.io.invalid>
wrote:

> Yash,
>
> When I rebuild, I go to the CloudBees CI page and I have to log in with my
> apache account.
> Not sure if the change in the build system or the need to sign in is part
> of the problem.
>
>
> On Wed, May 24, 2023 at 4:54 AM Federico Valeri <fedeval...@gmail.com>
> wrote:
>
> > +1 on Divij suggestions
> >
> >
> > On Wed, May 24, 2023 at 12:04 PM Divij Vaidya <divijvaidy...@gmail.com>
> > wrote:
> > >
> > > Hey folks
> > >
> > > A week into this experiment, I am finding the ability to add labels,
> > > request for reviewers and ability to close PRs very useful.
> > >
> > > 1. May I suggest an improvement to the process by requesting for some
> > > guidance on the interest areas for various committers. This would help
> us
> > > request for reviews from the right set of individuals.
> > > As a reference, we have tried something similar with Apache TinkerPop
> > (see
> > > TinkerPop Contributors section at the end) [1], where the committers
> self
> > > identify their preferred area of interest.
> > >
> > > 2. I would also request creation of the following new labels:
> > > tiered-storage, transactions, security, refactor, zk-migration,
> > > first-contribution (so that we can prioritize reviews for first time
> > > contributors as an encouragement), build, metrics
> > >
> > > [1] https://tinkerpop.apache.org/
> > >
> > > --
> > > Divij Vaidya
> > >
> > >
> > >
> > > On Mon, May 15, 2023 at 11:07 PM John Roesler <vvcep...@apache.org>
> > wrote:
> > >
> > > > Hello again, all,
> > > >
> > > > Just a quick update: after merging the changes to asf.yaml, I
> received
> > a
> > > > notification that the list is limited to only 10 people, not 20 as
> the
> > > > documentation states.
> > > >
> > > > Here is the list of folks who will now be able to triage PRs and
> > trigger
> > > > builds: Victoria Xia, Greg Harris, Divij Vaidya, Lucas Brutschy, Yash
> > > > Mayya, Philip Nee, vamossagar12, Christo Lolov, Federico Valeri, and
> > andymg3
> > > >
> > > > Thanks all,
> > > > -John
> > > >
> > > > On 2023/05/12 15:53:40 John Roesler wrote:
> > > > > Thanks again for bringing this up, David!
> > > > >
> > > > > As an update to the community, the PMC has approved a process to
> make
> > > > use of this feature.
> > > > >
> > > > > Here are the relevant updates:
> > > > >
> > > > > PR to add the policy:
> https://github.com/apache/kafka-site/pull/510
> > > > >
> > > > > PR to update the list: https://github.com/apache/kafka/pull/13713
> > > > >
> > > > > Ticket to automate this process.. Contributions welcome :)
> > > > https://issues.apache.org/jira/browse/KAFKA-14995
> > > > >
> > > > > And to make sure it doesn't fall through the cracks in the mean
> time,
> > > > here's the release process step:
> > > >
> >
> https://cwiki.apache.org/confluence/display/KAFKA/Release+Process#ReleaseProcess-UpdatetheCollaboratorsList
> > > > >
> > > > > Unfortunately, the "collaborator" feature only allows 20 usernames,
> > so
> > > > we have decided to simply take the top 20 non-committer authors from
> > the
> > > > past year (according to git shortlog). Congratulations to our new
> > > > collaborators!
> > > > >
> > > > > Victoria Xia, Greg Harris, Divij Vaidya, Lucas Brutschy, Yash
> Mayya,
> > > > Philip Nee, vamossagar12, Christo Lolov, Federico Valeri, and andymg3
> > > > >
> > > > > Thanks,
> > > > > -John
> > > > >
> > > > > On 2023/04/27 18:45:09 David Arthur wrote:
> > > > > > Hey folks,
> > > > > >
> > > > > > I stumbled across this wiki page from the infra team that
> > describes the
> > > > > > various features supported in the ".asf.yaml" file:
> > > > > >
> > > >
> >
> https://cwiki.apache.org/confluence/display/INFRA/Git+-+.asf.yaml+features
> > > > > >
> > > > > > One section that looked particularly interesting was
> > > > > >
> > > >
> >
> https://cwiki.apache.org/confluence/display/INFRA/Git+-+.asf.yaml+features#Git.asf.yamlfeatures-AssigningexternalcollaboratorswiththetriageroleonGitHub
> > > > > >
> > > > > > github:
> > > > > >   collaborators:
> > > > > >     - userA
> > > > > >     - userB
> > > > > >
> > > > > > This would allow us to define non-committers as collaborators on
> > the
> > > > Github
> > > > > > project. Concretely, this means they would receive the "triage"
> > Github
> > > > role
> > > > > > (defined here
> > > > > >
> > > >
> >
> https://docs.github.com/en/organizations/managing-user-access-to-your-organizations-repositories/repository-roles-for-an-organization#permissions-for-each-role
> > > > ).
> > > > > > Practically, this means we could let non-committers do things
> like
> > > > assign
> > > > > > labels and reviewers on Pull Requests.
> > > > > >
> > > > > > I wanted to see what the committer group thought about this
> > feature. I
> > > > > > think it could be useful.
> > > > > >
> > > > > > Cheers,
> > > > > > David
> > > > > >
> > > > >
> > > >
> >
>

Reply via email to