Sandro Bonazzola has posted comments on this change.

Change subject: packaging: setup: Removing build-conf files on failure
......................................................................


Patch Set 5:

(1 comment)

http://gerrit.ovirt.org/#/c/27167/5/packaging/setup/plugins/ovirt-engine-setup/ovirt-engine-reports/jasper/deploy.py
File 
packaging/setup/plugins/ovirt-engine-setup/ovirt-engine-reports/jasper/deploy.py:

Line 163:                     if os.path.exists(entry):
Line 164:                         shutil.rmtree(entry)
Line 165:                 except Exception as e:
Line 166:                     self._parent.logger.debug(
Line 167:                         'Exception during removing %s',
maybe "while removing" is better than "during removing"
Line 168:                         entry,
Line 169:                         exc_info=True,
Line 170:                     )
Line 171:                     self._parent.logger.error(


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I2507ea0c6ecd6d0a69f56d0693fcafd21f58992f
Gerrit-PatchSet: 5
Gerrit-Project: ovirt-reports
Gerrit-Branch: master
Gerrit-Owner: Simone Tiraboschi <[email protected]>
Gerrit-Reviewer: Sandro Bonazzola <[email protected]>
Gerrit-Reviewer: Simone Tiraboschi <[email protected]>
Gerrit-Reviewer: Yaniv Dary <[email protected]>
Gerrit-Reviewer: Yedidyah Bar David <[email protected]>
Gerrit-Reviewer: [email protected]
Gerrit-HasComments: Yes
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to