Roy Golan has submitted this change and it was merged.
Change subject: core: Fix error message, adding vm with disk profile, wrong storage.
......................................................................
core: Fix error message, adding vm with disk profile, wrong storage.
1. Get the name of the storage domain provided in the request.
2. Add the id of the disk profile to the error message.
Disk profile may have same name but in different storages.
Change-Id: Id7e39081f2c409513d865473a19e9e8fea71aa4c
Bug-Url:
https://bugzilla.redhat.com/1178018
Signed-off-by: Dudi Maroshi <dudi(a)redhat.com>
---
M
backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/profiles/DiskProfileValidator.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/userportal-gwtp/src/main/resources/org/ovirt/engine/ui/frontend/AppErrors.properties
M
frontend/webadmin/modules/webadmin/src/main/resources/org/ovirt/engine/ui/frontend/AppErrors.properties
5 files changed, 15 insertions(+), 7 deletions(-)
Approvals:
Dudi Maroshi: Verified
Roy Golan: Looks good to me, approved; Passed CI tests
--
To view, visit
https://gerrit.ovirt.org/43043
To unsubscribe, visit
https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Id7e39081f2c409513d865473a19e9e8fea71aa4c
Gerrit-PatchSet: 3
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Dudi Maroshi <dudi(a)redhat.com>
Gerrit-Reviewer: Dudi Maroshi <dudi(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Roy Golan <rgolan(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org