timmybee/travel
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Fix persistent LDAP volumes preventing bootstrap.ldif from reloading #11

Merged
timmybee merged 1 commits from fix/ldap-bootstrap into master 2026-03-06 12:08:18 +08:00
Conversation 0 Commits 1 Files Changed 1 -6
timmybee commented 2026-03-06 12:02:32 +08:00
Owner
Copy Link

Summary

  • Removed sail-ldap-data and sail-ldap-config named Docker volumes from compose.yaml
  • The osixia/openldap image only runs bootstrap LDIF on first initialisation (when the volume is empty); persistent volumes meant any changes to bootstrap.ldif were silently ignored after the first sail up
  • Without named volumes, LDAP data is ephemeral per container lifetime — correct behaviour for a dev/test environment that should always start fresh from the bootstrap file
  • Also deleted the stale leftover volumes from old worktrees (ldap-issues_sail-ldap-*)

Test plan

  • Run sail down then sail up -d
  • Open phpLDAPadmin and confirm all three users (admin, approver, staff) are visible under ou=people,dc=travel,dc=local
  • Modify bootstrap.ldif, repeat the above — changes should be reflected immediately
## Summary - Removed `sail-ldap-data` and `sail-ldap-config` named Docker volumes from `compose.yaml` - The `osixia/openldap` image only runs bootstrap LDIF on first initialisation (when the volume is empty); persistent volumes meant any changes to `bootstrap.ldif` were silently ignored after the first `sail up` - Without named volumes, LDAP data is ephemeral per container lifetime — correct behaviour for a dev/test environment that should always start fresh from the bootstrap file - Also deleted the stale leftover volumes from old worktrees (`ldap-issues_sail-ldap-*`) ## Test plan - [ ] Run `sail down` then `sail up -d` - [ ] Open phpLDAPadmin and confirm all three users (admin, approver, staff) are visible under `ou=people,dc=travel,dc=local` - [ ] Modify `bootstrap.ldif`, repeat the above — changes should be reflected immediately
timmybee added 1 commit 2026-03-06 12:02:32 +08:00
Remove persistent LDAP volumes so bootstrap.ldif always applies on startup
All checks were successful
linter / quality (pull_request) Successful in 1m45s
Details
security / Dependency Audit (pull_request) Successful in 1m21s
Details
security / Static Analysis (pull_request) Successful in 2m11s
Details
tests / ci (8.4) (pull_request) Successful in 1m27s
Details
tests / ci (8.5) (pull_request) Successful in 2m21s
Details
d59ec55999 
The osixia/openldap image only runs bootstrap LDIF when the database is
empty. Named volumes (sail-ldap-data, sail-ldap-config) caused changes
to bootstrap.ldif to be ignored after the first run. Removing these
volumes ensures the test LDAP directory is always seeded fresh from the
bootstrap file on each sail up.
timmybee merged commit 89b3ddb793 into master 2026-03-06 12:08:18 +08:00
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
No items
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
timmybee
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: timmybee/travel#11
Delete Branch "fix/ldap-bootstrap"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?