On 10/08/2015 09:13 AM, Oleg Fayans wrote:
On 10/07/2015 04:30 PM, Martin Basti wrote:
I agree, we should rather fix the original issue. But as a temporary
solution, to satisfy downstream, it could do.
On 10/07/2015 04:13 PM, Oleg Fayans wrote:
Workaround looks good, but I prefer not to push it in upstream tests,
because it is not test failure.
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
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.
Manage your subscription for the Freeipa-devel mailing list:
Contribute to FreeIPA: http://www.freeipa.org/page/Contribute/Code