#50959 Issue 49731 - set and use db_home_directory by default
Closed by spichugi. Opened by mreynolds.
mreynolds/389-ds-base issue49731  into  master

Download 50959.patch

Description:

New instances will automatically create and use db_home_dir /dev/shm/dirsrv/slapd-INSTANCE.

relates: https://pagure.io/389-ds-base/issue/49731

Just a dummy question regarding containers. Aren't they using the same /dev/shm ?
A risk for several containers running slapd-standalone1 to step on each other ?

Just a dummy question regarding containers. Aren't they using the same /dev/shm ?
A risk for several containers running slapd-standalone1 to step on each other ?

That's a good question. If there are multiple instances with the same name then that would definitely be a problem. I'd hate to have to globally disable this feature if it's "containerised". Is that how customers deploy containers? One system with multiple images? I ask because maybe we could document that caveat?

In containers they each get their own /dev/shm I think, I think they are seperated. So I think that shouldn't be arisk. Certainly we can test this and then roll back if it's a problem. :)

rebased onto b097eadfffda2be904ff9ca8cda1f6f38398d32c

The patch looks good to me. Ack

rebased onto 96c8abba57bea957b5dfa944a95f339a970ccad7

Pull-Request has been merged by mreynolds

389-ds-base is moving from Pagure to Github. This means that new issues and pull requests
will be accepted only in 389-ds-base's github repository.

This pull request has been cloned to Github as issue and is available here:
- https://github.com/389ds/389-ds-base/issues/4012

If you want to continue to work on the PR, please navigate to the github issue,
download the patch from the attachments and file a new pull request.

Thank you for understanding. We apologize for all inconvenience.

Pull-Request has been closed by spichugi

Metadata