Yedidyah Bar David has uploaded a new change for review.

Change subject: packaging: setup: weird message on cleanup if no db
......................................................................

packaging: setup: weird message on cleanup if no db

Make engine-cleanup continue gracefully if the database was manually
deleted.

Bug-Url: https://bugzilla.redhat.com/1015037
Change-Id: I440e66c9f99377c8dda02681f5e0761c31fdc839
Signed-off-by: Yedidyah Bar David <d...@redhat.com>
---
M packaging/setup/plugins/ovirt-engine-common/db/connection.py
1 file changed, 1 insertion(+), 1 deletion(-)


  git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/40/19840/1

diff --git a/packaging/setup/plugins/ovirt-engine-common/db/connection.py 
b/packaging/setup/plugins/ovirt-engine-common/db/connection.py
index ce9800f..984dafd 100644
--- a/packaging/setup/plugins/ovirt-engine-common/db/connection.py
+++ b/packaging/setup/plugins/ovirt-engine-common/db/connection.py
@@ -121,9 +121,9 @@
                     dbenv[osetupcons.DBEnv.PASSWORD]
                 )
 
+                self.environment.update(dbenv)
                 dbovirtutils = database.OvirtUtils(plugin=self)
                 dbovirtutils.tryDatabaseConnect(dbenv)
-                self.environment.update(dbenv)
                 self.environment[
                     osetupcons.DBEnv.NEW_DATABASE
                 ] = dbovirtutils.isNewDatabase()


-- 
To view, visit http://gerrit.ovirt.org/19840
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I440e66c9f99377c8dda02681f5e0761c31fdc839
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Yedidyah Bar David <d...@redhat.com>
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to