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@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@redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali@redhat.com> Gerrit-Reviewer: Yair Zaslavsky <yzaslavs@redhat.com> Gerrit-Reviewer: Yaniv Bronhaim <ybronhei@redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server
participants (1)
-
oourfali@redhat.com