[Pacemaker] pingd

[Andrew Beekhof]

On Wed, Sep 1, 2010 at 11:59 AM, Bernd Schubert
<bs_lists at aakef.fastmail.fm> wrote:
> Andrew,
>
> I think pindg is rather broken:
>
> strace -f /usr/lib64/heartbeat/pingd -a pingdnet2  -d 5s -i 1 -n 2 -h
> 10.0.1.16
>
> (which is in fact localhost)
>
> In an  endless loop:
>
> sendmsg(4, {msg_name(16)={sa_family=AF_INET, sin_port=htons(0),
> sin_addr=inet_addr("10.0.1.16")}, msg_iov(1)=[{"\10\0\324\fj\276\0\2pingd-
> v4\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0"..., 39}], msg_controllen=0, msg_flags=0},
> 0) = 39
> recvmsg(4, {msg_name(16)={sa_family=AF_INET, sin_port=htons(0),
> sin_addr=inet_addr("192.168.2.16")},
> msg_iov(1)=[{"E\320\1\22Y\35\0\0@\1\232\215\300\250\2\20\300\250\2\20\3\3B;
> \0\0\0\0E\20\0\366"..., 324}], msg_controllen=0, msg_flags=0}, 0) = 274
> recvmsg(4, 0x7fff498799d0, MSG_ERRQUEUE|MSG_DONTWAIT) = -1 EAGAIN (Resource
> temporarily unavailable)
> write(2, "No error message: -1: Resource t"..., 60No error message: -1:
> Resource temporarily unavailable (11)
>
> My proposal is to rip out all network code out of pingd and to add slightly
> modified files from 'iputils'.

Close, but thats not portable.
Instead use ocf:pacemaker:ping which goes a step further and ditches
the daemon piece altogether.

> As ping files from iputils are under a BSD
> license, those also really should go into different files. I also think there
> already is an existing licence violation, as pingd.c includes a function from
> iputils ping. While the function is marked accordingly, it still does not
> include the original license statement, which is IMHO a clear license
> violation.
>
> I could probably do that quicky, but don't want to do something that is not
> accepted upstream. Please let me know your thoughts.
>
> Thanks,
> Bernd
>
> --
> Bernd Schubert
> DataDirect Networks
>
> _______________________________________________
> Pacemaker mailing list: Pacemaker at oss.clusterlabs.org
> http://oss.clusterlabs.org/mailman/listinfo/pacemaker
>
> Project Home: http://www.clusterlabs.org
> Getting started: http://www.clusterlabs.org/doc/Cluster_from_Scratch.pdf
> Bugs: http://developerbugs.linux-foundation.org/enter_bug.cgi?product=Pacemaker
>