[SSSD] [PATCH] Duplicate detection in fail over did not work.

Jakub Hrozek jhrozek at redhat.com
Mon Aug 13 09:13:56 UTC 2012


On Fri, Aug 10, 2012 at 07:12:38PM +0200, Jakub Hrozek wrote:
> On Fri, Aug 10, 2012 at 07:09:13PM +0200, Jakub Hrozek wrote:
> > On Fri, Aug 10, 2012 at 05:13:48PM +0200, Michal Zidek wrote:
> > > This should fix the duplicate detection. Patch is attached. Please
> > > review with caution, i was in hurry.
> > > 
> > > https://fedorahosted.org/sssd/ticket/1472
> > > 
> > > Thanks
> > > Michal
> > 
> > I think the general approach with compare function is good, but it
> > should be a property of the fo_service. I don't think there is any
> > scenario where we'd like to treat two servers with same user_data
> > contents as the same.
> 
> ^^^ Sorry, this should have read "two servers from the same service".

You also need to amend the testsuite as well when changing the fo_
interface, otherwise the test suite doesn't work:

  CC     src/resolv/fail_over_tests-async_resolv.o
src/tests/fail_over-tests.c: In function ‘test_fo_resolve_service’:
src/tests/fail_over-tests.c:233:5: warning: passing argument 5 of ‘fo_add_server’ makes pointer from integer without a cast [enabled by default]
In file included from ./src/providers/dp_backend.h:26:0,
                 from ./src/providers/ldap/sdap.h:25,
                 from ./src/tests/common.h:31,
                 from src/tests/fail_over-tests.c:35:
./src/providers/fail_over.h:120:5: note: expected ‘int (*)(void *, void *)’ but argument is of type ‘int’
src/tests/fail_over-tests.c:233:5: error: too few arguments to function ‘fo_add_server’



More information about the sssd-devel mailing list