Change in vdsm[ovirt-4.0]: automation: remove bond0 hack

Irit Goihman has uploaded a new change for review.

Change subject: automation: remove bond0 hack
......................................................................

automation: remove bond0 hack

`echo -bond0 > /sys/class/net/bonding_masters` is causing
check-merged job to fail since bond0 doesn't exist.
bond0 used to be added implicitly when bonding kernel module
was loaded but there are merged patches that solve this issue.

Change-Id: I7b5ff287058bd9f4063d9eda6921a690645a3c5f
Label: ovirt-4.0-only
Signed-off-by: Irit Goihman <igoihman(a)redhat.com&gt;
---
M automation/deploy.sh
1 file changed, 0 insertions(+), 1 deletion(-)


  git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/89/62889/1

diff --git a/automation/deploy.sh b/automation/deploy.sh
index c024292..c1bfdb5 100644
--- a/automation/deploy.sh
+++ b/automation/deploy.sh
@@ -51,4 +51,3 @@
 echo "######################### Starting up vdsm"
 systemctl start vdsmd
 echo "@@@@@@@@@@@@@@@@@@@@ DONE"
-echo -bond0 > /sys/class/net/bonding_masters


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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I7b5ff287058bd9f4063d9eda6921a690645a3c5f
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: ovirt-4.0
Gerrit-Owner: Irit Goihman <igoihman(a)redhat.com&gt;

gerrit-hooks has posted comments on this change.

Change subject: automation: remove bond0 hack
......................................................................


Patch Set 1: Verified-1

* Update tracker: IGNORE, no Bug-Url found

* Check Bug-Url::ERROR, At least one bug-url is required for the stable branch
* Check merged to previous::WARN, Still missing on branches master

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I7b5ff287058bd9f4063d9eda6921a690645a3c5f
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: ovirt-4.0
Gerrit-Owner: Irit Goihman <igoihman(a)redhat.com&gt;
Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org&gt;
Gerrit-HasComments: No

Irit Goihman has posted comments on this change.

Change subject: automation: remove bond0 hack
......................................................................


Patch Set 1: Verified+1

passes on jenkins
http://jenkins.ovirt.org/job/vdsm_4.0_check-merged-fc23-x86_64/117/console

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I7b5ff287058bd9f4063d9eda6921a690645a3c5f
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: ovirt-4.0
Gerrit-Owner: Irit Goihman <igoihman(a)redhat.com&gt;
Gerrit-Reviewer: Irit Goihman <igoihman(a)redhat.com&gt;
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org&gt;
Gerrit-HasComments: No

Yaniv Bronhaim has posted comments on this change.

Change subject: automation: remove bond0 hack
......................................................................


Patch Set 1: Code-Review+2 Continuous-Integration+1

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I7b5ff287058bd9f4063d9eda6921a690645a3c5f
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: ovirt-4.0
Gerrit-Owner: Irit Goihman <igoihman(a)redhat.com&gt;
Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com&gt;
Gerrit-Reviewer: Irit Goihman <igoihman(a)redhat.com&gt;
Gerrit-Reviewer: Yaniv Bronhaim <ybronhei(a)redhat.com&gt;
Gerrit-HasComments: No

Edward Haas has posted comments on this change.

Change subject: automation: remove bond0 hack
......................................................................


Patch Set 1: Code-Review+1

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I7b5ff287058bd9f4063d9eda6921a690645a3c5f
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: ovirt-4.0
Gerrit-Owner: Irit Goihman <igoihman(a)redhat.com&gt;
Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com&gt;
Gerrit-Reviewer: Edward Haas <edwardh(a)redhat.com&gt;
Gerrit-Reviewer: Eyal Edri <eedri(a)redhat.com&gt;
Gerrit-Reviewer: Irit Goihman <igoihman(a)redhat.com&gt;
Gerrit-Reviewer: Yaniv Bronhaim <ybronhei(a)redhat.com&gt;
Gerrit-HasComments: No

Yaniv Bronhaim has submitted this change and it was merged.

Change subject: automation: remove bond0 hack
......................................................................


automation: remove bond0 hack

`echo -bond0 > /sys/class/net/bonding_masters` is causing
check-merged job to fail since bond0 doesn't exist.
bond0 used to be added implicitly when bonding kernel module
was loaded but there are merged patches that solve this issue.

Change-Id: I7b5ff287058bd9f4063d9eda6921a690645a3c5f
Label: ovirt-4.0-only
Signed-off-by: Irit Goihman <igoihman(a)redhat.com&gt;
Reviewed-on: https://gerrit.ovirt.org/62889
Reviewed-by: Yaniv Bronhaim <ybronhei(a)redhat.com&gt;
Continuous-Integration: Yaniv Bronhaim <ybronhei(a)redhat.com&gt;
Reviewed-by: Edward Haas <edwardh(a)redhat.com&gt;
---
M automation/deploy.sh
1 file changed, 0 insertions(+), 1 deletion(-)

Approvals:
  Yaniv Bronhaim: Looks good to me, approved; Passed CI tests
  Irit Goihman: Verified
  Edward Haas: Looks good to me, but someone else must approve



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I7b5ff287058bd9f4063d9eda6921a690645a3c5f
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: ovirt-4.0
Gerrit-Owner: Irit Goihman <igoihman(a)redhat.com&gt;
Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com&gt;
Gerrit-Reviewer: Edward Haas <edwardh(a)redhat.com&gt;
Gerrit-Reviewer: Eyal Edri <eedri(a)redhat.com&gt;
Gerrit-Reviewer: Irit Goihman <igoihman(a)redhat.com&gt;
Gerrit-Reviewer: Yaniv Bronhaim <ybronhei(a)redhat.com&gt;

gerrit-hooks has posted comments on this change.

Change subject: automation: remove bond0 hack
......................................................................


Patch Set 2:

* Update tracker: IGNORE, no Bug-Url found
* Set MODIFIED::IGNORE, no Bug-Url found.

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I7b5ff287058bd9f4063d9eda6921a690645a3c5f
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: ovirt-4.0
Gerrit-Owner: Irit Goihman <igoihman(a)redhat.com&gt;
Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com&gt;
Gerrit-Reviewer: Edward Haas <edwardh(a)redhat.com&gt;
Gerrit-Reviewer: Eyal Edri <eedri(a)redhat.com&gt;
Gerrit-Reviewer: Irit Goihman <igoihman(a)redhat.com&gt;
Gerrit-Reviewer: Yaniv Bronhaim <ybronhei(a)redhat.com&gt;
Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org&gt;
Gerrit-HasComments: No