Hello Federico Simoncelli, Dan Kenigsberg,
I'd like you to do a code review. Please visit
to review the following change.
Change subject: spec: Enable lvm selinux fix for Fedora ......................................................................
spec: Enable lvm selinux fix for Fedora
In case we cannot get the real fix in systemd quickly, this patch enables the temporary hack on Fedora.
Change-Id: I29e54ee1e52d323d15452c2c1975b63d6c9fa7e8 Bug-Url: https://bugzilla.redhat.com/1127460 Signed-off-by: Nir Soffer nsoffer@redhat.com Reviewed-on: http://gerrit.ovirt.org/33555 Reviewed-by: Federico Simoncelli fsimonce@redhat.com Reviewed-by: Dan Kenigsberg danken@redhat.com --- M vdsm.spec.in 1 file changed, 1 insertion(+), 1 deletion(-)
git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/28/33628/1
diff --git a/vdsm.spec.in b/vdsm.spec.in index 205715a..f867105 100644 --- a/vdsm.spec.in +++ b/vdsm.spec.in @@ -35,7 +35,7 @@ %global with_vhostmd 1 %endif
-%if 0%{?rhel} >= 7 +%if 0%{?fedora} >= 19 || 0%{?rhel} >= 7 %global with_chcon_hack 1 %endif
Nir Soffer has posted comments on this change.
Change subject: spec: Enable lvm selinux fix for Fedora ......................................................................
Patch Set 1: Verified+1
Verified build on el6 el7 and f20
Dan Kenigsberg has posted comments on this change.
Change subject: spec: Enable lvm selinux fix for Fedora ......................................................................
Patch Set 1: Code-Review+2
Yaniv Bronhaim has submitted this change and it was merged.
Change subject: spec: Enable lvm selinux fix for Fedora ......................................................................
spec: Enable lvm selinux fix for Fedora
In case we cannot get the real fix in systemd quickly, this patch enables the temporary hack on Fedora.
Change-Id: I29e54ee1e52d323d15452c2c1975b63d6c9fa7e8 Bug-Url: https://bugzilla.redhat.com/1127460 Signed-off-by: Nir Soffer nsoffer@redhat.com Reviewed-on: http://gerrit.ovirt.org/33555 Reviewed-by: Federico Simoncelli fsimonce@redhat.com Reviewed-by: Dan Kenigsberg danken@redhat.com Reviewed-on: http://gerrit.ovirt.org/33628 --- M vdsm.spec.in 1 file changed, 1 insertion(+), 1 deletion(-)
Approvals: Nir Soffer: Verified Dan Kenigsberg: Looks good to me, approved
oVirt Jenkins CI Server has posted comments on this change.
Change subject: spec: Enable lvm selinux fix for Fedora ......................................................................
Patch Set 2:
Build Failed
http://jenkins.ovirt.org/job/vdsm_3.5_create-rpms-el6-x86_64_merged/28/ : FAILURE
http://jenkins.ovirt.org/job/vdsm_3.5_create-rpms-fc19-x86_64_merged/24/ : SUCCESS
http://jenkins.ovirt.org/job/vdsm_3.5_create-rpms-fc20-x86_64_merged/23/ : SUCCESS
http://jenkins.ovirt.org/job/vdsm_3.5_create-rpms-el7-x86_64_merged/28/ : SUCCESS
vdsm-patches@lists.fedorahosted.org