[Freeipa-devel] [PATCHES] 0230-0244 Integration testing framework

[Jan Cholasta]

On 15.7.2013 16:43, Petr Viktorin wrote:
> On 07/11/2013 01:28 PM, Jan Cholasta wrote:
>> On 10.7.2013 17:50, Petr Viktorin wrote:
>>> Yes, I believe that for the integration tests it's better to see what is
>>> going on. Both for manual runs, and also this way the information is
>>> more easily picked up by CI tools.
>>
>> It's harder to see what's going on with so much noise IMHO, I would
>> prefer less verbose output for manual runs by default.
>
> On the other hand, the integration tests take too long and without
> output they may seem stuck. I'd rather not make that the default.
> I'll try to find some balance and make it more easily configurable in
> future patches.

Thanks. Anyway, it's nothing critical, so don't jump through hoops.

>>
>>>> Both the tests in test_simple_replication.py fail for me. I suspect it
>>>> is because the data isn't replicated fast enough, a little delay
>>>> between
>>>> user-show and user-add might fix this.
>>>
>>> Hm, they worked for me.
>>>   I've added a delay. It seems fragile, I wonder what the proper way to
>>> do this would be.
>>
>> Hmm, the "replica -> master" test still fails for me, even with delay
>> increased to 20 s.
>
> Is there anything in the DS logs?

Let me check.

>
>> Something less fragile than delay would certainly be nice.
>
> I'll read up on the DS replication docs, and ask for some guidance.

Great, thank you.

Honza

-- 
Jan Cholasta