]> review.fuel-infra Code Review - openstack-build/cinder-build.git/commitdiff
Fix running of migrations tests by Jenkins gate
authorRoman Podolyaka <rpodolyaka@mirantis.com>
Fri, 26 Jul 2013 15:44:19 +0000 (18:44 +0300)
committerRoman Podolyaka <rpodolyaka@mirantis.com>
Mon, 29 Jul 2013 16:11:59 +0000 (19:11 +0300)
DB schema migrations tests are always skipped for MySQL
and PostgreSQL backends, when running gate jobs. This is
due to the bug in the function, which checks, if given DB
backend is available.

Fixes bug 1205386

Change-Id: I80d5da64347279b8fb5a975ff0d11f31fe35c6ce

cinder/tests/test_migrations.py

index 3076b1c5b45f1cd21a64d7783317299d757cd47a..306cd69273b639719708b171bd664454ba1fad4c 100644 (file)
@@ -54,7 +54,7 @@ def _get_connect_string(backend,
     if backend == "postgres":
         backend = "postgresql+psycopg2"
 
-    return ("%(backend)s://%(user)s:%(passwd)s@localhost/%(database)s",
+    return ("%(backend)s://%(user)s:%(passwd)s@localhost/%(database)s" %
             {'backend': backend, 'user': user, 'passwd': passwd,
              'database': database})
 
@@ -79,6 +79,7 @@ def _is_backend_avail(backend,
     except Exception:
         # intentionally catch all to handle exceptions even if we don't
         # have any backend code loaded.
+        LOG.exception("Backend %s is not available", backend)
         return False
     else:
         connection.close()