[Gluster-devel] Development work flow

[Amar Tumballi]

On 04/05/2013 01:14 AM, Anand Avati wrote:
> I have added a new section "Submission Disqualifiers" clarifying the
> meaning of negative votes in Gerrit.
>
> http://www.gluster.org/community/documentation/index.php/Development_Work_Flow#Submission_Disqualifiers
>
>

Just wanted to mention few more updates which Avati introduced to the 
workflow

* rh-bugs tests for every patch :
   From now on, patches with bug id not belonging to GlusterFS product 
in Red Hat Bugzilla [1] will fail with -1 vote in verify section. This 
is a step in the direction to reduce possible mistakes of giving a Bug 
Id for a patch with 'private' or 'restricted' bugs. Also, would prevent 
typos if any while entering bug id.

* Review posts update comment in bugzilla:
   Once a patch goes in to gerrit, there will be a bugzilla comment made 
with subject of the patch and URL in bugzilla, so reporter of the issue 
can take action of reviewing the patch or even testing the patch.

* Merge posts commit message of patch as comment in bugzilla:
   When ever a patch gets merged to the upstream, the whole commit 
message will be posted as comment in bugzilla. This is done as a step to 
capture more details in bugzilla (ie, more detailed RCA etc). So it 
would be good if developers posting patches write very elaborate commit 
message which would help everyone to understand what was the problem the 
patch fixed. Future idea is to capture most of these updates in our 
documentation if required.

Regards,
Amar

[1] - https://bugzilla.redhat.com/buglist.cgi?product=GlusterFS