Yair Zaslavsky has posted comments on this change.
Change subject: core: setting proper timeout mechanism for GetRootDSE (#844733)
......................................................................
Patch Set 4: (1 inline comment)
....................................................
File backend/manager/dbscripts/upgrade/pre_upgrade/0000_config.sql
Line 181: --Handling Keystore URL
Line 182: select fn_db_add_config_value('keystoreUrl','.keystore','general');
Line 183: select fn_db_add_config_value('LdapQueryPageSize','1000','general');
Line 184: select fn_db_add_config_value('LDAPQueryTimeout','30','general');
Line 185: select fn_db_add_config_value('LDAPConnectTimeout','5','general');
My bad.
The value in config table is seconds.
We set it to java API in miliseconds , by multiplying in a factor of 1000, as
you mentioned.
If 5 is a bit short, let's change it to 10, I think we should test this, and
see how it feels. 30 seconds are too much, IMHO
Line 186: select fn_db_add_config_value('LDAPOperationTimeout','30','general');
Line 187: --Handling LDAP Security Authentication Method
Line 188: select
fn_db_add_config_value('LDAPSecurityAuthentication','GSSAPI','general');
Line 189: select fn_db_add_config_value('LDAPServerPort','389','general');
--
To view, visit http://gerrit.ovirt.org/7479
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I59e875997b904cbdc058a45efd640f5bf1fc6579
Gerrit-PatchSet: 4
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Yair Zaslavsky <[email protected]>
Gerrit-Reviewer: Oved Ourfali <[email protected]>
Gerrit-Reviewer: Yair Zaslavsky <[email protected]>
Gerrit-Reviewer: Yaniv Bronhaim <[email protected]>
Gerrit-Reviewer: oVirt Jenkins CI Server
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches
