[Freeipa-devel] [PATCH] Workaround for trac N 5348

[Oleg Fayans]

Hi Martin

On 10/07/2015 04:30 PM, Martin Basti wrote:
>
>
> On 10/07/2015 04:13 PM, Oleg Fayans wrote:
>> subj
>>
>>
>>
> Workaround looks good, but I prefer not to push it in upstream tests,
> because it is not test failure.
I agree, we should rather fix the original issue. But as a temporary 
solution, to satisfy downstream, it could do.
>
> Why is there this sleep, this might be useful in upstream tests too, but
> what is the reason to add sleep there?

Without it I kept getting this error:
E           CalledProcessError: Command '['drill', '@localhost', '-k', 
'/etc/trusted-key.key', '-S', 'example.test.', 'SOA']' returned non-zero 
exit status 29

with --pdb option, though, my attempts to re-run the command succeeded, 
so I assumed it was a timing issue, and indeed, this 1 second sleep helped.

>
>           # verify signatures
> +        time.sleep(1)
>           args = [
>
>

Attached is an updated version of the patch with Martin's remarks taken 
into account

-- 
Oleg Fayans
Quality Engineer
FreeIPA team
RedHat.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: freeipa-ofayans-0009.1-Added-a-workaround-for-ticket-N-5348.patch
Type: text/x-patch
Size: 2166 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/freeipa-devel/attachments/20151008/712094bb/attachment.bin>