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