Change in vdsm[ovirt-3.5]: spec: Enable lvm selinux fix for Fedora

2014-10-01 Thread nsoffer
Hello Federico Simoncelli, Dan Kenigsberg,

I'd like you to do a code review.  Please visit

http://gerrit.ovirt.org/33628

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 
Reviewed-on: http://gerrit.ovirt.org/33555
Reviewed-by: Federico Simoncelli 
Reviewed-by: Dan Kenigsberg 
---
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
 


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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I29e54ee1e52d323d15452c2c1975b63d6c9fa7e8
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: ovirt-3.5
Gerrit-Owner: Nir Soffer 
Gerrit-Reviewer: Dan Kenigsberg 
Gerrit-Reviewer: Federico Simoncelli 
___
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches


Change in vdsm[ovirt-3.5]: spec: Enable lvm selinux fix for Fedora

2014-10-01 Thread nsoffer
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

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I29e54ee1e52d323d15452c2c1975b63d6c9fa7e8
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: ovirt-3.5
Gerrit-Owner: Nir Soffer 
Gerrit-Reviewer: Allon Mureinik 
Gerrit-Reviewer: Dan Kenigsberg 
Gerrit-Reviewer: Federico Simoncelli 
Gerrit-Reviewer: Nir Soffer 
Gerrit-Reviewer: Yaniv Bronhaim 
Gerrit-Reviewer: automat...@ovirt.org
Gerrit-HasComments: No
___
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches


Change in vdsm[ovirt-3.5]: spec: Enable lvm selinux fix for Fedora

2014-10-01 Thread danken
Dan Kenigsberg has posted comments on this change.

Change subject: spec: Enable lvm selinux fix for Fedora
..


Patch Set 1: Code-Review+2

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I29e54ee1e52d323d15452c2c1975b63d6c9fa7e8
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: ovirt-3.5
Gerrit-Owner: Nir Soffer 
Gerrit-Reviewer: Allon Mureinik 
Gerrit-Reviewer: Dan Kenigsberg 
Gerrit-Reviewer: Federico Simoncelli 
Gerrit-Reviewer: Nir Soffer 
Gerrit-Reviewer: Yaniv Bronhaim 
Gerrit-Reviewer: automat...@ovirt.org
Gerrit-HasComments: No
___
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches


Change in vdsm[ovirt-3.5]: spec: Enable lvm selinux fix for Fedora

2014-10-01 Thread ybronhei
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 
Reviewed-on: http://gerrit.ovirt.org/33555
Reviewed-by: Federico Simoncelli 
Reviewed-by: Dan Kenigsberg 
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



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I29e54ee1e52d323d15452c2c1975b63d6c9fa7e8
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: ovirt-3.5
Gerrit-Owner: Nir Soffer 
Gerrit-Reviewer: Allon Mureinik 
Gerrit-Reviewer: Dan Kenigsberg 
Gerrit-Reviewer: Federico Simoncelli 
Gerrit-Reviewer: Nir Soffer 
Gerrit-Reviewer: Yaniv Bronhaim 
Gerrit-Reviewer: automat...@ovirt.org
___
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches


Change in vdsm[ovirt-3.5]: spec: Enable lvm selinux fix for Fedora

2014-10-01 Thread oVirt Jenkins CI Server
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

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I29e54ee1e52d323d15452c2c1975b63d6c9fa7e8
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: ovirt-3.5
Gerrit-Owner: Nir Soffer 
Gerrit-Reviewer: Allon Mureinik 
Gerrit-Reviewer: Dan Kenigsberg 
Gerrit-Reviewer: Federico Simoncelli 
Gerrit-Reviewer: Nir Soffer 
Gerrit-Reviewer: Yaniv Bronhaim 
Gerrit-Reviewer: automat...@ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: No
___
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches