#51234 Issue 51233 - ds-replcheck crashes in offline mode
Closed 2 years ago by spichugi. Opened 2 years ago by mreynolds.
mreynolds/389-ds-base issue51233  into  master

@@ -725,6 +725,10 @@ 

      missing = False

      for dn in master_dns:

          mresult = ldif_search(MLDIF, dn)

+         if mresult['entry'] is None and mresult['conflict'] is None and not mresult['tombstone']:

+             # Try from the beginning

+             MLDIF.seek(0)

+             mresult = ldif_search(MLDIF, dn)

          rresult = ldif_search(RLDIF, dn)


          if dn in replica_dns:

Bug Description:

When processing all the DN's found in the Master LDIF it is possible that the LDIF is not in the expected order and ldifsearch fails (crashing the tool).

Fix Description:

If ldifsearch does not find an entry, start from the beginning of the LDIF and try again.

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

rebased onto ea39a99

2 years ago

Pull-Request has been merged by mreynolds

2 years ago

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/4287

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

2 years ago