Change in ovirt-engine[master]: webadmin: Fixed error message in new Storage QoS.

gchaplik at redhat.com gchaplik at redhat.com
Thu Jan 15 09:17:54 UTC 2015


Gilad Chaplik has submitted this change and it was merged.

Change subject: webadmin: Fixed error message in new Storage QoS.
......................................................................


webadmin: Fixed error message in new Storage QoS.

When giving some natural number value to each textbox in the new
Stoage QoS, wrong error message appears that says missing values.

Now the error message says that a non-zero values was entered for total
along with a non-zero value for read/write (which is invalid).

Change-Id: I85412ffc86be8f903b9a3f07fbc3a10641be8e71
Bug-Url: https://bugzilla.redhat.com/1163805
Signed-off-by: Tomer Saban <tsaban at redhat.com>
---
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/validator/storage/StorageQosValidator.java
M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/errors/VdcBllMessages.java
M backend/manager/modules/dal/src/main/resources/bundles/AppErrors.properties
M frontend/webadmin/modules/frontend/src/main/java/org/ovirt/engine/ui/frontend/AppErrors.java
M frontend/webadmin/modules/webadmin/src/main/resources/org/ovirt/engine/ui/frontend/AppErrors.properties
5 files changed, 7 insertions(+), 1 deletion(-)

Approvals:
  Gilad Chaplik: Verified; Looks good to me, approved



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I85412ffc86be8f903b9a3f07fbc3a10641be8e71
Gerrit-PatchSet: 4
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Tomer Saban <tsaban at redhat.com>
Gerrit-Reviewer: Gilad Chaplik <gchaplik at redhat.com>
Gerrit-Reviewer: Jiří Moskovčák <jmoskovc at redhat.com>
Gerrit-Reviewer: Martin Sivák <msivak at redhat.com>
Gerrit-Reviewer: Roy Golan <rgolan at redhat.com>
Gerrit-Reviewer: Tomer Saban <tsaban at redhat.com>
Gerrit-Reviewer: automation at ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server



More information about the Engine-commits mailing list