Doron Fediuck has submitted this change and it was merged. Change subject: core: Make connection timeout configurable ...................................................................... core: Make connection timeout configurable The timeout that we currently use to connect to hosts is by default very large, as it is the same we use for VDSM operations. The default is 180 seconds but waiting that long for connections is not very reasonable, as establishing a connection is a quick operation, specially in a high speed local network. This large timeout severely affects the high availability support, as it means that we need to wait at least 6 minutes (we need at least 2 failures in a row) when a host fails and starts dropping packets before we can declare it failed and start the recovery process. This patch introduces two new parameters that control the time that it takes to react when there are connection issues: 1. vdsConnectionTimeout (default is 3 minutes) Timeout for establishment of connections with hosts. This should be quite small, a few seconds at most, as it the TCP handshake with hosts should be very quick in most networks. At the moment we are using 3 minutes in order to preserve the behaviour before this modification, but this will probably be changed to a much smaller value (2 seconds, for example) in the future. 2. vdsRetries (default is 3) The number of times to retry host operations when there are IO errors. The default value is 3 to match the default value of the XML RPC library that we use, so there is no change in behaviour. Bug-Url: https://bugzilla.redhat.com/863211 Change-Id: I55a49ae5d655c2105c5840decec81ae712e40c32 Signed-off-by: Juan Hernandez <juan.hernandez@redhat.com> --- M backend/manager/dbscripts/upgrade/pre_upgrade/0000_config.sql M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/config/ConfigValues.java M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/VdsManager.java M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/irsbroker/IrsBrokerCommand.java M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/xmlrpc/XmlRpcUtils.java M backend/manager/tools/src/main/conf/engine-config.properties 6 files changed, 72 insertions(+), 5 deletions(-) Approvals: Doron Fediuck: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/8410 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I55a49ae5d655c2105c5840decec81ae712e40c32 Gerrit-PatchSet: 11 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Juan Hernandez <juan.hernandez@redhat.com> Gerrit-Reviewer: Dan Kenigsberg <danken@redhat.com> Gerrit-Reviewer: Doron Fediuck <dfediuck@redhat.com> Gerrit-Reviewer: Juan Hernandez <juan.hernandez@redhat.com> Gerrit-Reviewer: Laszlo Hornyak <lhornyak@redhat.com> Gerrit-Reviewer: Michael Kublin <mkublin@redhat.com> Gerrit-Reviewer: Noam Slomianko <nslomian@redhat.com> Gerrit-Reviewer: Omer Frenkel <ofrenkel@redhat.com> Gerrit-Reviewer: Yair Zaslavsky <yzaslavs@redhat.com> Gerrit-Reviewer: Yaniv Kaul <ykaul@redhat.com>