Shirly Radco has uploaded a new change for review.

Change subject: reports: remove war file for new db
......................................................................

reports: remove war file for new db

updated abort, when clean install will remove
deployed files.

Change-Id: Id4a88458e583f93874ef6fc6a7f683c0e1760e40
Bug-Url: https://bugzilla.redhat.com/1072357
Signed-off-by: Shirly Radco <sra...@redhat.com>
---
M 
packaging/setup/plugins/ovirt-engine-setup/ovirt-engine-reports/jasper/deploy.py
1 file changed, 8 insertions(+), 4 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/ovirt-reports refs/changes/71/29071/1

diff --git 
a/packaging/setup/plugins/ovirt-engine-setup/ovirt-engine-reports/jasper/deploy.py
 
b/packaging/setup/plugins/ovirt-engine-setup/ovirt-engine-reports/jasper/deploy.py
index 83e1fe0..6133c33 100644
--- 
a/packaging/setup/plugins/ovirt-engine-setup/ovirt-engine-reports/jasper/deploy.py
+++ 
b/packaging/setup/plugins/ovirt-engine-setup/ovirt-engine-reports/jasper/deploy.py
@@ -81,6 +81,9 @@
                     dst = '%s.%s' % (src, suffix)
                     self._restore.append(dict(src=src, dst=dst))
                     os.rename(src, dst)
+                else:
+                    dst = None
+                    self._restore.append(dict(src=src, dst=dst))
 
             if not self._parent.environment[
                 oreportscons.DBEnv.NEW_DATABASE
@@ -132,9 +135,10 @@
             self._parent.logger.info(_('Rolling back Reports files'))
             for entry in self._restore:
                 try:
-                    if os.path.exists(entry['src']):
-                        shutil.rmtree(entry['src'])
-                    os.rename(entry['dst'], entry['src'])
+                    if entry['dst'] is not None:
+                        if os.path.exists(entry['src']):
+                            shutil.rmtree(entry['src'])
+                        os.rename(entry['dst'], entry['src'])
                 except Exception as e:
                     self._parent.logger.debug(
                         'Exception during rename %s->%s',
@@ -155,7 +159,7 @@
 
         def commit(self):
             for entry in self._restore:
-                if os.path.exists(entry['dst']):
+                if if entry['dst'] is not None and 
os.path.exists(entry['dst']):
                     shutil.rmtree(entry['dst'])
 
     def _buildJs(self, cmd, config, noSuffix=False):


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

Gerrit-MessageType: newchange
Gerrit-Change-Id: Id4a88458e583f93874ef6fc6a7f683c0e1760e40
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-reports
Gerrit-Branch: ovirt-engine-reports-3.4
Gerrit-Owner: Shirly Radco <sra...@redhat.com>
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to