Tal Nisan has submitted this change and it was merged.
Change subject: core: Add a CDA on attach to an uninitialized Data Center
......................................................................
core: Add a CDA on attach to an uninitialized Data Center
Adding a validation message when trying to attach a Storage Domain which
already have a storage pool id initiliazed in its meta data.
Storage Domain which have storage pool id already initialized in its
meta data (For example after disaster), needs to be detached first and
then to be attached to the relevant Data Center, since the detach
operation is an SPM operation, an un-initialized Data Center can't
perform this operation since it does not have an SPM yet.
Change-Id: I13ec66cc7ff4773221cc994cb4f0be1886ed9a6b
Bug-Url:
https://bugzilla.redhat.com/1176552
Signed-off-by: Maor Lipchuk <mlipchuk(a)redhat.com>
---
M
backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/AddStoragePoolWithStoragesCommand.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/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
6 files changed, 44 insertions(+), 5 deletions(-)
Approvals:
Tal Nisan: Verified; Looks good to me, approved
--
To view, visit
http://gerrit.ovirt.org/36546
To unsubscribe, visit
http://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I13ec66cc7ff4773221cc994cb4f0be1886ed9a6b
Gerrit-PatchSet: 6
Gerrit-Project: ovirt-engine
Gerrit-Branch: ovirt-engine-3.5
Gerrit-Owner: Maor Lipchuk <mlipchuk(a)redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com>
Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com>
Gerrit-Reviewer: Maor Lipchuk <mlipchuk(a)redhat.com>
Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server