Change in ovirt-engine[master]: core:Initialize host name for CDA message(BZ846771)
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(a)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(a)redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com>
Gerrit-Reviewer: Ayal Baron <abaron(a)redhat.com>
Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com>
Gerrit-Reviewer: Liron Aravot <laravot(a)redhat.com>
Gerrit-Reviewer: Maor Lipchuk <mlipchuk(a)redhat.com>
Gerrit-Reviewer: Omer Frenkel <ofrenkel(a)redhat.com>
Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com>