Bug #6108: Charge data in cache when openldap restart - Rudder - Issue Tracker

Actions

Copy link

Bug #6108

closed

Bug #5965: LDAP configuration is not optimized for Rudder use case

Charge data in cache when openldap restart

Added by François ARMAND over 9 years ago. Updated about 9 years ago.

Status:

Released

Priority:

Assignee:

Matthieu CERDA

Category:

Performance and scalability

Target version:

2.11.7

Pull Request:

https://github.com/Normation/rudder-p...

Severity:

UX impact:

User visibility:

Effort required:

Priority:

Name check:

Fix check:

Regression:

Description

All the optimisation ideas behind #5965 is to have all entries in OpenLDAP cache. If they are not, performances will be horrible.

So each time openldap start, we need to execute the following search:

/var/rudder/ldap/openldap-data# ldapsearch -x -D "cn=Manager,cn=rudder-configuration" -w secret -b cn=rudder-configuration -s sub * + > /dev/null

Perhaps could we handle that in the rudder-slapd start-up script, in a background process ?

Actions

Copy link

Updated by Benoît PECCATTE over 9 years ago

Status changed from 8 to Pending technical review
Assignee changed from Benoît PECCATTE to Matthieu CERDA
Pull Request set to https://github.com/Normation/rudder-packages/pull/585

PR https://github.com/Normation/rudder-packages/pull/585

Actions

Copy link

Updated by Benoît PECCATTE over 9 years ago

Status changed from Pending technical review to Pending release
% Done changed from 0 to 100

Applied in changeset packages:rudder-packages|commit:86ecff056fa218b35abe5fb143f9552f3142a1fa.

Actions

Copy link

Updated by Matthieu CERDA over 9 years ago

Applied in changeset packages:rudder-packages|commit:851faa4ddd58210d527d973c7354e860c5527853.

Actions

Copy link

Updated by François ARMAND over 9 years ago

Target version changed from 2.10.9 to 2.11.6

I'm reverting that pull request because we are going to target 2.11 for that, AND we are not sure that we want to have a cache preload.

So don't redo the PR right now (and wait for any merge).

Actions

Copy link

Updated by François ARMAND over 9 years ago

Status changed from Pending release to Discussion

The commit was reverted in commit:2b83cb0

Actions

Copy link

Updated by François ARMAND over 9 years ago

François ARMAND wrote:

The commit was reverted in commit:2b83cb0 (https://github.com/Normation/rudder-packages/commit/2b83cb0085b6ec75bdccc02b4de69e64fb11b70e)

Actions

Copy link

Updated by François ARMAND about 9 years ago

Status changed from Discussion to 8
Assignee changed from Matthieu CERDA to Benoît PECCATTE

Benoit, given the decision in http://www.rudder-project.org/redmine/issues/5965#note-16, this bug is again to be corrected, targetting 2.11.

The pr request is OK, the most important being that the ldapsearch does not impact the init script, netheir in duraction nor in result, is taken into account.

Actions

Copy link

Updated by Benoît PECCATTE about 9 years ago

Status changed from 8 to In progress

Actions

Copy link

Updated by Benoît PECCATTE about 9 years ago

Status changed from In progress to Pending technical review
Assignee changed from Benoît PECCATTE to Matthieu CERDA
Pull Request changed from https://github.com/Normation/rudder-packages/pull/585 to https://github.com/Normation/rudder-packages/pull/598

PR https://github.com/Normation/rudder-packages/pull/598

Actions

Copy link

#10