Douglas Schilling Landgraf has posted comments on this change.

Change subject: vdsm.spec: conditional to execute semanage
......................................................................


Patch Set 3:

(2 comments)

....................................................
File vdsm.spec.in
Line 655: fi
Line 656: 
Line 657: # hack until we replace core dump with abrt
Line 658: if /usr/sbin/selinuxenabled; then
Line 659:     /usr/sbin/semanage fcontext -a -t virt_cache_t 
'/var/log/core(/.*)?'
You meant, if it's disabled? If it's disable we are not executing the Mooli's 
hack during the install.
Line 660: fi
Line 661: 
Line 662: /sbin/restorecon -R -v /var/log/core
Line 663: 


Line 658: if /usr/sbin/selinuxenabled; then
Line 659:     /usr/sbin/semanage fcontext -a -t virt_cache_t 
'/var/log/core(/.*)?'
Line 660: fi
Line 661: 
Line 662: /sbin/restorecon -R -v /var/log/core
Ok
Line 663: 
Line 664: %if 0%{?rhel}
Line 665: if [ "$1" -eq 1 ] ; then
Line 666:     /sbin/chkconfig --add vdsmd


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I1d40e6d9dcaac16d74c685e4a6a6d5cb130ac531
Gerrit-PatchSet: 3
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Douglas Schilling Landgraf <[email protected]>
Gerrit-Reviewer: Barak Azulay <[email protected]>
Gerrit-Reviewer: Dan Kenigsberg <[email protected]>
Gerrit-Reviewer: Douglas Schilling Landgraf <[email protected]>
Gerrit-Reviewer: Fabian Deutsch <[email protected]>
Gerrit-Reviewer: Yaniv Bronhaim <[email protected]>
Gerrit-Reviewer: mooli tayer <[email protected]>
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: Yes
_______________________________________________
vdsm-patches mailing list
[email protected]
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches

Reply via email to