Change in ovirt-engine[master]: core: Don't validate MTU on host NIC

mkolesni at redhat.com mkolesni at redhat.com
Wed Jun 13 15:46:35 UTC 2012


Mike Kolesnik has submitted this change and it was merged.

Change subject: core: Don't validate MTU on host NIC
......................................................................


core: Don't validate MTU on host NIC

No need to validate the MTU field of VdsNetworkInterface since it is
not used in network commands (what is used is the MTU on the network).
A test needed to be fixed since it relied on this field (being a unit
test, it shouldn't have in the first place).

Change-Id: I6f5734480acdbe5603ab6b84a358479903ea5dfc
Signed-off-by: Mike Kolesnik <mkolesni at redhat.com>
---
M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/VdsNetworkInterface.java
M backend/manager/modules/common/src/test/java/org/ovirt/engine/core/common/utils/MTUValidatorTest.java
2 files changed, 16 insertions(+), 15 deletions(-)

Approvals:
  Mike Kolesnik: Verified; Looks good to me, approved


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

Gerrit-MessageType: merged
Gerrit-Change-Id: I6f5734480acdbe5603ab6b84a358479903ea5dfc
Gerrit-PatchSet: 3
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Mike Kolesnik <mkolesni at redhat.com>
Gerrit-Reviewer: Livnat Peer <lpeer at redhat.com>
Gerrit-Reviewer: Mike Kolesnik <mkolesni at redhat.com>



More information about the Engine-commits mailing list