[Gluster-devel] Review attention needed for refactoring of ping-timer implementation in glusterfs

Vijay Bellur vbellur at redhat.com
Sun Apr 20 16:28:07 UTC 2014

Looks like a good enhancement to me. Will review this patch soon.

Raghavendra, Pranith: Can you also please help with review of this patch?


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
> Gluster-devel at nongnu.org
> https://lists.nongnu.org/mailman/listinfo/gluster-devel

More information about the Gluster-devel mailing list