Tal Nisan 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/1220127
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:
Tal Nisan: Looks good to me, approved
Dudi Maroshi: Verified
Jenkins CI: Passed CI tests
--
To view, visit
https://gerrit.ovirt.org/43070
To unsubscribe, visit
https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Id7e39081f2c409513d865473a19e9e8fea71aa4c
Gerrit-PatchSet: 5
Gerrit-Project: ovirt-engine
Gerrit-Branch: ovirt-engine-3.5
Gerrit-Owner: Dudi Maroshi <dudi(a)redhat.com>
Gerrit-Reviewer: Dudi Maroshi <dudi(a)redhat.com>
Gerrit-Reviewer: Eyal Edri <eedri(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Roy Golan <rgolan(a)redhat.com>
Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org