diff options
| author | Stefan Metzmacher <metze@samba.org> | 2015-01-26 08:31:10 +0100 |
|---|---|---|
| committer | Stefan Metzmacher <metze@samba.org> | 2015-02-01 22:42:07 +0100 |
| commit | 0e37189dab01a80c10143a54b80859229e181e9b (patch) | |
| tree | 1fddc09f75793368a606e51fe1e8208bba2df309 /python | |
| parent | 3b482740355c32002f5d653ee022033feb2cdc15 (diff) | |
| download | samba-0e37189dab01a80c10143a54b80859229e181e9b.tar.gz samba-0e37189dab01a80c10143a54b80859229e181e9b.tar.xz samba-0e37189dab01a80c10143a54b80859229e181e9b.zip | |
python/samba/tests: don't lower case path names in connect_samdb()
We should not lower case file names, because we may get a path to sam.ldb.
Now we only lower case ldap urls.
For a long time I got failing private autobuild like this:
[1623(9233)/1718 at 1h28m9s] samba4.urgent_replication.python(dc)(dc:local)
Failed to connect to ldap URL
'ldap:///memdisk/metze/w/b12985/samba/bin/ab/dc/private/sam.ldb' - LDAP client
internal error: NT_STATUS_NO_MEMORY
Failed to connect to
'ldap:///memdisk/metze/w/b12985/samba/bin/ab/dc/private/sam.ldb' with backend
'ldap': (null)
UNEXPECTED(error):
samba4.urgent_replication.python(dc).__main__.UrgentReplicationTests.test_attributeSchema_object(dc:local)
REASON: _StringException: _StringException: Content-Type:
text/x-traceback;charset=utf8,language=python
traceback
322
The problem is that /memdisk/metze/W/ is my test directory instead
of /memdisk/metze/w/.
Signed-off-by: Stefan Metzmacher <metze@samba.org>
Reviewed-by: Jelmer Vernooij <jelmer@samba.org>
Diffstat (limited to 'python')
| -rw-r--r-- | python/samba/tests/__init__.py | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/python/samba/tests/__init__.py b/python/samba/tests/__init__.py index cca82bbd67..bda4adfc5e 100644 --- a/python/samba/tests/__init__.py +++ b/python/samba/tests/__init__.py @@ -185,7 +185,6 @@ def connect_samdb(samdb_url, lp=None, session_info=None, credentials=None, to make proper URL for ldb.connect() while using default parameters for connection based on test environment """ - samdb_url = samdb_url.lower() if not "://" in samdb_url: if not ldap_only and os.path.isfile(samdb_url): samdb_url = "tdb://%s" % samdb_url |