SURFscz / SRAM-deploy

Deploy scripts for the SCZ
Apache License 2.0
5 stars 5 forks source link

Add index for entryUUID #230

Closed mrvanes closed 4 years ago

mrvanes commented 4 years ago

Deployed and checked:

root@ldap2:/etc/ldap/slapd.d/cn=config# cat olcDatabase\=\{1\}mdb.ldif 
# AUTO-GENERATED FILE - DO NOT EDIT!! Use ldapmodify.
# CRC32 eba6bb71
dn: olcDatabase={1}mdb
objectClass: olcDatabaseConfig
objectClass: olcMdbConfig
olcDatabase: {1}mdb
olcDbDirectory: /var/lib/ldap
olcLastMod: TRUE
olcDbCheckpoint: 512 30
olcDbIndex: objectClass eq
olcDbIndex: cn,uid eq
olcDbIndex: uidNumber,gidNumber eq
olcDbIndex: member,memberUid eq
olcDbIndex: entryUUID eq
...
mrvanes commented 4 years ago

M'n PR is prima, maar er is weer eens iemand op z'n teentjes getrapt waar ik verder niet zoveel aan veranderd heb

====================================================
8203== Running idempotency check...
8204====================================================
8239          http_nginx: BAD (3 changed, 0 allowed)
8240
baszoetekouw commented 4 years ago

Die idempotency is inmiddels gefixed in master; de PR build prima.

Anyway, komt deze change ook goed door in een bestaande ldapserver? En zo ja, voegt hij dan niet bij iedere deploy een extra index toe?

mrvanes commented 4 years ago

Die idempotency is inmiddels gefixed in master; de PR build prima.

Anyway, komt deze change ook goed door in een bestaande ldapserver? En zo ja, voegt hij dan niet bij iedere deploy een extra index toe?

Goeie vraag, zal 'm zo nog eens draaien. Het lijkt helemaal goed te gaan?