[node-patches] Change in ovirt-node[master]: Fixing the virt who deamon starting

[fabiand at redhat.com]

Fabian Deutsch has posted comments on this change.

Change subject: Fixing the virt who deamon starting
......................................................................


Patch Set 4:

(3 comments)

See my inline commetns, then we should be good to go

http://gerrit.ovirt.org/#/c/34705/4//COMMIT_MSG
Commit Message:

Line 7: Fixing the virt who deamon starting
Line 8: 
Line 9: Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1152947
Line 10: 
Line 11: The bug makes the 
please remove the trailing space
Line 12: rsyslog socket to run by correctly symlinks the syslog to rsyslog.
Line 13: Added a target to start the virt services and adding it as requirment
Line 14: into multi-user target of systemd.
Line 15: 


Line 9: Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1152947
Line 10: 
Line 11: The bug makes the 
Line 12: rsyslog socket to run by correctly symlinks the syslog to rsyslog.
Line 13: Added a target to start the virt services and adding it as requirment
Please adjust thios part of the message to match the changed implementation
Line 14: into multi-user target of systemd.
Line 15: 
Line 16: Change-Id: Ia65a334e4fadbbb7ee6b6b7b066c356da147ee16


http://gerrit.ovirt.org/#/c/34705/4/recipe/rhevh7-post.ks
File recipe/rhevh7-post.ks:

Line 155: sed -i "s/syslog\.target/syslog.socket/" /usr/lib/systemd/system/virt-who.service
Line 156: sed -i "s/;Requires/Requires/" /lib/systemd/system/rsyslog.service
Line 157: ln -s /lib/systemd/system/rsyslog.service /etc/systemd/system/syslog.service
Line 158: 
Line 159: #adding extra target to start all the virtualization services
Please remove this incorrect comment
Line 160: #those service are libvirtd as described in its commens and virt-who as requested in bug
Line 161: systemctl enable libvirtd.service
Line 162: systemctl enable virt-who.service
Line 163: 


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

Gerrit-MessageType: comment
Gerrit-Change-Id: Ia65a334e4fadbbb7ee6b6b7b066c356da147ee16
Gerrit-PatchSet: 4
Gerrit-Project: ovirt-node
Gerrit-Branch: master
Gerrit-Owner: Anatoly Litovsky <tlitovsk at redhat.com>
Gerrit-Reviewer: Anatoly Litovsky <tlitovsk at redhat.com>
Gerrit-Reviewer: Fabian Deutsch <fabiand at redhat.com>
Gerrit-Reviewer: Ryan Barry <rbarry at redhat.com>
Gerrit-Reviewer: automation at ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: Yes