Change in ovirt-engine[master]: core: update storage connection without a domain
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(a)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(a)redhat.com>
Gerrit-Reviewer: Alissa Bonas <abonas(a)redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com>
Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com>
Gerrit-Reviewer: Liron Ar <laravot(a)redhat.com>
Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server