Dan Kenigsberg has submitted this change and it was merged. Change subject: vdsm-tool: extend service module to cover alternative service names ......................................................................
vdsm-tool: extend service module to cover alternative service names Service names are different on various systems, instead of hard-coded service names for each supported system, we can extend vdsm.tool.service to detect the right service name given some hints (possible alternative names). This makes the service management more flexible and helps to port us to new systems easily. This patch iterates over the alternative service names list for each management operation. Change-Id: I5308c9db0399dbe9c4f0d6308943307e51d3d447 Signed-off-by: Zhou Zheng Sheng <zhshz...@linux.vnet.ibm.com> Reviewed-on: http://gerrit.ovirt.org/14720 Reviewed-by: Dan Kenigsberg <dan...@redhat.com> --- M lib/vdsm/tool/service.py 1 file changed, 23 insertions(+), 14 deletions(-) Approvals: Dan Kenigsberg: Looks good to me, approved Zhou Zheng Sheng: Verified -- To view, visit http://gerrit.ovirt.org/14720 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I5308c9db0399dbe9c4f0d6308943307e51d3d447 Gerrit-PatchSet: 10 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: Mark Wu <wu...@linux.vnet.ibm.com> Gerrit-Reviewer: Yaniv Bronhaim <ybron...@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