Merge remote-tracking branch 'ansgar/fix-update-db' into merge

* ansgar/fix-update-db:
  Fix dak update-db

Signed-off-by: Joerg Jaspert <joerg@debian.org>

diff --git a/dak/update_db.py b/dak/update_db.py
index 424359a384c807b95247aa2f01f06828330aa24b..77d0097604743c40191da00bf970c11b8b0c90ec 100755 (executable)
--- a/dak/update_db.py
+++ b/dak/update_db.py
@@ -123,7 +123,7 @@ Updates dak's database schema to the lastest version. You should disable crontab
 
         try:
             # Build a connect string
-            if cnf["DB::Service"]:
+            if cnf.has_key("DB::Service"):
                 connect_str = "service=%s" % cnf["DB::Service"]
             else:
                 connect_str = "dbname=%s"% (cnf["DB::Name"])
@@ -134,7 +134,7 @@ Updates dak's database schema to the lastest version. You should disable crontab
 
         except:
             print "FATAL: Failed connect to database"
-            pass
+            sys.exit(1)
 
         database_revision = int(self.get_db_rev())
         logger.log(['transaction id before update: %s' % self.get_transaction_id()])