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 <[email protected]>
---
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 <[email protected]>
Gerrit-Reviewer: Alissa Bonas <[email protected]>
Gerrit-Reviewer: Allon Mureinik <[email protected]>
Gerrit-Reviewer: Daniel Erez <[email protected]>
Gerrit-Reviewer: Liron Ar <[email protected]>
Gerrit-Reviewer: Tal Nisan <[email protected]>
Gerrit-Reviewer: oVirt Jenkins CI Server
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to