Yaniv Bronhaim has posted comments on this change.

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


Patch Set 1:

It's a bit hard for me to explain.. maybe you can assist. The reason for that 
was the way we stop and start libvirtd, which controlled by initctl thanks to 
our hack in libvirtd_sysv2upstart. The service.py tool tries all alternatives, 
using SysV verbs will work and will ruin our upstart usage. This case mainly 
relevant over RHEL 6 which contains both inits to use. With the patch I'm 
avoiding the problem of mixing up the usage of the services management and 
prefer the initctl one which we use to start libvirt.

Without it, we check libvirtd status with systemctl (which works..) and get 
that the service is stopped which is wrong.

Any idea for another option to handle this ? or another option to explain it ?

-- 
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 <[email protected]>
Gerrit-Reviewer: Alon Bar-Lev <[email protected]>
Gerrit-Reviewer: Dan Kenigsberg <[email protected]>
Gerrit-Reviewer: Yaniv Bronhaim <[email protected]>
Gerrit-Reviewer: Zhou Zheng Sheng <[email protected]>
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: No
_______________________________________________
vdsm-patches mailing list
[email protected]
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches

Reply via email to