[Freeipa-devel] [PATCH 0015] Add wait_for_dns option to default.conf

[Petr Spacek]

On 19.2.2014 15:11, Petr Spacek wrote:
> On 18.2.2014 17:34, Nathaniel McCallum wrote:
>> On Tue, 2014-02-18 at 17:06 +0100, Petr Viktorin wrote:
>>> On 02/18/2014 04:45 PM, Petr Spacek wrote:
>>>> Hello,
>>>>
>>>> Add wait_for_dns option to default.conf.
>>>>
>>>> This option makes record changes in DNS tree synchronous.
>>>> IPA calls will wait until new data are visible over DNS protocol.
>>>>
>>>> It is intended only for testing - it should prevent tests from
>>>> failing if there is bigger delay between change in LDAP and DNS.
>>>>
>>>> I would recommend value like 10 seconds.
>>>
>>> Here are a few Python nitpicks you requested.
>
> Thank you very much. This new version solves problems you found + adds proper
> handling for real DNS timeouts.
>
>> It seems to me like a more general TimeoutError would be useful in a
>> broader context. DNSTimeout seems overly narrow to me, unless I'm
>> missing something.
>
> I would like to keep them separate. DNSTimeout shouldn't be handled at all
> because it means that your DNS server or database is dead or broken in some
> interesting way.
>
> I assume that generic TimeoutError could be interpreted as 'try it
> again'/'failover' or something like that.
>
> Maybe the DNSTimeout is not the best name, I'm open to suggestions.

I have sent the old version with new name, gggrrr.

-- 
Petr^2 Spacek
-------------- next part --------------
A non-text attachment was scrubbed...
Name: freeipa-pspacek-0015-3-Add-wait_for_dns-option-to-default.conf.patch
Type: text/x-patch
Size: 12947 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/freeipa-devel/attachments/20140219/07172d1a/attachment.bin>