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

[Petr Spacek]

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.

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