On Fri, Aug 12, 2016 at 05:12:44PM +0200, Lukas Slebodnik wrote:
> On (09/08/16 19:12), Lukas Slebodnik wrote:
> >On (09/08/16 14:54), Jakub Hrozek wrote:
> >>On Tue, Aug 09, 2016 at 12:36:11PM +0200, Jakub Hrozek wrote:
> >>> On Wed, Aug 03, 2016 at 09:56:40AM +0200, Lukas Slebodnik wrote:
> >>> > On (13/07/16 17:48), Lukas Slebodnik wrote:
> >>> > >ehlo,
> >>> > >
> >>> > >attched patch is an integration test for regression #3093.
> >>> > >I prepared a test and I let someone else to fix it :-)
> >>> > >
> >>> > >I will try to find more bugs in downstream tests.
> >>> > >
> >>> > >BTW we might move test from test_ts_cache somewhere else.
> >>> > >But it was the fastest way how to write a test without
enumeration.
> >>> > >
> >>> >
> >>> > As I previously wrote test_ts_cache is not the best place for this
> >>> > test. Attached are patches which some changes in intg tests.
> >>> > There is also test for grups with special characters
> >>> >
> >>> > LS
> >>>
> >>> Thank you, the only comment I have is that we should remove
> >>> ldap_auth_disable_tls_never_use_in_production option. We don't use
it in
> >>> tests (we don't test authentication there at all at the moment) and
it's
> >>> better to not advertise the option.
> >>>
> >I can remove it as part of "moving files" if you wish.
> >
> >>> I'll give a formal ACK once the CI run finishes.
> >>
> >>For some reason, I can't get a 'green' CI run on RHEL-6. I tried
three
> >>times, the last attempt is here:
> >>
http://sssd-ci.duckdns.org/logs/job/51/19/rhel6/ci-build-debug/ci-make-in...
> >>but all had the same errors.
> >>
> >>Did the patches change something in the way the server is set up on
> >>RHEL-6?
> >
> >E SERVER_DOWN: {'desc': "Can't contact LDAP
server"}
> >
> >I have no idea why it should be stopped.
> >We do not stop openldap server. It should be done
> >only in cleanup (end of all tests in module)
> >
> It took me some time to find root of cause these failures.
> The 1st patch fixes the issues. And i got a recomendation from
> freeipa develoepers that it's better to avoid using destructors
> for cleanup. With pytest we should use finalizers or
> with keyword the same as for files.
>
> LS
Thank you for fixing all this. CI passed for me both locally and in
Jenkins:
http://sssd-ci.duckdns.org/logs/job/52/00/summary.html
ACK to all
master:
* e59b73366d3067c576e39a214a34ace2f9a84878
* 1ba26271952de1beeb9e041bedf87d720d3f5680
* a949dfb6b03c70896e6ab3c7a10781e8ecbaadc2
* de19c0af27f1576c13bef183600136851baf767d
* 49a5412cbc98e630de17359c29cb8d6ce0e16168
* 8def4f07aedf2cebf77e816b201d91beac9b964b
* 1f4dc2971bac4ceb0803b18f86a746656a0f1990
* 361f9fe207f82693bbfd40dfe272a4513652856c
* a035366f622f42c865dbaa1579f741a81548fde2
LS