Dan Kenigsberg has submitted this change and it was merged. Change subject: service: make service compatible with systemctl reload rc ......................................................................
service: make service compatible with systemctl reload rc When reloading service with systemctl and the service is inactive rc is 1. Same scenario on service return rc 0. We want service reload rc to be consistent on vdsm and reload is failing when service is down so we will adapt sysV to systemd. Change-Id: I878fb898204f7a8a564941b43b12c024ef208765 Signed-off-by: Yeela Kaplan <[email protected]> Reviewed-on: http://gerrit.ovirt.org/36491 Reviewed-by: Dan Kenigsberg <[email protected]> --- M lib/vdsm/tool/service.py 1 file changed, 6 insertions(+), 1 deletion(-) Approvals: Yeela Kaplan: Verified Dan Kenigsberg: Looks good to me, approved Objections: Nir Soffer: I would prefer that you didn't submit this -- To view, visit http://gerrit.ovirt.org/36491 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I878fb898204f7a8a564941b43b12c024ef208765 Gerrit-PatchSet: 5 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Yeela Kaplan <[email protected]> Gerrit-Reviewer: Dan Kenigsberg <[email protected]> Gerrit-Reviewer: Nir Soffer <[email protected]> Gerrit-Reviewer: Yaniv Bronhaim <[email protected]> Gerrit-Reviewer: Yeela Kaplan <[email protected]> Gerrit-Reviewer: [email protected] Gerrit-Reviewer: oVirt Jenkins CI Server _______________________________________________ vdsm-patches mailing list [email protected] https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
