Change in vdsm[ovirt-4.0.5]: net: do not require openvswitch for now

[Code Review]

From Dan Kenigsberg <danken(a)redhat.com&gt;:

Dan Kenigsberg has uploaded a new change for review.

Change subject: net: do not require openvswitch for now
......................................................................

net: do not require openvswitch for now

Due to Bug 1378501, openvswitch currently does not start on AMD
machines. We'd like to soft-require OvS until that is solved.

The patch is ovirt-4.0 only because I believe openvswitch would be fixed
before ovirt-4.1 is released.

Change-Id: Ibc2f37e35d26996931008744fb8b6c71fe44f4e8
Bug-Url: https://bugzilla.redhat.com/1391856
Signed-off-by: Dan Kenigsberg <danken(a)redhat.com&gt;
---
M init/systemd/vdsm-network-init.service.in
1 file changed, 2 insertions(+), 1 deletion(-)


  git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/00/66100/1

diff --git a/init/systemd/vdsm-network-init.service.in
b/init/systemd/vdsm-network-init.service.in
index 82c7ded..9f42708 100644
--- a/init/systemd/vdsm-network-init.service.in
+++ b/init/systemd/vdsm-network-init.service.in
@@ -1,7 +1,8 @@
 [Unit]
 Description=Virtual Desktop Server Manager network IP+link restoration
 Wants=network.target
-Requires=@OPENVSWITCHSERVICE@
+# do not require openvswitch until it starts on AMD (BZ#1378501)
+# Requires=@OPENVSWITCHSERVICE@
 After=@OPENVSWITCHSERVICE@
 Before=libvirtd.service
 


-- 
To view, visit https://gerrit.ovirt.org/66100
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: Ibc2f37e35d26996931008744fb8b6c71fe44f4e8
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: ovirt-4.0.5
Gerrit-Owner: Dan Kenigsberg <danken(a)redhat.com&gt;

[Code Review]

From Dan Kenigsberg <danken(a)redhat.com&gt;:

Dan Kenigsberg has posted comments on this change.

Change subject: net: do not require openvswitch for now
......................................................................


Patch Set 1: Verified+1

builds, installs, and runs - even if OvS is off.

-- 
To view, visit https://gerrit.ovirt.org/66100
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Ibc2f37e35d26996931008744fb8b6c71fe44f4e8
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: ovirt-4.0.5
Gerrit-Owner: Dan Kenigsberg <danken(a)redhat.com&gt;
Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com&gt;
Gerrit-Reviewer: Edward Haas <edwardh(a)redhat.com&gt;
Gerrit-Reviewer: Petr Horáček <phoracek(a)redhat.com&gt;
Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org&gt;
Gerrit-HasComments: No

[Code Review]

From Dan Kenigsberg <danken(a)redhat.com&gt;:

Dan Kenigsberg has posted comments on this change.

Change subject: net: do not require openvswitch for now
......................................................................


Patch Set 1: Continuous-Integration+1

unit tests still pass

-- 
To view, visit https://gerrit.ovirt.org/66100
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Ibc2f37e35d26996931008744fb8b6c71fe44f4e8
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: ovirt-4.0.5
Gerrit-Owner: Dan Kenigsberg <danken(a)redhat.com&gt;
Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com&gt;
Gerrit-Reviewer: Edward Haas <edwardh(a)redhat.com&gt;
Gerrit-Reviewer: Petr Horáček <phoracek(a)redhat.com&gt;
Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org&gt;
Gerrit-HasComments: No

[Code Review]

From Dan Kenigsberg <danken(a)redhat.com&gt;:

Dan Kenigsberg has posted comments on this change.

Change subject: net: do not require openvswitch for now
......................................................................


Patch Set 2: Continuous-Integration+1 Verified+1

-- 
To view, visit https://gerrit.ovirt.org/66100
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Ibc2f37e35d26996931008744fb8b6c71fe44f4e8
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: ovirt-4.0.5
Gerrit-Owner: Dan Kenigsberg <danken(a)redhat.com&gt;
Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com&gt;
Gerrit-Reviewer: Edward Haas <edwardh(a)redhat.com&gt;
Gerrit-Reviewer: Petr Horáček <phoracek(a)redhat.com&gt;
Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org&gt;
Gerrit-HasComments: No