Change in ovirt-engine[master]: core:Initialize host name for CDA message(BZ846771)

[mlipchuk at redhat.com]

Maor Lipchuk has submitted this change and it was merged.

Change subject: core:Initialize host name for CDA message(BZ846771)
......................................................................


core:Initialize host name for CDA message(BZ846771)

https://bugzilla.redhat.com/show_bug.cgi?id=846771

the following patch propose a fix for the scenario when lun disk is
being extended and the host gets network exception while trying to
connect the storage domain.
This cause the CDA message not to be set with the host id that was not
manage to connect to the domain.

The solution is to Wrap the VDS calls with try and catch so failedHost list
can be set with the problematic host before throwing an exception.

Signed-off-by: Maor Lipchuk <mlipchuk at redhat.com>
Change-Id: I3aa40d4671dc99c2126c131fda36c25d8ed4826e
---
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/ConnectAllHostsToLunCommand.java
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/ExtendSANStorageDomainCommand.java
2 files changed, 23 insertions(+), 10 deletions(-)

Approvals:
  Maor Lipchuk: Verified; Looks good to me, approved


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

Gerrit-MessageType: merged
Gerrit-Change-Id: I3aa40d4671dc99c2126c131fda36c25d8ed4826e
Gerrit-PatchSet: 3
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Maor Lipchuk <mlipchuk at redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini at redhat.com>
Gerrit-Reviewer: Ayal Baron <abaron at redhat.com>
Gerrit-Reviewer: Daniel Erez <derez at redhat.com>
Gerrit-Reviewer: Liron Aravot <laravot at redhat.com>
Gerrit-Reviewer: Maor Lipchuk <mlipchuk at redhat.com>
Gerrit-Reviewer: Omer Frenkel <ofrenkel at redhat.com>
Gerrit-Reviewer: Tal Nisan <tnisan at redhat.com>