On (31/07/15 16:54), Michal Židek wrote:
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
Ticket #2741 seems to be duplicate of #2716
It means that regression is not fixed an patches canot be pushed
LS