Bug Description:
If the server crashes then the region files can become corrupted and this prevents the server from starting.
Fix Description:
If we encounter a disorderly shutdown, then remove the region files so there is a clean slate to start with.
Also fixed some indenting issues around log functions...
relates: https://pagure.io/389-ds-base/issue/50599
Looks reasonable to me, @lkrispen I think you agree with this change after we discussed it?
yes, it looks safe to delete the files in these places, but :-)
In theory there can be more or less than the 001-003 files. I think with a very large dbcache it can be split over more files and then ther will be more. Also if you run with private mem there is no or only the 001 file.
Could there be generic delete of __.db* ?
rebased onto fa1f69a16ae5530524cf077b695fc9d5f682cbad
yes, it looks safe to delete the files in these places, but :-) In theory there can be more or less than the 001-003 files. I think with a very large dbcache it can be split over more files and then ther will be more. Also if you run with private mem there is no or only the 001 file. Could there be generic delete of __.db* ?
Yes, it's done now, please review...
looks good and disorderly finally spelled correctly :-)
ACK
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/3865
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
Bug Description:
If the server crashes then the region files can become corrupted and this prevents the server from starting.
Fix Description:
If we encounter a disorderly shutdown, then remove the region files so there is a clean slate to start with.
Also fixed some indenting issues around log functions...
relates: https://pagure.io/389-ds-base/issue/50599