Project

General

Profile

Actions

Bug #6105

closed

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

Migration script to remove unused indexes and reindex

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

Status:
Released
Priority:
1
Assignee:
Matthieu CERDA
Category:
System integration
Target version:
Severity:
UX impact:
User visibility:
Effort required:
Priority:
Name check:
Fix check:
Regression:

Description

In #5965, we remove all OpenLDAP indexes from slapd.conf.

So, we need to also remove them from the filesystem and reindexes OpenLDAP to be sur to not have corrupted states in OpenLDAP.

The know indexes (for differente version of Rudder) are:

/var/rudder/ldap/openldap-data/activeTechniqueId.bdb
/var/rudder/ldap/openldap-data/cn.bdb
/var/rudder/ldap/openldap-data/container.bdb
/var/rudder/ldap/openldap-data/directiveId.bdb
/var/rudder/ldap/openldap-data/isDynamic.bdb
/var/rudder/ldap/openldap-data/isEnabled.bdb
/var/rudder/ldap/openldap-data/isModified.bdb
/var/rudder/ldap/openldap-data/isSystem.bdb
/var/rudder/ldap/openldap-data/machineId.bdb
/var/rudder/ldap/openldap-data/nodeGroupId.bdb
/var/rudder/ldap/openldap-data/nodeId.bdb
/var/rudder/ldap/openldap-data/ruleId.bdb
/var/rudder/ldap/openldap-data/software.bdb
/var/rudder/ldap/openldap-data/softwareId.bdb
/var/rudder/ldap/openldap-data/softwareVersion.bdb
/var/rudder/ldap/openldap-data/techniqueCategoryId.bdb
/var/rudder/ldap/openldap-data/techniqueId.bdb
/var/rudder/ldap/openldap-data/uuid.bdb

It is normal to have (and we must keep):

/var/rudder/ldap/openldap-data/dn2id.bdb
/var/rudder/ldap/openldap-data/id2entry.bdb
/var/rudder/ldap/openldap-data/objectClass.bdb

A reindex must be done, to.


Subtasks 1 (0 open1 closed)

Bug #6120: rudder-inventory-ldap is missing a "then" statementRejectedBenoît PECCATTE2015-01-13Actions
Actions #1

Updated by François ARMAND about 9 years ago

  • Description updated (diff)
Actions #2

Updated by Matthieu CERDA about 9 years ago

  • Assignee changed from Matthieu C to Matthieu CERDA
Actions #3

Updated by Matthieu CERDA about 9 years ago

  • Status changed from 8 to In progress
Actions #4

Updated by Matthieu CERDA about 9 years ago

  • Status changed from In progress to Pending technical review
  • Assignee changed from Matthieu CERDA to Benoît PECCATTE
  • % Done changed from 0 to 100
  • Pull Request set to https://github.com/Normation/rudder-packages/pull/584
Actions #5

Updated by Matthieu CERDA about 9 years ago

  • Status changed from Pending technical review to Pending release

Applied in changeset packages:rudder-packages|commit:b7207931d90f1e25f55e080494657995ba7e8a1a.

Actions #6

Updated by Benoît PECCATTE about 9 years ago

Applied in changeset packages:rudder-packages|commit:65563d0261b0690279abbba1c4f8d360253de590.

Actions #7

Updated by François ARMAND about 9 years ago

Reverting commit:b7207931 to target branch 2.11.

Moreover, the next PR must be merged only after the main in Rudder (#) is merged.

Actions #8

Updated by François ARMAND about 9 years ago

  • Status changed from Pending release to Discussion
  • Target version changed from 2.10.9 to 2.11.6

Commit on branche 2.10 were reverted on commit:30371a405c81828882232aa68a425efed5d9c050

Actions #9

Updated by François ARMAND about 9 years ago

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

Given the decision in http://www.rudder-project.org/redmine/issues/5965#note-16 , we need back that correction, targetting 2.11.

Actions #10

Updated by François ARMAND about 9 years ago

Matthieu, the pull request https://github.com/Normation/rudder-packages/pull/584 may mostly be taken as it is (at first sight, of course need validation :)

Actions #11

Updated by François ARMAND about 9 years ago

OK, forget my last comment: the migration script must take into account the logic for cachesize/idlcachesize described in http://www.rudder-project.org/redmine/issues/5965#note-16.

Actions #12

Updated by Matthieu CERDA about 9 years ago

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

I suggest to put the cachesize fixes in another ticket, for clarity. This ticket targets LDAP index management :)

PR ready for review: https://github.com/Normation/rudder-packages/pull/599

Actions #13

Updated by Benoît PECCATTE about 9 years ago

  • Status changed from Pending technical review to Discussion
  • Assignee changed from Benoît PECCATTE to Matthieu CERDA
Actions #14

Updated by Matthieu CERDA about 9 years ago

  • Status changed from Discussion to Pending release

Applied in changeset packages:rudder-packages|commit:561a158dffb9cb987618b02444f15a26df2a89d6.

Actions #15

Updated by Benoît PECCATTE about 9 years ago

Applied in changeset packages:rudder-packages|commit:176d5850d0d824dc73372d379bffc1e50ca4d21a.

Actions #16

Updated by Vincent MEMBRÉ about 9 years ago

  • Target version changed from 2.11.6 to 2.11.7
Actions #17

Updated by Vincent MEMBRÉ about 9 years ago

  • Status changed from Pending release to Released

This bug has been fixed in Rudder 2.11.7, which was released these days.

Actions

Also available in: Atom PDF