ofri masad has posted comments on this change.
Change subject: webadmin: Changed units of Network QoS in UI
......................................................................
Patch Set 1:
(5 comments)
....................................................
Commit Message
Line 6:
Line 7: webadmin: Changed units of Network QoS in UI
Line 8:
Line 9: Network QoS 'Burst' property is commonly counted in Megabytes and not
Line 10: Megabits. vdsm API also recives this vlaue in Megabytes.
Done
Line 11:
Line 12: UI was changed (label) to support this convention and tooltip added to
Line 13: point the difference between 'Burst' and 'Average' and 'Peak' fields.
Line 14:
....................................................
File
frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/ApplicationConstants.java
Line 3042:
Line 3043: @DefaultStringValue("Burst")
Line 3044: String burstNetworkQoSPopup();
Line 3045:
Line 3046: @DefaultStringValue("(Mega-bits per second)")
Done
Line 3047: String inMegabitsNetworkQoSPopup();
Line 3048:
Line 3049: @DefaultStringValue("(Mega-Bytes)")
Line 3050: String inMegabytesNetworkQoSPopup();
Line 3045:
Line 3046: @DefaultStringValue("(Mega-bits per second)")
Line 3047: String inMegabitsNetworkQoSPopup();
Line 3048:
Line 3049: @DefaultStringValue("(Mega-Bytes)")
Done
Line 3050: String inMegabytesNetworkQoSPopup();
Line 3051:
Line 3052: @DefaultStringValue("Inbound")
Line 3053: String inboundLabelQoSPopup();
Line 3057:
Line 3058: @DefaultStringValue("Mbps")
Line 3059: String mbpsLabelQoSPopup();
Line 3060:
Line 3061: @DefaultStringValue("MiB")
just like Lior said. the patch changes from Megabits to Megabytes.
Lior, I don't think they make that distinction between MiB and MB.
Line 3062: String mbLabelQoSPopup();
Line 3063:
Line 3064: @DefaultStringValue("Cluster Policies")
Line 3065: String configureClusterPolicyTabLabel();
....................................................
File
frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/networkQoS/NetworkQoSPopupView.java
Line 140: inboundPeakEditor.setTitle(constants.peakNetworkQoSPopup() +
constants.inMegabitsNetworkQoSPopup());
Line 141: inboundBurstEditor.setTitle(constants.burstNetworkQoSPopup()
+ constants.inMegabytesNetworkQoSPopup());
Line 142:
outboundAverageEditor.setTitle(constants.averageNetworkQoSPopup() +
constants.inMegabitsNetworkQoSPopup());
Line 143: outboundPeakEditor.setTitle(constants.peakNetworkQoSPopup() +
constants.inMegabitsNetworkQoSPopup());
Line 144: outboundBurstEditor.setTitle(constants.burstNetworkQoSPopup()
+ constants.inMegabytesNetworkQoSPopup());
Done
Line 145: }
Line 146:
Line 147: @Override
Line 148: public void edit(NetworkQoSModel object) {
--
To view, visit http://gerrit.ovirt.org/18395
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I1dd2367b136d547f7f93fdc8a1b76561920ce59a
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: ofri masad <[email protected]>
Gerrit-Reviewer: Alona Kaplan <[email protected]>
Gerrit-Reviewer: Gilad Chaplik <[email protected]>
Gerrit-Reviewer: Lior Vernia <[email protected]>
Gerrit-Reviewer: Livnat Peer <[email protected]>
Gerrit-Reviewer: Moti Asayag <[email protected]>
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-Reviewer: ofri masad <[email protected]>
Gerrit-HasComments: Yes
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches
