Michael Kublin has uploaded a new change for review. Change subject: engine: Fixing SPM election in case that we have IrsNoMasterDomainException ......................................................................
engine: Fixing SPM election in case that we have IrsNoMasterDomainException The following patch should add a current host to tried ones, otherwise the host will not be added and in case of throwing IrsNoMasterDomainException we will never trigger a reconstruct and will continue to perfrom a spm election for next 5 minutes. The error is that list of tried ids is left empty Change-Id: I14af7f4459b54a0d74bece777d72da8b1a0e2b84 Signed-off-by: Michael Kublin <[email protected]> --- M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/irsbroker/IrsBrokerCommand.java 1 file changed, 1 insertion(+), 0 deletions(-) git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/03/10203/1 diff --git a/backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/irsbroker/IrsBrokerCommand.java b/backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/irsbroker/IrsBrokerCommand.java index c475ef4..8b9e741 100644 --- a/backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/irsbroker/IrsBrokerCommand.java +++ b/backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/irsbroker/IrsBrokerCommand.java @@ -594,6 +594,7 @@ // Stores origin host id in case and will be needed to disconnect from storage pool Guid selectedVdsId = selectedVds.getId(); Integer selectedVdsSpmId = selectedVds.getvds_spm_id(); + mTriedVdssList.add(selectedVdsId); VDSReturnValue returnValueFromVds = ResourceManager.getInstance().runVdsCommand( VDSCommandType.SpmStatus, -- To view, visit http://gerrit.ovirt.org/10203 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: newchange Gerrit-Change-Id: I14af7f4459b54a0d74bece777d72da8b1a0e2b84 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Michael Kublin <[email protected]> _______________________________________________ Engine-patches mailing list [email protected] http://lists.ovirt.org/mailman/listinfo/engine-patches
