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 <[email protected]> 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 <[email protected]> Gerrit-Reviewer: Allon Mureinik <[email protected]> Gerrit-Reviewer: Ayal Baron <[email protected]> Gerrit-Reviewer: Daniel Erez <[email protected]> Gerrit-Reviewer: Liron Aravot <[email protected]> Gerrit-Reviewer: Maor Lipchuk <[email protected]> Gerrit-Reviewer: Omer Frenkel <[email protected]> Gerrit-Reviewer: Tal Nisan <[email protected]> _______________________________________________ Engine-patches mailing list [email protected] http://lists.ovirt.org/mailman/listinfo/engine-patches
