[Gluster-devel] [Gluster-infra] freebsd smoke failure

[Niels de Vos]

On Tue, Jan 12, 2016 at 05:40:39AM -0500, Rajesh Joseph wrote:
> 
> 
> ----- Original Message -----
> > From: "Niels de Vos" <ndevos at redhat.com>
> > To: "Rajesh Joseph" <rjoseph at redhat.com>
> > Cc: "Atin Mukherjee" <amukherj at redhat.com>, "Gluster Devel" <gluster-devel at gluster.org>, "Gluster Infra"
> > <gluster-infra at gluster.org>
> > Sent: Tuesday, January 12, 2016 2:48:36 PM
> > Subject: Re: [Gluster-devel] [Gluster-infra] freebsd smoke failure
> > 
> > On Tue, Jan 12, 2016 at 03:52:43AM -0500, Rajesh Joseph wrote:
> > > 
> > > 
> > > ----- Original Message -----
> > > > From: "Niels de Vos" <ndevos at redhat.com>
> > > > To: "Atin Mukherjee" <amukherj at redhat.com>
> > > > Cc: "Gluster Devel" <gluster-devel at gluster.org>, "Gluster Infra"
> > > > <gluster-infra at gluster.org>
> > > > Sent: Tuesday, January 12, 2016 1:37:50 PM
> > > > Subject: Re: [Gluster-devel] [Gluster-infra] freebsd smoke failure
> > > > 
> > > > On Tue, Jan 12, 2016 at 11:19:30AM +0530, Atin Mukherjee wrote:
> > > > > I've been observing freebsd smoke failure for all the patches for last
> > > > > few days with the following error:
> > > > > 
> > > > > mkdir: /usr/local/lib/python2.7/site-packages/gluster: Permission
> > > > > denied
> > > > > mkdir: /usr/local/lib/python2.7/site-packages/gluster: Permission
> > > > > denied
> > > > > 
> > > > > Can any one from infra team can help here?
> > > > 
> > > > This is not an infra issue, it is a build/installation issue in Gluster
> > > > that the infra caught.
> > > > 
> > > > http://review.gluster.org/13208 should fix it, but I was at loss why
> > > > only smoke tests got triggered and regression tests not.
> > > > 
> > > 
> > > I am just wondering why are we seeing this smoke failure now and not on
> > > earlier patches?
> > > Is it a regression? I believe smoke was passing sometime back.
> > 
> > I think/suspect that the FreeBSD slaves are now managed by Salt and
> > manual changes might have been undone. If people suggested (like now)
> > that this was an issue on the slave, one of the admins might have opened
> > up the permissions on the directories on request.
> > 
> > Things like this are really bugs in the build system though.
> > 
> > > Anyway, if this is the fix then can we fast track the patch merge?
> > > Thanks Niels for sending the patch.
> > 
> > Well, the failing of the smoke test does not seem to add Verified=-1
> > anymore (no idea why)? Whats the reason of the hurry?
> 
> I was in the impression that all smoke runs should pass. I used to check
> result of smoke test before merging any patch. I guess I am wrong.

No, you were right. Of course all of the smoke tests should pass. I
would love to have different Verified check-boxes in Gerrit so that the
results of the smoke tests do not get overwritten by the regression
test.

We do need to check the results of the smoke test manually now. But, it
seems that only very few maintainers actually pay attention to those :-/

Niels
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: not available
URL: <http://www.gluster.org/pipermail/gluster-devel/attachments/20160113/272fa961/attachment-0001.sig>