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

[Martin Basti]

On 10/08/2015 11:18 AM, Oleg Fayans wrote:
> Done
> On 10/08/2015 10:37 AM, Martin Basti wrote:
>>
>>
>> On 10/08/2015 09:13 AM, Oleg Fayans wrote:
>>> 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
>>>
>> Can you please send this as separate patch? I would like to push this 
>> one.
>
ACK

Pushed to:
master: 2b4354f37e7e775dae833d5e2e8824b43800855f
ipa-4-2: f076da99946c0405162c88174e771a5cecfe9664