Univention Bugzilla – Full Text Bug Listing |
Summary: | uldap.py: Try again when the LDAP server is not reachable | ||
---|---|---|---|
Product: | UCS | Reporter: | Stefan Gohmann <gohmann> |
Component: | LDAP | Assignee: | Stefan Gohmann <gohmann> |
Status: | CLOSED FIXED | QA Contact: | Janek Walkenhorst <walkenhorst> |
Severity: | normal | ||
Priority: | P5 | CC: | jmm, schwardt |
Version: | UCS 3.2 | ||
Target Milestone: | UCS 3.2-1-errata | ||
Hardware: | Other | ||
OS: | Linux | ||
See Also: | https://forge.univention.org/bugzilla/show_bug.cgi?id=35741 | ||
What kind of report is it?: | --- | What type of bug is this?: | --- |
Who will be affected by this bug?: | --- | How will those affected feel about the bug?: | --- |
User Pain: | Enterprise Customer affected?: | ||
School Customer affected?: | ISV affected?: | ||
Waiting Support: | Flags outvoted (downgraded) after PO Review: | ||
Ticket number: | Bug group (optional): | ||
Max CVSS v3 score: | |||
Bug Depends on: | |||
Bug Blocks: | 34293, 35852 |
Description
Stefan Gohmann
2014-03-10 09:23:25 CET
> Defaults:
> ldap/client/retry=true
> ldap/client/retry/count=5
> ldap/client/retry/count=1
^^^^^ ==> wait
I doubt that the slapd is ready within 5 seconds after initiating a restart.
I would suggest to wait about 30 seconds. What about using an exponentially increasing timeout with given minimum (e.g. 1) and maximum value (e.g. 5)?
(In reply to Sönke Schwardt-Krummrich from comment #1) > > Defaults: > > ldap/client/retry=true > > ldap/client/retry/count=5 > > ldap/client/retry/count=1 > ^^^^^ ==> wait > > I doubt that the slapd is ready within 5 seconds after initiating a restart. > I would suggest to wait about 30 seconds. What about using an exponentially > increasing timeout with given minimum (e.g. 1) and maximum value (e.g. 5)? In my tests it took round about one or two seconds. But you are right we should use a higher value. I would suggest we use only one variable for the count and wait for always one second. I think it is much easier: ldap/client/retry/count?30 First code was added to univention.uldap (r48474). The following things are still missing: - YAML - Cleanup - UCR description - Tests (In reply to Stefan Gohmann from comment #3) > First code was added to univention.uldap (r48474). The following things are > still missing: > - YAML > - Cleanup > - UCR description > - Tests The name of the variable is now ldap/client/retry/count and it is set to 10 by default. This means uldap tries it again for 10 times. This value is also given to the LDAPReconnectObject instance. The UCR description was added to univention-base-files. Code fixes: - univention-python: r48474, r48498, r48520 - univention-base-files: r48479, r48519, r48539 YAML: - univention-python: r48540 - univention-base-files: r48540 ucs-test: - r48477, r48478, r48487, r48507 UMC still works when slapd runs only intermittently. ldap/client/retry/count works as advertised. Advisories: OK |