389-ds-base

#50233 Ticket 50232 - export creates not importable ldif file
Closed by spichugi. Opened by lkrispen.
lkrispen/389-ds-base t50232  into  master

Download 50233.patch

Bug: If the RUV entry hasa smaller entryid than the suffix entry it will be
exported before the suffix. If that ldif is used for import the RUV entry
is skipped and a new one generated with a different database generation

Fix: Before exporting the RUV check that the suffix is alread exported, if not
make the RUV entry pending and write it after all othere entries

Reviewed by: ?

I would think the changelog creation is ensured during the topology setup? But this doesn't really hurt here either.

Ignore my last comment, you are doing a standalone, and enabling their changelogs.

I think the test looks good, and the C patch looks fine too. I think I would like to see some more comments in the C code explaining the conditions and what is occuring to prompt these checks being revelant, I think reading this test and that patch, I'm hazy on what the fault is, and how this fixes it.

int32_t?

The patch looks good to me. ACK

rebased onto eb1b5c5171b548eb9c13464c351f8d8c102a6264

changet int type and addded comment to explain why we need this

Pull-Request has been merged by lkrispen

Ack!

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

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
None
No Tags
Static archive. pagure.io was sunset in 2026; this is a read-only snapshot.
Red Hat Logo

Fedora is sponsored by Red Hat. Learn more about the relationship between Red Hat and Fedora »

© Red Hat, Inc. and others.