Change in ovirt-engine[master]: engine: Removing canDoaction from ConnectHostToStoragePoolSe...

[mkublin at redhat.com]

Michael Kublin has submitted this change and it was merged.

Change subject: engine: Removing canDoaction from ConnectHostToStoragePoolServersCommand
......................................................................


engine: Removing canDoaction from ConnectHostToStoragePoolServersCommand

The following change is removes canDoAction from ConnectHostToStoragePoolServersCommand.
The purpose of canDoAction was to validate storage connections at VDSM, the following API
is depreacted and not useful at vdsm side, and doesn't provide any benefit to engine, also
the following command is internal so no reason to perfrom additional xml rpc call in order to
validate storage server connection.
Benefit, reduced number of XML RPC calls via InitVdsOnUp, meaning vds should be moved to status UP
much faster.
Additional benefit, less code, less bugs

Change-Id: I0124e47b2f1194e5de9b40216cd285c47b1b6f94
Signed-off-by: Michael Kublin <mkublin at redhat.com>
---
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/ConnectHostToStoragePoolServersCommand.java
1 file changed, 8 insertions(+), 42 deletions(-)

Approvals:
  Michael Kublin: Verified; Looks good to me, approved


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

Gerrit-MessageType: merged
Gerrit-Change-Id: I0124e47b2f1194e5de9b40216cd285c47b1b6f94
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Michael Kublin <mkublin at redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini at redhat.com>
Gerrit-Reviewer: Barak Azulay <bazulay at redhat.com>
Gerrit-Reviewer: Eli Mesika <emesika at redhat.com>
Gerrit-Reviewer: Michael Kublin <mkublin at redhat.com>
Gerrit-Reviewer: Yair Zaslavsky <yzaslavs at redhat.com>