Dan Kenigsberg has submitted this change and it was merged. Change subject: multipath.py: use vdsm.tool.service to restart multipathd ......................................................................
multipath.py: use vdsm.tool.service to restart multipathd multipath.py restart and reload multipathd using service command directly. This patch uses vdsm-tool service module to manage the system service improves portability. It covers SysV, SystemD and Upstart as well as possible daemon names in various systems (multipathd is named multipath-tools in Ubuntu). Change-Id: I1130a4baf770b805aaea8de97556d18a2da86207 Signed-off-by: Zhou Zheng Sheng <zhshz...@linux.vnet.ibm.com> Reviewed-on: http://gerrit.ovirt.org/14728 Reviewed-by: Aravinda VK <avish...@redhat.com> Reviewed-by: Ewoud Kohl van Wijngaarden <ew...@kohlvanwijngaarden.nl> Reviewed-by: Dan Kenigsberg <dan...@redhat.com> --- M vdsm/storage/multipath.py M vdsm/sudoers.vdsm.in 2 files changed, 4 insertions(+), 4 deletions(-) Approvals: Ewoud Kohl van Wijngaarden: Looks good to me, but someone else must approve Aravinda VK: Looks good to me, but someone else must approve Dan Kenigsberg: Looks good to me, approved Zhou Zheng Sheng: Verified -- To view, visit http://gerrit.ovirt.org/14728 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I1130a4baf770b805aaea8de97556d18a2da86207 Gerrit-PatchSet: 7 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Zhou Zheng Sheng <zhshz...@linux.vnet.ibm.com> Gerrit-Reviewer: Aravinda VK <avish...@redhat.com> Gerrit-Reviewer: Dan Kenigsberg <dan...@redhat.com> Gerrit-Reviewer: Ewoud Kohl van Wijngaarden <ew...@kohlvanwijngaarden.nl> Gerrit-Reviewer: Vered Volansky <vvola...@redhat.com> Gerrit-Reviewer: Zhou Zheng Sheng <zhshz...@linux.vnet.ibm.com> Gerrit-Reviewer: oVirt Jenkins CI Server _______________________________________________ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches