Alon Bar-Lev has uploaded a new change for review.

Change subject: pki: upgrade: we do not update database any more
......................................................................

pki: upgrade: we do not update database any more

Change-Id: I66fc2f5c5224116547dc247b664da44bf02dbc3a
Signed-off-by: Alon Bar-Lev <[email protected]>
---
M packaging/fedora/setup/engine-upgrade.py
1 file changed, 0 insertions(+), 10 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/01/14601/1

diff --git a/packaging/fedora/setup/engine-upgrade.py 
b/packaging/fedora/setup/engine-upgrade.py
index cf3c924..712e662 100755
--- a/packaging/fedora/setup/engine-upgrade.py
+++ b/packaging/fedora/setup/engine-upgrade.py
@@ -461,15 +461,6 @@
     JKSKEYSTORE = "/etc/pki/ovirt-engine/.keystore"
     TMPAPACHECONF = basedefs.FILE_HTTPD_SSL_CONFIG + ".tmp"
 
-    def mayUpdateDB(self):
-        """Returns True if we may update database.
-
-        It can be false positive but will trigger database transaction,
-        so database will be rolled back upon failure
-
-        """
-        return os.path.exists(self.JKSKEYSTORE)
-
     def prepare(self):
         if os.path.exists(self.JKSKEYSTORE):
             logging.debug("PKI: convert JKS to PKCS#12")
@@ -1291,7 +1282,6 @@
                 if package in name_packages:
                     updateRelatedToDB = True
                     logging.debug("related to database package %s" % package)
-            updateRelatedToDB = updateRelatedToDB or ca.mayUpdateDB()
             updateRelatedToDB = updateRelatedToDB or hostids
 
             # No rollback in this case


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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I66fc2f5c5224116547dc247b664da44bf02dbc3a
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Alon Bar-Lev <[email protected]>
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to