On 07/30/2015 05:23 PM, Michal Židek wrote:
On 07/30/2015 05:06 PM, Michal Židek wrote:
> On 07/30/2015 04:15 PM, Lukas Slebodnik wrote:
>> On (29/07/15 17:05), Lukas Slebodnik wrote:
>>> Attached is a new patch set. I also contains fix for #2712 and not
>>> just for
>>> #2716. Otherwise there would be a crash. So it would be better
>>> push them together.
>>>
>>
>> Integration test were failing sporadically due to bug in clien code.
>> 3rd patch fix this problem.
>>
>> Other patches are the same.
>>
>> LS
>>
>
> Integration tests still do not work for me even with
> this version.
>
> $ contrib/ci/run -n -r
> autoreconf: success 00:00:14 ci-autoreconf.log
> DEBUG BUILD: ci-build-debug
> configure: success 00:00:10 ci-build-debug/ci-configure.log
> make-tests: success 00:02:18
> ci-build-debug/ci-make-tests.log
> make-check-valgrind: success 00:01:41
> ci-build-debug/ci-make-check-valgrind.log
> make-intgcheck: failure 00:01:55
> ci-build-debug/ci-make-intgcheck.log
> FAILURE
>
> See sanitized log. These are the newly added _with_mc tests.
>
> Michal
>
Ah... nevermind, I had exported SSS_NSS_USE_MEMCACHE=no in
the terminal where I was running the tests.
On the other hand this proves that your tests really
catch situation when memcache does not work :)
Thank you for expanding the tests coverage.
Tentative ACK to the patches (will fully ack when
CI results finish).
I filled a ticket to track one issue,
but it does not seem to be related to the memcache.
https://fedorahosted.org/sssd/ticket/2741
Michal
--
Senior Principal Intern