Michael Kublin has submitted this change and it was merged. Change subject: core: initVdsOnUp- remove reconstruct and host status. ...................................................................... core: initVdsOnUp- remove reconstruct and host status. *Host that runs through init vds on up shouldn't attempt to reconstruct. *In case of failure during ConnectStoragePool, the host should fail in initalizeStorage() only if the master domain isn't in inactive/ unknown status and the exception wasn't XmlRpcRunTimeException (which is represented by VdcBllErrors.VDS_NETWORK_ERROR. Change-Id: I6418b9d2826146e1e4ceff4341c6d7cd3a0024af Signed-off-by: Liron Aravot <laravot@redhat.com> --- M backend/manager/dbscripts/storages_sp.sql M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/InitVdsOnUpCommand.java M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/StorageDomainDAO.java M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/StorageDomainDAODbFacadeImpl.java M backend/manager/modules/dal/src/test/java/org/ovirt/engine/core/dao/StorageDomainDAOTest.java 5 files changed, 63 insertions(+), 24 deletions(-) Approvals: Michael Kublin: Verified; Looks good to me, approved Liron Ar: Verified -- To view, visit http://gerrit.ovirt.org/13709 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I6418b9d2826146e1e4ceff4341c6d7cd3a0024af Gerrit-PatchSet: 10 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Liron Ar <laravot@redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini@redhat.com> Gerrit-Reviewer: Ayal Baron <abaron@redhat.com> Gerrit-Reviewer: Barak Azulay <bazulay@redhat.com> Gerrit-Reviewer: Liron Ar <laravot@redhat.com> Gerrit-Reviewer: Michael Kublin <mkublin@redhat.com>