On Wed, Aug 08, 2012 at 03:48:13PM +0300, Dan Kenigsberg wrote:
> On Wed, Aug 08, 2012 at 07:47:02AM -0400, Robert Middleswarth wrote:
> > I have setup patch review on Jenkins.info for newly submitted
> > patches and it seems to be working pretty well over all but last
> > night well tweaking the process I broken it for a few min but that
> > was long enough that about 50 jobs were marked -1 I will be fixing
> > that today by rerunning the jobs. I am sorry if one of your patches
> > was dinged and it should be fixed by this time tomorrow.
> Thanks, Robert, for working on this. It is highly important for me to
> know that something is going to break the build before taking it in.
> However, would it be possible to have a repository where we can review
> the code of the robot?
It's Gerrit Trigger and the code is on github.
> I think it is important for the robot to be less noisy, and
> particularly, never give V+1. This task is reserved to humans that
> actually know what the patch should be doing.
The V+1 has been fixed. Will give 0 when they pass, -1 when they fail.
> Also, I am not at all sure that the robot is limitting itself to be
> running code of trustworthy authors.
Eyal added a feature request for this. This was the result of a
discussion on the infra mailing list.
vdsm-devel mailing list