Lior Vernia has submitted this change and it was merged.
Change subject: webadmin: Lift max MTU limit
......................................................................
webadmin: Lift max MTU limit
Don't limit users to a maximum MTU value when configuring a network,
values too high will cause errors when provisioned on a non-supporting
host. Also removed the reference to the MaxMTU configuration value
which had never been exposed to users anyway.
Change-Id: I352ac3e00d657f00b6cec5dcb23475a143a9c8fd
Bug-Url:
https://bugzilla.redhat.com/1010663
Signed-off-by: Lior Vernia <lvernia(a)redhat.com>
---
M
backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/config/ConfigValues.java
M
backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/queries/ConfigurationValues.java
M
backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/validation/MTUValidator.java
M
backend/manager/modules/common/src/test/java/org/ovirt/engine/core/common/utils/MTUValidatorTest.java
M
frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/datacenters/NetworkModel.java
5 files changed, 1 insertion(+), 20 deletions(-)
Approvals:
Lior Vernia: Verified; Looks good to me, approved
--
To view, visit
http://gerrit.ovirt.org/23482
To unsubscribe, visit
http://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I352ac3e00d657f00b6cec5dcb23475a143a9c8fd
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Lior Vernia <lvernia(a)redhat.com>
Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com>
Gerrit-Reviewer: Lior Vernia <lvernia(a)redhat.com>
Gerrit-Reviewer: Mike Kolesnik <mkolesni(a)redhat.com>
Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com>