[PATCH] intg_tests: Add regression test for 2163

List overview All Threads
Download

newer

older

[PATCH] GPO: Use sss_unique_file...

Code style -- for loop iterative...

Lukas Slebodnik

20 Jul 2015 20 Jul '15

2:46 p.m.

ehlo,

attached is a regression test for old ticket 2163. Currently it fails due to fix for ticket 2717

Show replies by date

Lukas Slebodnik

20 Jul 20 Jul

2:52 p.m.

On (20/07/15 15:46), Lukas Slebodnik wrote:

...

ehlo,

attached is a regression test for old ticket 2163. Currently it fails due to fix for ticket 2717

And now with attached patch.

BTW I think that we should split fixtures and statiting sssd and loading data to LDAP should be two separate fixtures.

Jakub Hrozek

22 Jul 22 Jul

10:43 a.m.

On Mon, Jul 20, 2015 at 03:52:52PM +0200, Lukas Slebodnik wrote:

...

On (20/07/15 15:46), Lukas Slebodnik wrote:

...
ehlo,

attached is a regression test for old ticket 2163. Currently it fails due to fix for ticket 2717

And now with attached patch.

BTW I think that we should split fixtures and statiting sssd and loading data to LDAP should be two separate fixtures.

LS

...

From cd765024b9662eef537e1a84f1da23ce29173dcd Mon Sep 17 00:00:00 2001 From: Lukas Slebodnik lslebodn@redhat.com Date: Mon, 20 Jul 2015 14:09:52 +0200 Subject: [PATCH] intg_tests: Add regression test for 2163

src/tests/intg/ldap_test.py | 40 ++++++++++++++++++++++++++++++++++++++++ 1 file changed, 40 insertions(+)

diff --git a/src/tests/intg/ldap_test.py b/src/tests/intg/ldap_test.py index afc77d7023ea31757def84f21e5d15997ba72013..914e0d0cc8680d73f0caed233965dfc537daf896 100644 --- a/src/tests/intg/ldap_test.py +++ b/src/tests/intg/ldap_test.py @@ -141,6 +141,38 @@ def sanity_rfc2307(request, ldap_conn): create_sssd_fixture(request) return None

+@pytest.fixture +def simple_rfc2307(request, ldap_conn):
ent_list = ldap_ent.List(LDAP_BASE_DN)

ent_list.add_user('usr\\001', 181818, 181818)

ent_list.add_group("group1", 181818)

create_ldap_fixture(request, ldap_conn, ent_list)

conf = unindent("""\
   [sssd]
   config_file_version = 2
   domains             = LDAP
   services            = nss, pam
   [nss]
   [pam]
   [domain/LDAP]
   ldap_auth_disable_tls_never_use_in_production = true
   ldap_schema         = rfc2307
   id_provider         = ldap
   auth_provider       = ldap
   ldap_uri            = {ldap_conn.ds_inst.ldap_url}
   ldap_search_base    = {ldap_conn.ds_inst.base_dn}
""").format(**locals())

create_conf_fixture(request, conf)

create_sssd_fixture(request)

return None
@pytest.fixture def sanity_rfc2307_bis(request, ldap_conn): ent_list = ldap_ent.List(LDAP_BASE_DN) @@ -200,6 +232,14 @@ def sanity_rfc2307_bis(request, ldap_conn): create_sssd_fixture(request) return None

+def test_refression_ticket2163(ldap_conn, simple_rfc2307):

~~~~~~~~~~ typo: Should read regression.

Otherwise LGTM.

...

ent.assert_passwd_by_name(
   'usr\\001',
   dict(name='usr\\001', passwd='*', uid=181818, gid=181818,
        gecos='181818', shell='/bin/bash'))
def test_sanity_rfc2307(ldap_conn, sanity_rfc2307): passwd_pattern = ent.contains_only( dict(name='user1', passwd='*', uid=1001, gid=2001, gecos='1001', dir='/home/user1', shell='/bin/bash'), -- 2.4.3

...

sssd-devel mailing list sssd-devel@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/sssd-devel

Lukas Slebodnik

24 Jul 24 Jul

4:24 p.m.

On (22/07/15 11:43), Jakub Hrozek wrote:

...

On Mon, Jul 20, 2015 at 03:52:52PM +0200, Lukas Slebodnik wrote:

...
On (20/07/15 15:46), Lukas Slebodnik wrote:

...
ehlo,

attached is a regression test for old ticket 2163. Currently it fails due to fix for ticket 2717

And now with attached patch.

BTW I think that we should split fixtures and statiting sssd and loading data to LDAP should be two separate fixtures.

LS

...
From cd765024b9662eef537e1a84f1da23ce29173dcd Mon Sep 17 00:00:00 2001 From: Lukas Slebodnik lslebodn@redhat.com Date: Mon, 20 Jul 2015 14:09:52 +0200 Subject: [PATCH] intg_tests: Add regression test for 2163

src/tests/intg/ldap_test.py | 40 ++++++++++++++++++++++++++++++++++++++++ 1 file changed, 40 insertions(+)

diff --git a/src/tests/intg/ldap_test.py b/src/tests/intg/ldap_test.py index afc77d7023ea31757def84f21e5d15997ba72013..914e0d0cc8680d73f0caed233965dfc537daf896 100644 --- a/src/tests/intg/ldap_test.py +++ b/src/tests/intg/ldap_test.py @@ -141,6 +141,38 @@ def sanity_rfc2307(request, ldap_conn): create_sssd_fixture(request) return None

+@pytest.fixture +def simple_rfc2307(request, ldap_conn):
ent_list = ldap_ent.List(LDAP_BASE_DN)

ent_list.add_user('usr\\001', 181818, 181818)

ent_list.add_group("group1", 181818)

create_ldap_fixture(request, ldap_conn, ent_list)

conf = unindent("""\
   [sssd]
   config_file_version = 2
   domains             = LDAP
   services            = nss, pam
   [nss]
   [pam]
   [domain/LDAP]
   ldap_auth_disable_tls_never_use_in_production = true
   ldap_schema         = rfc2307
   id_provider         = ldap
   auth_provider       = ldap
   ldap_uri            = {ldap_conn.ds_inst.ldap_url}
   ldap_search_base    = {ldap_conn.ds_inst.base_dn}
""").format(**locals())

create_conf_fixture(request, conf)

create_sssd_fixture(request)

return None
@pytest.fixture def sanity_rfc2307_bis(request, ldap_conn): ent_list = ldap_ent.List(LDAP_BASE_DN) @@ -200,6 +232,14 @@ def sanity_rfc2307_bis(request, ldap_conn): create_sssd_fixture(request) return None

+def test_refression_ticket2163(ldap_conn, simple_rfc2307):
  ~~~~~~~~~~
  typo: Should read regression.
Otherwise LGTM.

Updated patch is attached but it is still failing because bug is not fixed yet.

Lukas Slebodnik

1 Sep 1 Sep

7:49 a.m.

On (24/07/15 17:24), Lukas Slebodnik wrote:

...

On (22/07/15 11:43), Jakub Hrozek wrote:

...
On Mon, Jul 20, 2015 at 03:52:52PM +0200, Lukas Slebodnik wrote:

...
On (20/07/15 15:46), Lukas Slebodnik wrote:

...
ehlo,

attached is a regression test for old ticket 2163. Currently it fails due to fix for ticket 2717

And now with attached patch.

BTW I think that we should split fixtures and statiting sssd and loading data to LDAP should be two separate fixtures.

LS

...
From cd765024b9662eef537e1a84f1da23ce29173dcd Mon Sep 17 00:00:00 2001 From: Lukas Slebodnik lslebodn@redhat.com Date: Mon, 20 Jul 2015 14:09:52 +0200 Subject: [PATCH] intg_tests: Add regression test for 2163

src/tests/intg/ldap_test.py | 40 ++++++++++++++++++++++++++++++++++++++++ 1 file changed, 40 insertions(+)

diff --git a/src/tests/intg/ldap_test.py b/src/tests/intg/ldap_test.py index afc77d7023ea31757def84f21e5d15997ba72013..914e0d0cc8680d73f0caed233965dfc537daf896 100644 --- a/src/tests/intg/ldap_test.py +++ b/src/tests/intg/ldap_test.py @@ -141,6 +141,38 @@ def sanity_rfc2307(request, ldap_conn): create_sssd_fixture(request) return None

+@pytest.fixture +def simple_rfc2307(request, ldap_conn):
ent_list = ldap_ent.List(LDAP_BASE_DN)

ent_list.add_user('usr\\001', 181818, 181818)

ent_list.add_group("group1", 181818)

create_ldap_fixture(request, ldap_conn, ent_list)

conf = unindent("""\
   [sssd]
   config_file_version = 2
   domains             = LDAP
   services            = nss, pam
   [nss]
   [pam]
   [domain/LDAP]
   ldap_auth_disable_tls_never_use_in_production = true
   ldap_schema         = rfc2307
   id_provider         = ldap
   auth_provider       = ldap
   ldap_uri            = {ldap_conn.ds_inst.ldap_url}
   ldap_search_base    = {ldap_conn.ds_inst.base_dn}
""").format(**locals())

create_conf_fixture(request, conf)

create_sssd_fixture(request)

return None
@pytest.fixture def sanity_rfc2307_bis(request, ldap_conn): ent_list = ldap_ent.List(LDAP_BASE_DN) @@ -200,6 +232,14 @@ def sanity_rfc2307_bis(request, ldap_conn): create_sssd_fixture(request) return None

+def test_refression_ticket2163(ldap_conn, simple_rfc2307):
  ~~~~~~~~~~
  typo: Should read regression.
Otherwise LGTM.
Updated patch is attached but it is still failing because bug is not fixed yet.

LS

bump

Jakub Hrozek

10:55 a.m.

On Tue, Sep 01, 2015 at 08:49:49AM +0200, Lukas Slebodnik wrote:

...

On (24/07/15 17:24), Lukas Slebodnik wrote:

...
On (22/07/15 11:43), Jakub Hrozek wrote:

...
On Mon, Jul 20, 2015 at 03:52:52PM +0200, Lukas Slebodnik wrote:

...
On (20/07/15 15:46), Lukas Slebodnik wrote:

...
ehlo,

attached is a regression test for old ticket 2163. Currently it fails due to fix for ticket 2717

And now with attached patch.

BTW I think that we should split fixtures and statiting sssd and loading data to LDAP should be two separate fixtures.

LS

...
From cd765024b9662eef537e1a84f1da23ce29173dcd Mon Sep 17 00:00:00 2001 From: Lukas Slebodnik lslebodn@redhat.com Date: Mon, 20 Jul 2015 14:09:52 +0200 Subject: [PATCH] intg_tests: Add regression test for 2163

src/tests/intg/ldap_test.py | 40 ++++++++++++++++++++++++++++++++++++++++ 1 file changed, 40 insertions(+)

diff --git a/src/tests/intg/ldap_test.py b/src/tests/intg/ldap_test.py index afc77d7023ea31757def84f21e5d15997ba72013..914e0d0cc8680d73f0caed233965dfc537daf896 100644 --- a/src/tests/intg/ldap_test.py +++ b/src/tests/intg/ldap_test.py @@ -141,6 +141,38 @@ def sanity_rfc2307(request, ldap_conn): create_sssd_fixture(request) return None

+@pytest.fixture +def simple_rfc2307(request, ldap_conn):
ent_list = ldap_ent.List(LDAP_BASE_DN)

ent_list.add_user('usr\\001', 181818, 181818)

ent_list.add_group("group1", 181818)

create_ldap_fixture(request, ldap_conn, ent_list)

conf = unindent("""\
   [sssd]
   config_file_version = 2
   domains             = LDAP
   services            = nss, pam
   [nss]
   [pam]
   [domain/LDAP]
   ldap_auth_disable_tls_never_use_in_production = true
   ldap_schema         = rfc2307
   id_provider         = ldap
   auth_provider       = ldap
   ldap_uri            = {ldap_conn.ds_inst.ldap_url}
   ldap_search_base    = {ldap_conn.ds_inst.base_dn}
""").format(**locals())

create_conf_fixture(request, conf)

create_sssd_fixture(request)

return None
@pytest.fixture def sanity_rfc2307_bis(request, ldap_conn): ent_list = ldap_ent.List(LDAP_BASE_DN) @@ -200,6 +232,14 @@ def sanity_rfc2307_bis(request, ldap_conn): create_sssd_fixture(request) return None

+def test_refression_ticket2163(ldap_conn, simple_rfc2307):
  ~~~~~~~~~~
  typo: Should read regression.
Otherwise LGTM.
Updated patch is attached but it is still failing because bug is not fixed yet.

LS
bump

LS

Sorry about the late review.

ACK: http://sssd-ci.idm.lab.eng.brq.redhat.com:8080/job/ci/2404/

Lukas Slebodnik

10:57 a.m.

On (01/09/15 11:55), Jakub Hrozek wrote:

...

On Tue, Sep 01, 2015 at 08:49:49AM +0200, Lukas Slebodnik wrote:

...
On (24/07/15 17:24), Lukas Slebodnik wrote:

...
On (22/07/15 11:43), Jakub Hrozek wrote:

...
On Mon, Jul 20, 2015 at 03:52:52PM +0200, Lukas Slebodnik wrote:

...
On (20/07/15 15:46), Lukas Slebodnik wrote:

...
ehlo,

attached is a regression test for old ticket 2163. Currently it fails due to fix for ticket 2717

And now with attached patch.

BTW I think that we should split fixtures and statiting sssd and loading data to LDAP should be two separate fixtures.

LS

...
From cd765024b9662eef537e1a84f1da23ce29173dcd Mon Sep 17 00:00:00 2001 From: Lukas Slebodnik lslebodn@redhat.com Date: Mon, 20 Jul 2015 14:09:52 +0200 Subject: [PATCH] intg_tests: Add regression test for 2163

src/tests/intg/ldap_test.py | 40 ++++++++++++++++++++++++++++++++++++++++ 1 file changed, 40 insertions(+)

diff --git a/src/tests/intg/ldap_test.py b/src/tests/intg/ldap_test.py index afc77d7023ea31757def84f21e5d15997ba72013..914e0d0cc8680d73f0caed233965dfc537daf896 100644 --- a/src/tests/intg/ldap_test.py +++ b/src/tests/intg/ldap_test.py @@ -141,6 +141,38 @@ def sanity_rfc2307(request, ldap_conn): create_sssd_fixture(request) return None

+@pytest.fixture +def simple_rfc2307(request, ldap_conn):
ent_list = ldap_ent.List(LDAP_BASE_DN)

ent_list.add_user('usr\\001', 181818, 181818)

ent_list.add_group("group1", 181818)

create_ldap_fixture(request, ldap_conn, ent_list)

conf = unindent("""\
   [sssd]
   config_file_version = 2
   domains             = LDAP
   services            = nss, pam
   [nss]
   [pam]
   [domain/LDAP]
   ldap_auth_disable_tls_never_use_in_production = true
   ldap_schema         = rfc2307
   id_provider         = ldap
   auth_provider       = ldap
   ldap_uri            = {ldap_conn.ds_inst.ldap_url}
   ldap_search_base    = {ldap_conn.ds_inst.base_dn}
""").format(**locals())

create_conf_fixture(request, conf)

create_sssd_fixture(request)

return None
@pytest.fixture def sanity_rfc2307_bis(request, ldap_conn): ent_list = ldap_ent.List(LDAP_BASE_DN) @@ -200,6 +232,14 @@ def sanity_rfc2307_bis(request, ldap_conn): create_sssd_fixture(request) return None

+def test_refression_ticket2163(ldap_conn, simple_rfc2307):
  ~~~~~~~~~~
  typo: Should read regression.
Otherwise LGTM.
Updated patch is attached but it is still failing because bug is not fixed yet.

LS
bump

LS
Sorry about the late review.

ACK: http://sssd-ci.idm.lab.eng.brq.redhat.com:8080/job/ci/2404/

Poslal si zlu linku.

Lukas Slebodnik

10:58 a.m.

On (01/09/15 11:57), Lukas Slebodnik wrote:

...

On (01/09/15 11:55), Jakub Hrozek wrote:

...
On Tue, Sep 01, 2015 at 08:49:49AM +0200, Lukas Slebodnik wrote:

...
On (24/07/15 17:24), Lukas Slebodnik wrote:

...
On (22/07/15 11:43), Jakub Hrozek wrote:

...
On Mon, Jul 20, 2015 at 03:52:52PM +0200, Lukas Slebodnik wrote:

...
On (20/07/15 15:46), Lukas Slebodnik wrote: >ehlo, > >attached is a regression test for old ticket 2163. >Currently it fails due to fix for ticket 2717 > And now with attached patch.

BTW I think that we should split fixtures and statiting sssd and loading data to LDAP should be two separate fixtures.

LS

...
From cd765024b9662eef537e1a84f1da23ce29173dcd Mon Sep 17 00:00:00 2001 From: Lukas Slebodnik lslebodn@redhat.com Date: Mon, 20 Jul 2015 14:09:52 +0200 Subject: [PATCH] intg_tests: Add regression test for 2163

src/tests/intg/ldap_test.py | 40 ++++++++++++++++++++++++++++++++++++++++ 1 file changed, 40 insertions(+)

diff --git a/src/tests/intg/ldap_test.py b/src/tests/intg/ldap_test.py index afc77d7023ea31757def84f21e5d15997ba72013..914e0d0cc8680d73f0caed233965dfc537daf896 100644 --- a/src/tests/intg/ldap_test.py +++ b/src/tests/intg/ldap_test.py @@ -141,6 +141,38 @@ def sanity_rfc2307(request, ldap_conn): create_sssd_fixture(request) return None

+@pytest.fixture +def simple_rfc2307(request, ldap_conn):
ent_list = ldap_ent.List(LDAP_BASE_DN)

ent_list.add_user('usr\\001', 181818, 181818)

ent_list.add_group("group1", 181818)

create_ldap_fixture(request, ldap_conn, ent_list)

conf = unindent("""\
   [sssd]
   config_file_version = 2
   domains             = LDAP
   services            = nss, pam
   [nss]
   [pam]
   [domain/LDAP]
   ldap_auth_disable_tls_never_use_in_production = true
   ldap_schema         = rfc2307
   id_provider         = ldap
   auth_provider       = ldap
   ldap_uri            = {ldap_conn.ds_inst.ldap_url}
   ldap_search_base    = {ldap_conn.ds_inst.base_dn}
""").format(**locals())

create_conf_fixture(request, conf)

create_sssd_fixture(request)

return None
@pytest.fixture def sanity_rfc2307_bis(request, ldap_conn): ent_list = ldap_ent.List(LDAP_BASE_DN) @@ -200,6 +232,14 @@ def sanity_rfc2307_bis(request, ldap_conn): create_sssd_fixture(request) return None

+def test_refression_ticket2163(ldap_conn, simple_rfc2307):
  ~~~~~~~~~~
  typo: Should read regression.
Otherwise LGTM.
Updated patch is attached but it is still failing because bug is not fixed yet.

LS
bump

LS
Sorry about the late review.

ACK: http://sssd-ci.idm.lab.eng.brq.redhat.com:8080/job/ci/2404/
Poslal si zlu linku.

Sorry for non-English message. It should have been a private mail.

Jakub Hrozek

11:14 a.m.

On Tue, Sep 01, 2015 at 11:57:36AM +0200, Lukas Slebodnik wrote:

...

On (01/09/15 11:55), Jakub Hrozek wrote:

...
On Tue, Sep 01, 2015 at 08:49:49AM +0200, Lukas Slebodnik wrote:

...
On (24/07/15 17:24), Lukas Slebodnik wrote:

...
On (22/07/15 11:43), Jakub Hrozek wrote:

...
On Mon, Jul 20, 2015 at 03:52:52PM +0200, Lukas Slebodnik wrote:

...
On (20/07/15 15:46), Lukas Slebodnik wrote: >ehlo, > >attached is a regression test for old ticket 2163. >Currently it fails due to fix for ticket 2717 > And now with attached patch.

BTW I think that we should split fixtures and statiting sssd and loading data to LDAP should be two separate fixtures.

LS

...
From cd765024b9662eef537e1a84f1da23ce29173dcd Mon Sep 17 00:00:00 2001 From: Lukas Slebodnik lslebodn@redhat.com Date: Mon, 20 Jul 2015 14:09:52 +0200 Subject: [PATCH] intg_tests: Add regression test for 2163

src/tests/intg/ldap_test.py | 40 ++++++++++++++++++++++++++++++++++++++++ 1 file changed, 40 insertions(+)

diff --git a/src/tests/intg/ldap_test.py b/src/tests/intg/ldap_test.py index afc77d7023ea31757def84f21e5d15997ba72013..914e0d0cc8680d73f0caed233965dfc537daf896 100644 --- a/src/tests/intg/ldap_test.py +++ b/src/tests/intg/ldap_test.py @@ -141,6 +141,38 @@ def sanity_rfc2307(request, ldap_conn): create_sssd_fixture(request) return None

+@pytest.fixture +def simple_rfc2307(request, ldap_conn):
ent_list = ldap_ent.List(LDAP_BASE_DN)

ent_list.add_user('usr\\001', 181818, 181818)

ent_list.add_group("group1", 181818)

create_ldap_fixture(request, ldap_conn, ent_list)

conf = unindent("""\
   [sssd]
   config_file_version = 2
   domains             = LDAP
   services            = nss, pam
   [nss]
   [pam]
   [domain/LDAP]
   ldap_auth_disable_tls_never_use_in_production = true
   ldap_schema         = rfc2307
   id_provider         = ldap
   auth_provider       = ldap
   ldap_uri            = {ldap_conn.ds_inst.ldap_url}
   ldap_search_base    = {ldap_conn.ds_inst.base_dn}
""").format(**locals())

create_conf_fixture(request, conf)

create_sssd_fixture(request)

return None
@pytest.fixture def sanity_rfc2307_bis(request, ldap_conn): ent_list = ldap_ent.List(LDAP_BASE_DN) @@ -200,6 +232,14 @@ def sanity_rfc2307_bis(request, ldap_conn): create_sssd_fixture(request) return None

+def test_refression_ticket2163(ldap_conn, simple_rfc2307):
  ~~~~~~~~~~
  typo: Should read regression.
Otherwise LGTM.
Updated patch is attached but it is still failing because bug is not fixed yet.

LS
bump

LS
Sorry about the late review.

ACK: http://sssd-ci.idm.lab.eng.brq.redhat.com:8080/job/ci/2404/
Poslal si zlu linku.

Indeed, wrong URL, sorry about that. Here is the correct one: http://sssd-ci.duckdns.org/logs/job/24/04/summary.html

Jakub Hrozek

12:01 p.m.

On Tue, Sep 01, 2015 at 12:14:25PM +0200, Jakub Hrozek wrote:

...

On Tue, Sep 01, 2015 at 11:57:36AM +0200, Lukas Slebodnik wrote:

...
On (01/09/15 11:55), Jakub Hrozek wrote:

...
On Tue, Sep 01, 2015 at 08:49:49AM +0200, Lukas Slebodnik wrote:

...
On (24/07/15 17:24), Lukas Slebodnik wrote:

...
On (22/07/15 11:43), Jakub Hrozek wrote:

...
On Mon, Jul 20, 2015 at 03:52:52PM +0200, Lukas Slebodnik wrote: > On (20/07/15 15:46), Lukas Slebodnik wrote: > >ehlo, > > > >attached is a regression test for old ticket 2163. > >Currently it fails due to fix for ticket 2717 > > > And now with attached patch. > > BTW I think that we should split fixtures > and statiting sssd and loading data to LDAP should be two separate fixtures. > > LS

> From cd765024b9662eef537e1a84f1da23ce29173dcd Mon Sep 17 00:00:00 2001 > From: Lukas Slebodnik lslebodn@redhat.com > Date: Mon, 20 Jul 2015 14:09:52 +0200 > Subject: [PATCH] intg_tests: Add regression test for 2163 > > --- > src/tests/intg/ldap_test.py | 40 ++++++++++++++++++++++++++++++++++++++++ > 1 file changed, 40 insertions(+) > > diff --git a/src/tests/intg/ldap_test.py b/src/tests/intg/ldap_test.py > index afc77d7023ea31757def84f21e5d15997ba72013..914e0d0cc8680d73f0caed233965dfc537daf896 100644 > --- a/src/tests/intg/ldap_test.py > +++ b/src/tests/intg/ldap_test.py > @@ -141,6 +141,38 @@ def sanity_rfc2307(request, ldap_conn): > create_sssd_fixture(request) > return None > > + > +@pytest.fixture > +def simple_rfc2307(request, ldap_conn): > + ent_list = ldap_ent.List(LDAP_BASE_DN) > + ent_list.add_user('usr\\001', 181818, 181818) > + ent_list.add_group("group1", 181818) > + > + create_ldap_fixture(request, ldap_conn, ent_list) > + > + conf = unindent("""\ > + [sssd] > + config_file_version = 2 > + domains = LDAP > + services = nss, pam > + > + [nss] > + > + [pam] > + > + [domain/LDAP] > + ldap_auth_disable_tls_never_use_in_production = true > + ldap_schema = rfc2307 > + id_provider = ldap > + auth_provider = ldap > + ldap_uri = {ldap_conn.ds_inst.ldap_url} > + ldap_search_base = {ldap_conn.ds_inst.base_dn} > + """).format(**locals()) > + create_conf_fixture(request, conf) > + create_sssd_fixture(request) > + return None > + > + > @pytest.fixture > def sanity_rfc2307_bis(request, ldap_conn): > ent_list = ldap_ent.List(LDAP_BASE_DN) > @@ -200,6 +232,14 @@ def sanity_rfc2307_bis(request, ldap_conn): > create_sssd_fixture(request) > return None > > + > +def test_refression_ticket2163(ldap_conn, simple_rfc2307): ~~~~~~~~~~ typo: Should read regression.

Otherwise LGTM.

Updated patch is attached but it is still failing because bug is not fixed yet.

LS

bump

LS

Sorry about the late review.

ACK: http://sssd-ci.idm.lab.eng.brq.redhat.com:8080/job/ci/2404/

Poslal si zlu linku.

Indeed, wrong URL, sorry about that. Here is the correct one: http://sssd-ci.duckdns.org/logs/job/24/04/summary.html

* master: 03a4bb0706635ef6dcb238d90e8abce9a3d8e45f

Jakub Hrozek

20 Jul 20 Jul

2:55 p.m.

On Mon, Jul 20, 2015 at 03:46:57PM +0200, Lukas Slebodnik wrote:

...

ehlo,

attached is a regression test for old ticket 2163. Currently it fails due to fix for ticket 2717

LS

You forgot the patch.

3432

Age (days ago)

3475

Last active (days ago)

sssd-devel@lists.fedorahosted.org

10 comments

2 participants

tags (0)

participants (2)

Jakub Hrozek
Lukas Slebodnik