LVS
lvs-devel
Google
 
Web LinuxVirtualServer.org

Re: ldirectord: reaping defunct processes when fork=yes and checktimeout

To: Sohgo Takeuchi <sohgo@xxxxxxxxxxxxxxxx>
Subject: Re: ldirectord: reaping defunct processes when fork=yes and checktimeout expires
Cc: lvs-devel@xxxxxxxxxxxxxxx
From: Simon Horman <horms@xxxxxxxxxxxx>
Date: Wed, 22 Feb 2012 09:11:00 +0900
On Wed, Feb 22, 2012 at 12:00:41AM +0900, Sohgo Takeuchi wrote:
> 
> Hi Simon,
> 
> I wrote a patch for ldirectord to fix an issue mentioned in the
> following thread.
> 
>     [lvs-users] defunct checkcommand processes w/ ldirectord
>     http://archive.linuxvirtualserver.org/html/lvs-users/2012-02/msg00018.html
> 
> The patch is attached in this E-mail. You can get it from my
> github's repository too.
> 
> https://github.com/sohgo/resource-agents/tree/reap-child-fork-yes
> 
> I would appreciate you checked and applied my patch to
> ldirectord when you had free time.

Thanks, I will get that pushed upstream.

> 
> Thank you.
> 
> 
> ---
>  ldirectord/ldirectord.in |    2 ++
>  1 files changed, 2 insertions(+), 0 deletions(-)
> 
> diff --git a/ldirectord/ldirectord.in b/ldirectord/ldirectord.in
> index 646d687..ded1cb2 100644
> --- a/ldirectord/ldirectord.in
> +++ b/ldirectord/ldirectord.in
> @@ -2676,9 +2676,11 @@ sub run_child
>               foreach my $r (@$real) {
>                       $0 = "ldirectord $virtual_id checking $$r{server}";
>                       _check_real($v, $r);
> +                     check_signal();
>               }
>               $0 = "ldirectord $virtual_id";
>               sleep $checkinterval;
> +             check_signal();
>               ld_emailalert_resend();
>       }
>  }
> -- 
> 1.7.0.4
> --
> To unsubscribe from this list: send the line "unsubscribe lvs-devel" in
> the body of a message to majordomo@xxxxxxxxxxxxxxx
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
> 
--
To unsubscribe from this list: send the line "unsubscribe lvs-devel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

<Prev in Thread] Current Thread [Next in Thread>