Description:
on some platforms the maximum FD limit is high it can cause a OOM at server startup. So we need to add a hard maximum limit.
relates: https://pagure.io/389-ds-base/issue/49990
looks fine to me - I think it will make the freelist patch more important though ....
Ack btw
Fix looks good, just a question @mreynolds any reason to increase the default maxdescriptor of the daemon from 512K to 1M ? It looks to me that 512K is already high regarding C10K capability.
@vashirov suggested 1m as it's the kernel default. See the issue for his exact comment: https://pagure.io/389-ds-base/issue/49990
rebased onto 54b941dea5b595302c47ed2ebecf8ec30dc76050
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/3906
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
Description:
on some platforms the maximum FD limit is high it can cause a OOM at server startup. So we need to add a hard maximum limit.
relates: https://pagure.io/389-ds-base/issue/49990