Yaniv Dary has submitted this change and it was merged. Change subject: packaging: setup: check service status by rc ......................................................................
packaging: setup: check service status by rc Before this change, the status of services was checked by searching for keywords in the output of 'service $service status'. This didn't work well on locales where the output did not contain the expected keywords. With this change we only check the return code of that command. Bug-Url: https://bugzilla.redhat.com/1074909 Change-Id: I3c7365cb7947a2d79bfaffa669f28ae4a8aa3ba3 Signed-off-by: Yedidyah Bar David <[email protected]> --- M packaging/common_utils.py 1 file changed, 1 insertion(+), 8 deletions(-) Approvals: Yedidyah Bar David: Verified Yaniv Dary: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/25597 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I3c7365cb7947a2d79bfaffa669f28ae4a8aa3ba3 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-reports Gerrit-Branch: ovirt-3.3 Gerrit-Owner: Yedidyah Bar David <[email protected]> Gerrit-Reviewer: Yaniv Dary <[email protected]> Gerrit-Reviewer: Yedidyah Bar David <[email protected]> _______________________________________________ Engine-patches mailing list [email protected] http://lists.ovirt.org/mailman/listinfo/engine-patches
