Yair Zaslavsky has posted comments on this change.
Change subject: Modify error to info report after fencing and fixing print
......................................................................
Patch Set 3: (1 inline comment)
sorry, was easier if i had provided the exact text.
Hope it will be last round ;)
....................................................
File
backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/VdsManager.java
Line 633: }
Line 634: setStatus(VDSStatus.NonResponsive, vds);
Line 635: log.infoFormat(
Line 636: "Server failed to respond, vds_id = {0}, vds_name
= {1}, vm_count = {2}, " +
Line 637: "spm_status = {3}, non-responsive_seconds = {4},
error = {5}",
i would suggest non-responsive_timeout (seconds)
Line 638: vds.getId(), vds.getName(), vds.getVmCount(),
vds.getSpmStatus(),
Line 639: TimeUnit.MILLISECONDS.toSeconds(timeoutToFence),
ex.getMessage());
Line 640:
Line 641: AuditLogableBase logable = new
AuditLogableBase(vds.getId());
--
To view, visit http://gerrit.ovirt.org/14495
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I9411c286b20f6e0627d5699ca8ea294e443cf6ae
Gerrit-PatchSet: 3
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Yaniv Bronhaim <[email protected]>
Gerrit-Reviewer: Barak Azulay <[email protected]>
Gerrit-Reviewer: Yair Zaslavsky <[email protected]>
Gerrit-Reviewer: Yaniv Bronhaim <[email protected]>
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches
