Change in ovirt-engine[master]: engine: Prevent networks with different MTU (#852052)

msalem at redhat.com msalem at redhat.com
Mon Sep 3 15:47:09 UTC 2012


Muli Salem has submitted this change and it was merged.

Change subject: engine: Prevent networks with different MTU (#852052)
......................................................................


engine: Prevent networks with different MTU (#852052)

https://bugzilla.redhat.com/852052

The patch prevents attaching a network to an interface if one of the
following cases occur:
* A non-VM network is already attached to the interface with MTU other
  than default and the user attempt to attach a Vlan which its MTU is
  other than the non-VM network.
* A Vlan with non-default MTU is attached to an interface and the user
  attempt to attach a non-VM network to it with MTU other than the Vlan.

Attaching several Vlans (VM network) with different MTU will not be blocked, as VDSM
should support MTU configuration for the underlying interface.

Change-Id: I69f5c5f515d782c0311fd7fe458408ed634c1c83
Signed-off-by: Moti Asayag <masayag at redhat.com>
---
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/SetupNetworksHelper.java
M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/SetupNetworksHelperTest.java
M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dal/VdcBllMessages.java
M backend/manager/modules/dal/src/main/resources/bundles/AppErrors.properties
M backend/manager/modules/utils/src/main/java/org/ovirt/engine/core/utils/NetworkUtils.java
M frontend/webadmin/modules/frontend/src/main/java/org/ovirt/engine/ui/frontend/AppErrors.java
M frontend/webadmin/modules/userportal-gwtp/src/main/resources/org/ovirt/engine/ui/frontend/AppErrors.properties
M frontend/webadmin/modules/webadmin/src/main/resources/org/ovirt/engine/ui/frontend/AppErrors.properties
8 files changed, 360 insertions(+), 0 deletions(-)

Approvals:
  Muli Salem: Verified; Looks good to me, approved
  Moti Asayag: Verified


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

Gerrit-MessageType: merged
Gerrit-Change-Id: I69f5c5f515d782c0311fd7fe458408ed634c1c83
Gerrit-PatchSet: 4
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Moti Asayag <masayag at redhat.com>
Gerrit-Reviewer: Livnat Peer <lpeer at redhat.com>
Gerrit-Reviewer: Mike Kolesnik <mkolesni at redhat.com>
Gerrit-Reviewer: Moti Asayag <masayag at redhat.com>
Gerrit-Reviewer: Muli Salem <msalem at redhat.com>



More information about the Engine-commits mailing list