Nir Soffer has posted comments on this change.

Change subject: Makes the vdsmd load after iscsi
......................................................................


Patch Set 2:

(1 comment)

https://gerrit.ovirt.org/#/c/46486/2/init/systemd/vdsmd.service.in
File init/systemd/vdsmd.service.in:

Line 2: Description=Virtual Desktop Server Manager
Line 3: Requires=multipathd.service libvirtd.service time-sync.target \
Line 4:          rpcbind.service supervdsmd.service sanlock.service \
Line 5:          vdsm-network.service
Line 6: After=vdsm-network.service iscsid.service
If iscsid starts automatically when there is a session, there is no point for 
this dependency, and we should simply remove it.

We should discuss this with platform people, most likely vdsm dependency is 
useless on systemd based systems.
Line 7: Conflicts=libvirt-guests.service ksmtuned.service
Line 8: 
Line 9: [Service]
Line 10: Type=simple


-- 
To view, visit https://gerrit.ovirt.org/46486
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Iad87c430ed4257cb2b7def445ea0020403951392
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: ovirt-3.5.4
Gerrit-Owner: Anatoly Litovsky <[email protected]>
Gerrit-Reviewer: Allon Mureinik <[email protected]>
Gerrit-Reviewer: Anatoly Litovsky <[email protected]>
Gerrit-Reviewer: Dan Kenigsberg <[email protected]>
Gerrit-Reviewer: Fabian Deutsch <[email protected]>
Gerrit-Reviewer: Nir Soffer <[email protected]>
Gerrit-Reviewer: Yaniv Bronhaim
Gerrit-Reviewer: [email protected]
Gerrit-HasComments: Yes
_______________________________________________
vdsm-patches mailing list
[email protected]
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches

Reply via email to