Change in vdsm[master]: vdsm-tool: extend service module to cover alternative servic...

danken at redhat.com danken at redhat.com
Mon Jun 17 12:11:44 UTC 2013


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 <zhshzhou at linux.vnet.ibm.com>
Reviewed-on: http://gerrit.ovirt.org/14720
Reviewed-by: Dan Kenigsberg <danken at 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 <zhshzhou at linux.vnet.ibm.com>
Gerrit-Reviewer: Aravinda VK <avishwan at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Mark Wu <wudxw at linux.vnet.ibm.com>
Gerrit-Reviewer: Yaniv Bronhaim <ybronhei at redhat.com>
Gerrit-Reviewer: Zhou Zheng Sheng <zhshzhou at linux.vnet.ibm.com>
Gerrit-Reviewer: oVirt Jenkins CI Server


More information about the vdsm-patches mailing list