Yedidyah Bar David has uploaded a new change for review. Change subject: packaging: setup: filter errors on js-import/export ......................................................................
packaging: setup: filter errors on js-import/export Change-Id: I6e1cd0ff49c2e37649263302dce0a1a44ee4bb53 Signed-off-by: Yedidyah Bar David <[email protected]> --- M packaging/setup/ovirt_engine_setup/reports/reportsutil.py 1 file changed, 17 insertions(+), 1 deletion(-) git pull ssh://gerrit.ovirt.org:29418/ovirt-reports refs/changes/74/39974/1 diff --git a/packaging/setup/ovirt_engine_setup/reports/reportsutil.py b/packaging/setup/ovirt_engine_setup/reports/reportsutil.py index 7a367b2..80aaf6f 100644 --- a/packaging/setup/ovirt_engine_setup/reports/reportsutil.py +++ b/packaging/setup/ovirt_engine_setup/reports/reportsutil.py @@ -22,6 +22,7 @@ import atexit import gettext import os +import re import shutil import tempfile @@ -101,9 +102,24 @@ self._javatmp = os.path.join(self._temproot, 'tmp') os.mkdir(self._javatmp) + _IGNORED_ERRORS = ( + '.*This normal shutdown operation.*', + ) + + _RE_IGNORED_ERRORS = re.compile( + pattern='|'.join(_IGNORED_ERRORS), + ) + def _execute(self, *eargs, **kwargs): rc, stdout, stderr = self._plugin.execute(*eargs, **kwargs) - if rc != 0 or stderr: + + if stderr: + errors = [ + l for l in stderr + if l and not self._RE_IGNORED_ERRORS.match(l) + ] + + if rc != 0 or errors: self._plugin.logger.error('JasperUtil execute failed') raise RuntimeError('JasperUtil execute failed') -- To view, visit https://gerrit.ovirt.org/39974 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: newchange Gerrit-Change-Id: I6e1cd0ff49c2e37649263302dce0a1a44ee4bb53 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
