[Freeipa-devel] [PATCH] 0033 Check all LDAP servers during IPA discovery

[Rob Crittenden]

Alexander Bokovoy wrote:
> On Fri, 02 Dec 2011, Rob Crittenden wrote:
>> Alexander Bokovoy wrote:
>>> Hi,
>>>
>>> This is patch proposal, I haven't checked it with multiple servers
>>> setup yet.
>>>
>>> When discovering IPA LDAP servers through DNS records, look through all
>>> servers found until first success. A master might be not available or
>>> denied access due to anonymous binds disabled, for example, but
>>> replica may succeed.
>>>
>>> Ticket #1827
>>> https://fedorahosted.org/freeipa/ticket/1827
>>
>> Needs a rebase.
>>
>> This works fine but I wonder if someone specifies --server on the
>> command-line if we should try only that server and fail if we can't
>> connect. I can see someone using that so they can specify which
>> server the client uses.
> Rebase attached.
>
> If --server is specified, DNS discovery is bypassed in search() and
> self.server will have the value of --server. That means the code I
> changed will still work as parse_items() accepts a single item as
> well.

I don't see the --server code included in the patch.

rob