[Gluster-infra] [Gluster-devel] Gerrit Access Control

[Nigel Babu]

On Thu, Sep 01, 2016 at 12:43:06PM +0530, Nigel Babu wrote:
> > > Just need a clarification. Does a "commit in the last 90 days" means
> > > merging a patch sent by someone else by maintainer or maintainer sending a
> > > patch to be merged?
> >
>
> Your email needs to either be in Reviewed-By or Author in git log. So you
> either need to send patches or review patches. Ideally, I'm looking for
> activity on Gerrit and this is the easiest way to figure that out. Yes, I'm
> checking across all active branches.
>
> As an additional bonus, this will also give us a list of people who should be
> on the maintainers team, but aren't.
>
> > Interesting question. I was wondering about something similar as well.
> > What about commits/permissions for the different repositories we host on
> > Gerrit? Does each repository has its own maintainers, or is it one group
> > of maintainers that has merge permissions for all repos?
> >
>
> Each repo on Gerrit seems to mostly have it's own permissions. That's
> a sensible way to go about it. Some of them are unused a clean up is coming
> along, but that's later.

I've answered everyone's concerns on this thread. If nobody is opposed to the
idea, shall I go ahead with this?


--
nigelb