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" <amukh...@redhat.com>
To: "Raghavendra Gowdappa" <rgowd...@redhat.com>
Cc: "Niels de Vos" <nde...@redhat.com>, "Vijaikumar M" <vmall...@redhat.com>, 
"Raghavendra G"
<raghaven...@gluster.com>, "Gluster Devel" <gluster-devel@gluster.org>
Sent: Wednesday, June 24, 2015 10:55:02 AM
Subject: Re: [Gluster-devel] /tests/bugs/quota/bug-1153964.t is consistently 
failing



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

----- Original Message -----
From: "Atin Mukherjee" <amukh...@redhat.com>
To: "Niels de Vos" <nde...@redhat.com>, "Vijaikumar M"
<vmall...@redhat.com>
Cc: "Raghavendra G" <raghaven...@gluster.com>, "Gluster Devel"
<gluster-devel@gluster.org>
Sent: Wednesday, June 24, 2015 10:15:12 AM
Subject: Re: [Gluster-devel] /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.shan...@gmail.com
<mailto:yknev.shan...@gmail.com>> wrote:

    OK. Two reverts of the same patch ;)

    Pick one.

    On Tue, Jun 23, 2015 at 12:51 PM, Raghavendra Gowdappa
    <rgowd...@redhat.com <mailto:rgowd...@redhat.com>> 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" <xhernan...@datalab.es
    <mailto:xhernan...@datalab.es>>
    >> To: "Gluster Devel" <gluster-devel@gluster.org
    <mailto:gluster-devel@gluster.org>>
    >> Sent: Tuesday, June 23, 2015 12:44:47 PM
    >> Subject: [Gluster-devel] /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@gluster.org <mailto:Gluster-devel@gluster.org>
    >> http://www.gluster.org/mailman/listinfo/gluster-devel
    >>
    > _______________________________________________
    > Gluster-devel mailing list
    > Gluster-devel@gluster.org <mailto:Gluster-devel@gluster.org>
    > http://www.gluster.org/mailman/listinfo/gluster-devel
    _______________________________________________
    Gluster-devel mailing list
    Gluster-devel@gluster.org <mailto:Gluster-devel@gluster.org>
    http://www.gluster.org/mailman/listinfo/gluster-devel




--
Raghavendra G


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

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

--
~Atin


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

Reply via email to