Re: [Gluster-infra] freebsd smoke failure

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Tue, Jan 12, 2016 at 05:40:39AM -0500, Rajesh Joseph wrote:
> 
> 
> ----- Original Message -----
> > From: "Niels de Vos" <ndevos@xxxxxxxxxx>
> > To: "Rajesh Joseph" <rjoseph@xxxxxxxxxx>
> > Cc: "Atin Mukherjee" <amukherj@xxxxxxxxxx>, "Gluster Devel" <gluster-devel@xxxxxxxxxxx>, "Gluster Infra"
> > <gluster-infra@xxxxxxxxxxx>
> > Sent: Tuesday, January 12, 2016 2:48:36 PM
> > Subject: Re:  [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@xxxxxxxxxx>
> > > > To: "Atin Mukherjee" <amukherj@xxxxxxxxxx>
> > > > Cc: "Gluster Devel" <gluster-devel@xxxxxxxxxxx>, "Gluster Infra"
> > > > <gluster-infra@xxxxxxxxxxx>
> > > > Sent: Tuesday, January 12, 2016 1:37:50 PM
> > > > Subject: Re:  [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

Attachment: signature.asc
Description: PGP signature

_______________________________________________
Gluster-devel mailing list
Gluster-devel@xxxxxxxxxxx
http://www.gluster.org/mailman/listinfo/gluster-devel

[Index of Archives]     [Gluster Users]     [Ceph Users]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux]     [Linux OMAP]     [Linux MIPS]     [eCos]     [Asterisk Internet PBX]     [Linux API]

  Powered by Linux