Project

General

Profile

Actions
Copy link

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 almost 10 years ago. Updated almost 10 years ago.

Status:
Released
Priority:
1 (highest)
Assignee:
Matthieu CERDA
Category:
System integration
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

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 open — 1 closed)

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

Updated by François ARMAND almost 10 years ago

  • Description updated (diff)
Actions
Copy link
 #2

Updated by Matthieu CERDA almost 10 years ago

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

Updated by Matthieu CERDA almost 10 years ago

  • Status changed from 8 to In progress
Actions
Copy link
 #4

Updated by Matthieu CERDA almost 10 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
Copy link
 #5

Updated by Matthieu CERDA almost 10 years ago

  • Status changed from Pending technical review to Pending release

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

Actions
Copy link
 #6

Updated by Benoît PECCATTE almost 10 years ago

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

Actions
Copy link
 #7

Updated by François ARMAND almost 10 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
Copy link
 #8

Updated by François ARMAND almost 10 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
Copy link
 #9

Updated by François ARMAND almost 10 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
Copy link
 #10

Updated by François ARMAND almost 10 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
Copy link
 #11

Updated by François ARMAND almost 10 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
Copy link
 #12

Updated by Matthieu CERDA almost 10 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
Copy link
 #13

Updated by Benoît PECCATTE almost 10 years ago

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

Updated by Matthieu CERDA almost 10 years ago

  • Status changed from Discussion to Pending release

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

Actions
Copy link
 #15

Updated by Benoît PECCATTE almost 10 years ago

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

Actions
Copy link
 #16

Updated by Vincent MEMBRÉ almost 10 years ago

  • Target version changed from 2.11.6 to 2.11.7
Actions
Copy link
 #17

Updated by Vincent MEMBRÉ almost 10 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
Copy link

Also available in: Atom PDF