#50811 Issue 50599 - Remove db region files prior to db recovery
Closed by spichugi. Opened by mreynolds.
mreynolds/389-ds-base issue50599  into  master

Download 50811.patch

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

Metadata