Dan Kenigsberg has posted comments on this change.
Change subject: BZ#832199: selinux - move selinux from init to spec
......................................................................
Patch Set 1: Looks good to me, approved
(1 inline comment)
....................................................
File vdsm.spec.in
Line 388:
we've introduced `semanage boolean -m` to solve
BZ#662942 set virt_use_nfs in policy even if selinux is disabled
so I'd rather keep it. If you have a better fix for this problem, let us solve it in
another patch, as BZ#832199 was deemed a blocker.
btw, this whole discussion should have happened in
http://gerrit.ovirt.org/#/c/5600/ .
this is only a backport (that should have happened only *after* the upstream commit has
been accepted)
--
To view, visit
http://gerrit.ovirt.org/5614
To unsubscribe, visit
http://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: Id515ddb96cbfb4f3a936336b3f7e261658df662a
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: ovirt-3.1
Gerrit-Owner: Douglas Schilling Landgraf <dougsland(a)redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com>
Gerrit-Reviewer: Federico Simoncelli <fsimonce(a)redhat.com>
Gerrit-Reviewer: Mark Wu <wudxw(a)linux.vnet.ibm.com>