Yedidyah Bar David has uploaded a new change for review. Change subject: packaging: setup: Fail if a jasper import failed ......................................................................
packaging: setup: Fail if a jasper import failed Change-Id: Ib56f521da16a69546d9c591ae693044f82fec745 Signed-off-by: Yedidyah Bar David <[email protected]> --- M packaging/setup/ovirt_engine_setup/reports/reportsutil.py 1 file changed, 8 insertions(+), 2 deletions(-) git pull ssh://gerrit.ovirt.org:29418/ovirt-reports refs/changes/62/38262/1 diff --git a/packaging/setup/ovirt_engine_setup/reports/reportsutil.py b/packaging/setup/ovirt_engine_setup/reports/reportsutil.py index fa400d7..8128de1 100644 --- a/packaging/setup/ovirt_engine_setup/reports/reportsutil.py +++ b/packaging/setup/ovirt_engine_setup/reports/reportsutil.py @@ -98,12 +98,18 @@ self._javatmp = os.path.join(self._temproot, 'tmp') os.mkdir(self._javatmp) + def _execute(self, *eargs, **kwargs): + rc, stdout, stderr = self._plugin.execute(*eargs, **kwargs) + if rc != 0 or stderr: + self._plugin.logger.error('JasperUtil execute failed') + raise RuntimeError('JasperUtil execute failed') + def jsexport(self, what, args): dest = os.path.join( self._temproot, what, ) - self._plugin.execute( + self._execute( args=( './js-export.sh', '--output-dir', dest, @@ -128,7 +134,7 @@ return dest def jsimport(self, src): - self._plugin.execute( + self._execute( args=( './js-import.sh', '--input-dir', src, -- To view, visit https://gerrit.ovirt.org/38262 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: newchange Gerrit-Change-Id: Ib56f521da16a69546d9c591ae693044f82fec745 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-reports Gerrit-Branch: master Gerrit-Owner: Yedidyah Bar David <[email protected]> _______________________________________________ Engine-patches mailing list [email protected] http://lists.ovirt.org/mailman/listinfo/engine-patches
