Lior Vernia has uploaded a new change for review. Change subject: webadmin: Allow setting MTU > 9000 ......................................................................
webadmin: Allow setting MTU > 9000 Fetch the max MTU value from the database (as is done in the backend) instead of hard-coding it to be 9000. Change-Id: Ie427c578d79df6c37faac4ef80e697566b8e9fb3 Bug-Url: https://bugzilla.redhat.com/1010663 Signed-off-by: Lior Vernia <[email protected]> --- M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/queries/ConfigurationValues.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/datacenters/NetworkModel.java 2 files changed, 4 insertions(+), 1 deletion(-) git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/29/23429/1 diff --git a/backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/queries/ConfigurationValues.java b/backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/queries/ConfigurationValues.java index 50244f6..8eba34f 100644 --- a/backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/queries/ConfigurationValues.java +++ b/backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/queries/ConfigurationValues.java @@ -69,6 +69,7 @@ HotPlugEnabled(ConfigAuthType.User), NetworkLinkingSupported(ConfigAuthType.User), SupportBridgesReportByVDSM(ConfigAuthType.User), + MaxMTU, ManagementNetwork, ApplicationMode(ConfigAuthType.User), ShareableDiskEnabled(ConfigAuthType.User), diff --git a/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/datacenters/NetworkModel.java b/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/datacenters/NetworkModel.java index c0cdaea..905d52e 100644 --- a/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/datacenters/NetworkModel.java +++ b/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/datacenters/NetworkModel.java @@ -46,6 +46,8 @@ public abstract class NetworkModel extends Model { + private static final Integer MAX_MTU = + (Integer) AsyncDataProvider.getConfigValuePreConverted(ConfigurationValues.MaxMTU); protected static final String ENGINE_NETWORK = (String) AsyncDataProvider.getConfigValuePreConverted(ConfigurationValues.ManagementNetwork); @@ -458,7 +460,7 @@ { IntegerValidation tempVar5 = new IntegerValidation(); tempVar5.setMinimum(68); - tempVar5.setMaximum(9000); + tempVar5.setMaximum(MAX_MTU); getMtu().validateEntity(new IValidation[] { new NotEmptyValidation(), tempVar5 }); } -- To view, visit http://gerrit.ovirt.org/23429 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: newchange Gerrit-Change-Id: Ie427c578d79df6c37faac4ef80e697566b8e9fb3 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Lior Vernia <[email protected]> _______________________________________________ Engine-patches mailing list [email protected] http://lists.ovirt.org/mailman/listinfo/engine-patches
