Yaniv Bronhaim has uploaded a new change for review. Change subject: Adding fencing parameters to log print ......................................................................
Adding fencing parameters to log print Change-Id: Ic14f3ae9e302ffaf8e4fc883f2a177864e512c4f Signed-off-by: Yaniv Bronhaim <[email protected]> --- M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/VdsManager.java 1 file changed, 6 insertions(+), 4 deletions(-) git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/36/14436/1 diff --git a/backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/VdsManager.java b/backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/VdsManager.java index fe594dd..55cec74 100644 --- a/backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/VdsManager.java +++ b/backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/VdsManager.java @@ -614,9 +614,9 @@ */ public boolean handleNetworkException(VDSNetworkException ex, VDS vds) { if (vds.getStatus() != VDSStatus.Down) { + long timeoutToFence = calcTimeoutToFence(vds.getVmCount(), vds.getSpmStatus()); if (mUnrespondedAttempts.get() < Config.<Integer> GetValue(ConfigValues.VDSAttemptsToResetCount) - || (lastUpdate - + calcTimeoutToFence(vds.getVmCount(), vds.getSpmStatus())) > System.currentTimeMillis()) { + || (lastUpdate + timeoutToFence) > System.currentTimeMillis()) { boolean result = false; if (vds.getStatus() != VDSStatus.Connecting && vds.getStatus() != VDSStatus.PreparingForMaintenance && vds.getStatus() != VDSStatus.NonResponsive) { @@ -633,8 +633,10 @@ } setStatus(VDSStatus.NonResponsive, vds); log.errorFormat( - "Server failed to respond, vds_id = {0}, vds_name = {1}, error = {2}", - vds.getId(), vds.getName(), ex.getMessage()); + "Server failed to respond, vds_id = {0}, vds_name = {1}, vm_count = {2}, " + + "spm_status = {3}, non-responsive_timeout = {5} error = {4}", + vds.getId(), vds.getName(), vds.getVmCount(), vds.getSpmStatus(), timeoutToFence, + ex.getMessage()); AuditLogableBase logable = new AuditLogableBase(vds.getId()); AuditLogDirector.log(logable, AuditLogType.VDS_FAILURE); -- To view, visit http://gerrit.ovirt.org/14436 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: newchange Gerrit-Change-Id: Ic14f3ae9e302ffaf8e4fc883f2a177864e512c4f Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Yaniv Bronhaim <[email protected]> _______________________________________________ Engine-patches mailing list [email protected] http://lists.ovirt.org/mailman/listinfo/engine-patches
