Tal Nisan has uploaded a new change for review. Change subject: core: Clear current SPM upon "Not SPM" error ......................................................................
core: Clear current SPM upon "Not SPM" error Clear the current SPM when we get a "Not SPM" error from the current SPM, this is needed when we have a host which is not an SPM anymore (Cause of being fenced for instance) which still gets SPM commands sent to it and returns "Not SPM", so instead waiting for the next SPM status check, we immediately remove it and start spmStart process to find an SPM host Change-Id: Id0c9aa8c11ba115580d64c2c64873e059c986bbc Signed-off-by: Tal Nisan <[email protected]> --- M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/irsbroker/IrsBrokerCommand.java 1 file changed, 1 insertion(+), 1 deletion(-) git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/27/7727/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 5e28921..9a9e86e 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 @@ -1593,7 +1593,7 @@ getVDSReturnValue().setExceptionObject(ex); getVDSReturnValue().setVdsError(ex.getVdsError()); logException(ex); - if (VdcBllErrors.SpmStatusError.equals(ex.getVdsError())) { + if (ex.getVdsError() != null && VdcBllErrors.SpmStatusError.equals(ex.getVdsError().getCode())) { getCurrentIrsProxyData().setCurrentVdsId(Guid.Empty); } failover(); -- To view, visit http://gerrit.ovirt.org/7727 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: newchange Gerrit-Change-Id: Id0c9aa8c11ba115580d64c2c64873e059c986bbc Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Tal Nisan <[email protected]> _______________________________________________ Engine-patches mailing list [email protected] http://lists.ovirt.org/mailman/listinfo/engine-patches
