[Gluster-Maintainers] [Gluster-devel] 'Reviewd-by' tag for commits

Niels de Vos ndevos at redhat.com
Fri Sep 30 15:52:52 UTC 2016


On Fri, Sep 30, 2016 at 08:50:12PM +0530, Ravishankar N wrote:
> On 09/30/2016 06:38 PM, Niels de Vos wrote:
> > On Fri, Sep 30, 2016 at 07:11:51AM +0530, Pranith Kumar Karampuri wrote:
...
> > Maybe we can add an additional tag that mentions all the people that
> > did do reviews of older versions of the patch. Not sure what the tag
> > would be, maybe just CC?
> It depends on what tags would be processed to obtain statistics on review
> contributions.

Real statistics would come from Gerrit, not from the 'git log' output.
We do have a ./extras/who-wrote-glusterfs/ in the sources, but that is
only to get an idea about the changes that were made and should not be
used for serious statistics.

It is possible to feed the Gerrit comment-stream into things like
Elasticsearch and get an accurate impression how many reviews people do
(and much more). I hope we can get some contribution diagrams from
someting like this at one point.

Would some kind of Gave-feedback tag for people that left a comment on
earlier versions of the patch be appreciated by others? It will show in
the 'git log' who was involved in some way or form.

Do others think this would be useful?
Niels
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 801 bytes
Desc: not available
URL: <http://www.gluster.org/pipermail/maintainers/attachments/20160930/54114adc/attachment.sig>


More information about the maintainers mailing list