Change in ovirt-engine[master]: core: setting proper timeout mechanism for GetRootDSE (#8447...
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(a)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(a)redhat.com>
Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com>
Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com>
Gerrit-Reviewer: Yaniv Bronhaim <ybronhei(a)redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server