Change in vdsm[master]: Changing the order of the services management alternatives

ybronhei at redhat.com ybronhei at redhat.com
Sun Jan 12 13:06:01 UTC 2014


Yaniv Bronhaim has posted comments on this change.

Change subject: Changing the order of the services management alternatives
......................................................................


Patch Set 1:

sure, but service.py doesn't work that way. it tries all alternatives in specific order. both works for retrieving the service status, but it depends on what you used at first to start the service. 

its not wrong to start libvirtd with sysV, but vdsmd already starts libvirtrd with initctl, and now when using the service_status we should use initctl for that.

its either changing the alternatives order or have a specific solution for libvirtd, as far as I see it (as you describe as a table with one column for libvirtd)

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I678cee34d02ebbdcccd49f4c95cca9aa90f6fab3
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Yaniv Bronhaim <ybronhei at redhat.com>
Gerrit-Reviewer: Alon Bar-Lev <alonbl at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Nir Soffer <nsoffer 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
Gerrit-HasComments: No


More information about the vdsm-patches mailing list