Change in vdsm[master]: vdsm-tool: extend "service" module to check existence of a s...

danken at redhat.com danken at redhat.com
Thu Jun 13 10:45:33 UTC 2013


Dan Kenigsberg has submitted this change and it was merged.

Change subject: vdsm-tool: extend "service" module to check existence of a service
......................................................................


vdsm-tool: extend "service" module to check existence of a service

Add a service-is-managed command to check if a service is managed by
either of the supported init systems. With the decorator _XXXNative in
previous commit, the implementation of service-is-managed command is
straight forward: just add _XXXNative tag to a action function that
always succeeds.

Change-Id: I23a7d98e39eaacd5f2dfa817bfae6defb5fc9ebe
Signed-off-by: Zhou Zheng Sheng <zhshzhou at linux.vnet.ibm.com>
Reviewed-on: http://gerrit.ovirt.org/15100
Reviewed-by: Dan Kenigsberg <danken at redhat.com>
---
M lib/vdsm/tool/service.py
1 file changed, 28 insertions(+), 0 deletions(-)

Approvals:
  Dan Kenigsberg: Looks good to me, approved
  Zhou Zheng Sheng: Verified


-- 
To view, visit http://gerrit.ovirt.org/15100
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I23a7d98e39eaacd5f2dfa817bfae6defb5fc9ebe
Gerrit-PatchSet: 6
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: 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