gluster-devel
[Top][All Lists]
Advanced

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

Re: [Gluster-devel] Review attention needed for refactoring of ping-time


From: Raghavendra Gowdappa
Subject: Re: [Gluster-devel] Review attention needed for refactoring of ping-timer implementation in glusterfs
Date: Mon, 21 Apr 2014 05:53:14 -0400 (EDT)

Sure vijay, currently reviewing other patches. Will take it up soon.

----- Original Message -----
| From: "Vijay Bellur" <address@hidden>
| To: "Krishnan Parthasarathi" <address@hidden>, "gluster-devel" 
<address@hidden>, "Raghavendra
| Gowdappa" <address@hidden>, "Pranith Kumar Karampuri" <address@hidden>
| Sent: Sunday, April 20, 2014 9:58:07 PM
| Subject: Re: [Gluster-devel] Review attention needed for refactoring of 
ping-timer implementation in glusterfs
| 
| Looks like a good enhancement to me. Will review this patch soon.
| 
| Raghavendra, Pranith: Can you also please help with review of this patch?
| 
| Thanks,
| Vijay
| 
| On 04/14/2014 04:22 AM, Krishnan Parthasarathi wrote:
| > All,
| >
| > The patch, http://review.gluster.org/5202, refactors the existing
| > ping-timer implementation
| > such that any new rpc program that is introduced into glusterfs codebase
| > gets a heart-beating mechanism, which is already in use between gluster
| > client(s)
| > and bricks, for 'free'. The problem its trying to solve is the lack of
| > heart-beating
| > mechanism among glusterd processes in a cluster. Without this, one is
| > likely to
| > see the cluster is 'hung', when a node goes down, until the network
| > disconnection
| > is detected by other peers. This can take upto 30mins (default TCP
| > Re-transmission timeout).
| >
| > This patch also moves the ping-timer logic to the 'right' layer.
| > Previously, the client xlator had its own private ping timer
| > implementation.
| > With this patch, this implementation is moved into the 'rpc'layer, so that
| > other message channels like glusterd-glusterd can benefit from the
| > ping-timer.
| >
| > This patch has been out there for review for quite some time. It would be
| > really helpful if it gets some review attention. It has been tested in the
| > following scenarios,
| >
| > - Performed both dropping of incoming and outgoing packets to glusterd,
| > using iptables
| >    To block incoming packets,
| >    eg.  iptables -I INPUT -p tcp --dport 553:24007 -j DROP
| >
| >    To block outgoing packets,
| >    eg. iptables -I INPUT 1 -p tcp --dport 553:24007 -j DROP
| >
| >    // please use the above iptables rules carefully and only in your test
| >    machines :-)
| >
| > - Tested if 'old' client, one without the new ping timer implementation,
| > works with 'new' server,
| >    one with the ping timer implementation as in this patch.
| >
| > thanks,
| > Krish
| >
| > _______________________________________________
| > Gluster-devel mailing list
| > address@hidden
| > https://lists.nongnu.org/mailman/listinfo/gluster-devel
| >
| >
| 
| 



reply via email to

[Prev in Thread] Current Thread [Next in Thread]