[Gluster-devel] review.gluster.com misses a verification

Niels de Vos ndevos at redhat.com
Tue Jun 4 12:38:48 UTC 2013


On Tue, Jun 04, 2013 at 01:34:56PM +0200, Emmanuel Dreyfus wrote:
> Hi
> 
> I received this message telling me that my change wwas verified. However, 
> http://review.gluster.com/#/c/5139/ says "Need verified"

There are two test-runs done by the Gluster Build System (Jenkins). The 
first is a smoke test that checks for compile problems and the validity 
of the bug. The second test-run is the verification, all regression 
tests under <glusterfs>/tests/ are run. When the regression test has 
finished successfully, the Gluster Build System adds the verification 
status to the posted review.

I have now scheduled a regression test for your patch, and it is 
currently pending:
- http://build.gluster.org/job/regression/ 

HTH,
Niels

> 
> What is wrong here?
> 
> ------- Begin Forwarded Message -------
> 
> Subject:     Change in glusterfs[release-3.4]: Fix crash in 
>              dht_migration_complete_check_task because of NU...
> From:        "Gluster Build System (Code Review)" 
>              <review at dev.gluster.org>
> To:          Emmanuel Dreyfus <manu at netbsd.org>
> Date:        Mon, 3 Jun 2013 21:15:30 -0700
> 
> Gluster Build System has posted comments on this change.
> 
> Change subject: Fix crash in dht_migration_complete_check_task because of NULL
> fd
> ......................................................................
> 
> 
> Patch Set 1:
> 
> Build Successful 
> 
> http://build.gluster.org/job/smoke/2882/ : SUCCESS
> 
> http://build.gluster.org/job/rh-bugid/986/ : SUCCESS
> 
> -- 
> To view, visit http://review.gluster.org/5139
> To unsubscribe, visit http://review.gluster.org/settings
> 
> Gerrit-MessageType: comment
> Gerrit-Change-Id: Ibf3547a775d7ca2f3a097c880cdf38ffafb322da
> Gerrit-PatchSet: 1
> Gerrit-Project: glusterfs
> Gerrit-Branch: release-3.4
> Gerrit-Owner: Emmanuel Dreyfus <manu at netbsd.org>
> Gerrit-Reviewer: Gluster Build System <jenkins at build.gluster.com>
> Gerrit-HasComments: No
> 
> -------- End Forwarded Message --------
> 
> 
> -- 
> Emmanuel Dreyfus
> http://hcpnet.free.fr/pubz
> manu at netbsd.org
> 
> _______________________________________________
> Gluster-devel mailing list
> Gluster-devel at nongnu.org
> https://lists.nongnu.org/mailman/listinfo/gluster-devel

-- 
Niels de Vos
Sr. Software Maintenance Engineer
Support Engineering Group
Red Hat Global Support Services




More information about the Gluster-devel mailing list