Change in ovirt-engine[master]: core: setting proper timeout mechanism for GetRootDSE (#8447...

[oourfali at redhat.com]

Oved Ourfali has submitted this change and it was merged.

Change subject: core: setting proper timeout mechanism for GetRootDSE (#844733)
......................................................................


core: setting proper timeout mechanism for GetRootDSE (#844733)

https://bugzilla.redhat.com/844733

This is a first patch in series of patches that deals with
a problem that search users reaches a timeout.
In this patch, a proper timeout mechanism is used.
Although it was assumed that changing LdapQueryTimeout
to a low value (i.e - 3 seconds) will yield a timeout after
the provided amount of time, in fact it took longer for
GetRootDSE to timeout.
This patch fixes this issue.

Change-Id: I59e875997b904cbdc058a45efd640f5bf1fc6579
Signed-off-by: Yair Zaslavsky <yzaslavs at redhat.com>
---
M backend/manager/dbscripts/upgrade/pre_upgrade/0000_config.sql
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/adbroker/GetRootDSE.java
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/adbroker/LDAPTemplateWrapper.java
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/adbroker/LdapBrokerUtils.java
M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/adbroker/DirectorySearcherTest.java
M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/config/ConfigValues.java
M backend/manager/tools/engine-config/src/main/resources/engine-config.properties
7 files changed, 37 insertions(+), 4 deletions(-)

Approvals:
  Oved Ourfali: Verified; Looks good to me, approved


--
To view, visit http://gerrit.ovirt.org/7479
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I59e875997b904cbdc058a45efd640f5bf1fc6579
Gerrit-PatchSet: 8
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Yair Zaslavsky <yzaslavs at redhat.com>
Gerrit-Reviewer: Oved Ourfali <oourfali at redhat.com>
Gerrit-Reviewer: Yair Zaslavsky <yzaslavs at redhat.com>
Gerrit-Reviewer: Yaniv Bronhaim <ybronhei at redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server