Re: /tests/bugs/quota/bug-1153964.t is consistently failing

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

 



Sorry, my bad. I didn't see an error while doing the pull on my local repository and I thought it was updated while it wasn't.

Xavi

On 06/25/2015 02:26 PM, Sachin Pandit wrote:


----- Original Message -----
From: "Sachin Pandit" <spandit@xxxxxxxxxx>
To: "Xavier Hernandez" <xhernandez@xxxxxxxxxx>
Cc: "Gluster Devel" <gluster-devel@xxxxxxxxxxx>, "Raghavendra G" <raghavendra@xxxxxxxxxxx>
Sent: Thursday, June 25, 2015 5:50:01 PM
Subject: Re:  /tests/bugs/quota/bug-1153964.t is consistently failing

Hi Xavier,

I did a git pull and I could see the commit
"01e42f25ebdc44847e8b1dce02f7fd486b40dbc2"
in my repository. This commit is the revert of the commit
"8ab6608accb62d6320d1fc1fbe651fcafd376270"

~ Sachin.

----- Original Message -----
From: "Xavier Hernandez" <xhernandez@xxxxxxxxxx>
To: "Vijaikumar M" <vmallika@xxxxxxxxxx>, "Gluster Devel"
<gluster-devel@xxxxxxxxxxx>
Cc: "Raghavendra G" <raghavendra@xxxxxxxxxxx>
Sent: Thursday, June 25, 2015 5:41:14 PM
Subject: Re:  /tests/bugs/quota/bug-1153964.t is
consistently failing

It seems that the patch has not been correctly committed into master. I
see it as merged in gerrit, but doing a 'git pull' on master on a local
repository doesn't seem to include it.

And it looks like [1] was rebased on Jun 24 at "10:02 AM"
and the reverted patch [2] got merged on Jun 24 at "11:41 AM".
That is the reason why [1] is failing the same test case.

[1] http://review.gluster.org/#/c/10465/
[2] http://review.gluster.org/#/c/11362/

~ Sachin.


Am I doing something wrong ?

It also seems that a recently rebased patch is still failing because of
the same problem:

http://build.gluster.org/job/rackspace-regression-2GB-triggered/11258/consoleFull

Xavi

On 06/24/2015 08:21 AM, Vijaikumar M wrote:
Hi All,

I request you to re-base your patch which are failed regression with
test-case bug-1153964.t

Thanks,
Vijay


On Wednesday 24 June 2015 11:42 AM, Raghavendra Gowdappa wrote:
http://review.gluster.org/#/c/11362/ has been merged.

----- Original Message -----
From: "Atin Mukherjee" <amukherj@xxxxxxxxxx>
To: "Raghavendra Gowdappa" <rgowdapp@xxxxxxxxxx>
Cc: "Niels de Vos" <ndevos@xxxxxxxxxx>, "Vijaikumar M"
<vmallika@xxxxxxxxxx>, "Raghavendra G"
<raghavendra@xxxxxxxxxxx>, "Gluster Devel" <gluster-devel@xxxxxxxxxxx>
Sent: Wednesday, June 24, 2015 10:55:02 AM
Subject: Re:  /tests/bugs/quota/bug-1153964.t is
consistently failing



On 06/24/2015 10:53 AM, Raghavendra Gowdappa wrote:

----- Original Message -----
From: "Atin Mukherjee" <amukherj@xxxxxxxxxx>
To: "Niels de Vos" <ndevos@xxxxxxxxxx>, "Vijaikumar M"
<vmallika@xxxxxxxxxx>
Cc: "Raghavendra G" <raghavendra@xxxxxxxxxxx>, "Gluster Devel"
<gluster-devel@xxxxxxxxxxx>
Sent: Wednesday, June 24, 2015 10:15:12 AM
Subject: Re:  /tests/bugs/quota/bug-1153964.t is
consistently failing

When is this patch getting merged, this is blocking other patches
to get
in.
Revert of http://review.gluster.org/11311, is waiting for regression
runs
to pass. There are three patches (duplicates of each other). If
anyone of
them pass both regression runs, I'll merge them. As far as refcounting
mechanism go, it'll take some time to review and merge the patch.
Once the revert patch is merged, we are good to go. Please let us know
once it's merged as post that all patches need rebase.

~Atin
~Atin

On 06/23/2015 06:26 PM, Niels de Vos wrote:
On Tue, Jun 23, 2015 at 05:30:39PM +0530, Vijaikumar M wrote:

On Tuesday 23 June 2015 04:28 PM, Niels de Vos wrote:
On Tue, Jun 23, 2015 at 03:45:43PM +0530, Vijaikumar M wrote:
I have submitted below patch which fixes this issue. I am
handling
memory
clean-up with reference countmechanism.

http://review.gluster.org/#/c/11361
Is there a reason you can not use the (new) refcounting
functions that
were introduceed with http://review.gluster.org/11022 ?
I was not aware that ref-counting patch was merged. Sure we will
use
these
function and re-submit my patch.
Ok, thanks!
Niels

Thanks,
Vijay


It would be nicer to standardize all refcounting mechanisms on one
implementation. I hope we can replace existing refcounting with
this
one
too. Introducing more refcounting ways is not going to be helpful.

Thanks,
Niels

Thanks,
Vijay




On Tuesday 23 June 2015 12:58 PM, Raghavendra G wrote:
Multiple replies to same query. Pick one ;).

On Tue, Jun 23, 2015 at 12:55 PM, Venky Shankar
<yknev.shankar@xxxxxxxxx
<mailto:yknev.shankar@xxxxxxxxx>> wrote:

     OK. Two reverts of the same patch ;)

     Pick one.

     On Tue, Jun 23, 2015 at 12:51 PM, Raghavendra Gowdappa
     <rgowdapp@xxxxxxxxxx <mailto:rgowdapp@xxxxxxxxxx>> wrote:
     > Seems like its a memory corruption caused by:
     > http://review.gluster.org/11311
     >
     > I've reverted the patch at:
     > http://review.gluster.org/11360
     >
     > ----- Original Message -----
     >> From: "Xavier Hernandez" <xhernandez@xxxxxxxxxx
     <mailto:xhernandez@xxxxxxxxxx>>
     >> To: "Gluster Devel" <gluster-devel@xxxxxxxxxxx
     <mailto:gluster-devel@xxxxxxxxxxx>>
     >> Sent: Tuesday, June 23, 2015 12:44:47 PM
>> Subject: /tests/bugs/quota/bug-1153964.t is
     consistently failing
     >>
     >> Hi,
     >>
     >> the quota test bug-1153964.t is failing consistently for
     >> a
     >> totally
     >> unrelated patch. Is this a known issue ?
     >>
     >>

http://build.gluster.org/job/rackspace-regression-2GB-triggered/11142/consoleFull

     >>

http://build.gluster.org/job/rackspace-regression-2GB-triggered/11165/consoleFull

     >>

http://build.gluster.org/job/rackspace-regression-2GB-triggered/11172/consoleFull

     >>

http://build.gluster.org/job/rackspace-regression-2GB-triggered/11191/consoleFull

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




--
Raghavendra G


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

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

--
~Atin


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

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

_______________________________________________
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