[PATCH] Fix monitor ping timeout

[Stephen Gallagher]

Our configuration specifies the monitor config timeout as seconds,
but we were passing it directly to dbus commands that require
milliseconds. Fixing this in get_monitor_config(). Also, the
default value of -1 for the timeout resulted in a timeout much too
short to be useful, so I'm making it 10s instead.

This fix solves the problem where a busy backend (for example, one
that is enumerating a large number of LDAP entries) would fail to
respond to the ping in time.

-- 
Stephen Gallagher
RHCE 804006346421761

Looking to carve out IT costs?
www.redhat.com/carveoutcosts/

[Simo Sorce]

On Tue, 2009-08-11 at 10:49 -0400, Stephen Gallagher wrote:
>  
>  Our configuration specifies the monitor config timeout as seconds,
>  but we were passing it directly to dbus commands that require
>  milliseconds. Fixing this in get_monitor_config(). Also, the
>  default value of -1 for the timeout resulted in a timeout much too
>  short to be useful, so I'm making it 10s instead.
>  
>  This fix solves the problem where a busy backend (for example, one
>  that is enumerating a large number of LDAP entries) would fail to
>  respond to the ping in time. 
ack.

Simo.

[Stephen Gallagher]

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

On 08/11/2009 12:24 PM, Simo Sorce wrote:
>  On Tue, 2009-08-11 at 10:49 -0400, Stephen Gallagher wrote:
> > Our configuration specifies the monitor config timeout as seconds,
> > but we were passing it directly to dbus commands that require
> > milliseconds. Fixing this in get_monitor_config(). Also, the
> > default value of -1 for the timeout resulted in a timeout much too
> > short to be useful, so I'm making it 10s instead.
> >
> > This fix solves the problem where a busy backend (for example, one
> > that is enumerating a large number of LDAP entries) would fail to
> > respond to the ping in time.
>  
>  ack.
>  
>  Simo.
>  
>  _______________________________________________
>  sssd-devel mailing list
>  sssd-devel(a)lists.fedorahosted.org
>  https://fedorahosted.org/mailman/listinfo/sssd-devel 
Pushed to master.

- -- 
Stephen Gallagher
RHCE 804006346421761

Looking to carve out IT costs?
www.redhat.com/carveoutcosts/
> -----BEGIN PGP SIGNATURE-----
> Version: GnuPG v1.4.9 (GNU/Linux)
> Comment: Using GnuPG with Fedora - http://enigmail.mozdev.org/
>
> iEYEARECAAYFAkqBnKkACgkQeiVVYja6o6NwxwCfS0FpgxfuAdBJEoA72yPQNodo
> XLoAnjtnmnc/fWnByppQJYBtjD4seegB
> =zTBl
> -----END PGP SIGNATURE-----