Change in ovirt-engine[master]: core: update storage connection without a domain

[amureini at redhat.com]

Allon Mureinik has submitted this change and it was merged.

Change subject: core: update storage connection without a domain
......................................................................


core: update storage connection without a domain

Allow updating a storage connection regardless whether
there are storage domains using it or not. Add relevant unitests.

Till now it was allowed to update a connection only
if there were storage domains using it. As part of the agenda
to allow users to manage the storage connections independently,
removing this restriction is necessary,
since connections can be added and edited before a storage domain
started to use it. This will be possible via REST, while in
webadmin UI storage domain and the connection are still presented
and managed as one entity.

Change-Id: I4b72f48d6ae47e3d6d2a50ce0b51693d22840161
Signed-off-by: Alissa Bonas <abonas at redhat.com>
---
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/UpdateStorageServerConnectionCommand.java
M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/storage/UpdateStorageServerConnectionCommandTest.java
2 files changed, 69 insertions(+), 39 deletions(-)

Approvals:
  Allon Mureinik: Verified; Looks good to me, approved


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

Gerrit-MessageType: merged
Gerrit-Change-Id: I4b72f48d6ae47e3d6d2a50ce0b51693d22840161
Gerrit-PatchSet: 4
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Alissa Bonas <abonas at redhat.com>
Gerrit-Reviewer: Alissa Bonas <abonas at redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini at redhat.com>
Gerrit-Reviewer: Daniel Erez <derez at redhat.com>
Gerrit-Reviewer: Liron Ar <laravot at redhat.com>
Gerrit-Reviewer: Tal Nisan <tnisan at redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server