Change in vdsm[master]: Restore previous MTU on bond when VLAN with bigger MTU remov...

2012-08-23 Thread danken
Dan Kenigsberg has posted comments on this change.

Change subject: Restore previous MTU on bond when VLAN with bigger MTU removed
..


Patch Set 4: Verified; Looks good to me, approved

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

Gerrit-MessageType: comment
Gerrit-Change-Id: Ifefa5dec03be7795e528821a4d17991eb0f60aea
Gerrit-PatchSet: 4
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Igor Lvovsky ilvov...@redhat.com
Gerrit-Reviewer: Dan Kenigsberg dan...@redhat.com
Gerrit-Reviewer: Igor Lvovsky ilvov...@redhat.com
Gerrit-Reviewer: Livnat Peer lp...@redhat.com
Gerrit-Reviewer: oVirt Jenkins CI Server
___
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches


Change in vdsm[master]: Restore previous MTU on bond when VLAN with bigger MTU remov...

2012-08-23 Thread danken
Dan Kenigsberg has submitted this change and it was merged.

Change subject: Restore previous MTU on bond when VLAN with bigger MTU removed
..


Restore previous MTU on bond when VLAN with bigger MTU removed

Change-Id: Ifefa5dec03be7795e528821a4d17991eb0f60aea
Signed-off-by: Igor Lvovsky ilvov...@redhat.com
---
M vdsm/configNetwork.py
1 file changed, 2 insertions(+), 0 deletions(-)

Approvals:
  Dan Kenigsberg: Verified; Looks good to me, approved


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

Gerrit-MessageType: merged
Gerrit-Change-Id: Ifefa5dec03be7795e528821a4d17991eb0f60aea
Gerrit-PatchSet: 4
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Igor Lvovsky ilvov...@redhat.com
Gerrit-Reviewer: Dan Kenigsberg dan...@redhat.com
Gerrit-Reviewer: Igor Lvovsky ilvov...@redhat.com
Gerrit-Reviewer: Livnat Peer lp...@redhat.com
Gerrit-Reviewer: oVirt Jenkins CI Server
___
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches


Change in vdsm[master]: Restore previous MTU on bond when VLAN with bigger MTU remov...

2012-08-22 Thread Gerrit Code Review
oVirt Jenkins CI Server has posted comments on this change.

Change subject: Restore previous MTU on bond when VLAN with bigger MTU removed
..


Patch Set 4:

Build Successful 

http://jenkins.ovirt.info/job/patch_vdsm_unit_tests/580/ : SUCCESS

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

Gerrit-MessageType: comment
Gerrit-Change-Id: Ifefa5dec03be7795e528821a4d17991eb0f60aea
Gerrit-PatchSet: 4
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Igor Lvovsky ilvov...@redhat.com
Gerrit-Reviewer: Dan Kenigsberg dan...@redhat.com
Gerrit-Reviewer: Igor Lvovsky ilvov...@redhat.com
Gerrit-Reviewer: Livnat Peer lp...@redhat.com
Gerrit-Reviewer: oVirt Jenkins CI Server
___
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches


Change in vdsm[master]: Restore previous MTU on bond when VLAN with bigger MTU remov...

2012-08-20 Thread danken
Dan Kenigsberg has posted comments on this change.

Change subject: Restore previous MTU on bond when VLAN with bigger MTU removed
..


Patch Set 1: Looks good to me, approved

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

Gerrit-MessageType: comment
Gerrit-Change-Id: Ifefa5dec03be7795e528821a4d17991eb0f60aea
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Igor Lvovsky ilvov...@redhat.com
Gerrit-Reviewer: Dan Kenigsberg dan...@redhat.com
Gerrit-Reviewer: Livnat Peer lp...@redhat.com
Gerrit-Reviewer: oVirt Jenkins CI Server
___
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches


Change in vdsm[master]: Restore previous MTU on bond when VLAN with bigger MTU remov...

2012-08-20 Thread Gerrit Code Review
oVirt Jenkins CI Server has posted comments on this change.

Change subject: Restore previous MTU on bond when VLAN with bigger MTU removed
..


Patch Set 2:

Build Successful 

http://jenkins.ovirt.info/job/patch_vdsm_unit_tests/547/ : SUCCESS

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

Gerrit-MessageType: comment
Gerrit-Change-Id: Ifefa5dec03be7795e528821a4d17991eb0f60aea
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Igor Lvovsky ilvov...@redhat.com
Gerrit-Reviewer: Dan Kenigsberg dan...@redhat.com
Gerrit-Reviewer: Igor Lvovsky ilvov...@redhat.com
Gerrit-Reviewer: Livnat Peer lp...@redhat.com
Gerrit-Reviewer: oVirt Jenkins CI Server
___
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches


Change in vdsm[master]: Restore previous MTU on bond when VLAN with bigger MTU remov...

2012-08-20 Thread Gerrit Code Review
oVirt Jenkins CI Server has posted comments on this change.

Change subject: Restore previous MTU on bond when VLAN with bigger MTU removed
..


Patch Set 3:

Build Successful 

http://jenkins.ovirt.info/job/patch_vdsm_unit_tests/550/ : SUCCESS

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

Gerrit-MessageType: comment
Gerrit-Change-Id: Ifefa5dec03be7795e528821a4d17991eb0f60aea
Gerrit-PatchSet: 3
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Igor Lvovsky ilvov...@redhat.com
Gerrit-Reviewer: Dan Kenigsberg dan...@redhat.com
Gerrit-Reviewer: Igor Lvovsky ilvov...@redhat.com
Gerrit-Reviewer: Livnat Peer lp...@redhat.com
Gerrit-Reviewer: oVirt Jenkins CI Server
___
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches


Change in vdsm[master]: Restore previous MTU on bond when VLAN with bigger MTU remov...

2012-08-19 Thread ilvovsky
Igor Lvovsky has uploaded a new change for review.

Change subject: Restore previous MTU on bond when VLAN with bigger MTU removed
..

Restore previous MTU on bond when VLAN with bigger MTU removed

Change-Id: Ifefa5dec03be7795e528821a4d17991eb0f60aea
Signed-off-by: Igor Lvovsky ilvov...@redhat.com
---
M vdsm/configNetwork.py
1 file changed, 2 insertions(+), 0 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/23/7323/1

diff --git a/vdsm/configNetwork.py b/vdsm/configNetwork.py
index 199adfa..14b9cfb 100755
--- a/vdsm/configNetwork.py
+++ b/vdsm/configNetwork.py
@@ -609,6 +609,8 @@
 
 if newmtu != currmtu:
 if bonding:
+cf = self.NET_CONF_PREF + bonding
+self._updateConfigValue(cf, 'MTU', newmtu, newmtu is None)
 slaves = netinfo.slaves(bonding)
 for slave in slaves:
 cf = self.NET_CONF_PREF + slave


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

Gerrit-MessageType: newchange
Gerrit-Change-Id: Ifefa5dec03be7795e528821a4d17991eb0f60aea
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Igor Lvovsky ilvov...@redhat.com
___
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches


Change in vdsm[master]: Restore previous MTU on bond when VLAN with bigger MTU remov...

2012-08-19 Thread Gerrit Code Review
oVirt Jenkins CI Server has posted comments on this change.

Change subject: Restore previous MTU on bond when VLAN with bigger MTU removed
..


Patch Set 1:

Build Successful 

http://jenkins.ovirt.info/job/patch_vdsm_unit_tests/523/ : SUCCESS

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

Gerrit-MessageType: comment
Gerrit-Change-Id: Ifefa5dec03be7795e528821a4d17991eb0f60aea
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Igor Lvovsky ilvov...@redhat.com
Gerrit-Reviewer: Dan Kenigsberg dan...@redhat.com
Gerrit-Reviewer: Livnat Peer lp...@redhat.com
Gerrit-Reviewer: oVirt Jenkins CI Server
___
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches