As reported by https://fedorahosted.org/freeipa/ticket/4636, configuration using only uniqueness-subtrees introduces some constraints.
In the described use case, it would be more rapid to define uniqueness-subtrees-exclude from part of the DIT that should not be enforced
The IPA ticket https://fedorahosted.org/freeipa/ticket/4636 is related to 'Views'.
'Views' are targeted for Freeipa 4.1 which is going to be RHEL 7.1
Alexander plans to work on this improvement together with tbordaz.
Ticket has been cloned to Bugzilla: https://bugzilla.redhat.com/show_bug.cgi?id=1154147
Attached is the first cut to add exclude subtrees.
The fix looks good.
Just a remark in search_one_berval. I think that if one of the retrieved entry is part of an excluded subtree, it should rather continue to test the others entries than report LDAP_SUCCESS.
It does continue to the other entries, it only skips this entry because of the excluded match. This is a nested loop and a break within the nested one only brings you to the outer loop, not out of it.
Alexander, you are right, no concern regarding search_one_berval.
I'm afraid uid.c had been modified and your patch fails to apply by "git am". Could it be possible to rebase with master on your checked out branch and recreate a patch? If you have the right to push the patch, please go ahead and do so. If not, we will do it.
Thanks Rich for the review
'''push to master'''
dirsrvtests/tickets/ticket47823_test.py | 13 ++--
dirsrvtests/tickets/ticket47927_test.py | 300 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
ldap/servers/plugins/uiduniq/uid.c | 97 +++++++++++++++++++++-----
3 files changed, 389 insertions(+), 21 deletions(-)
create mode 100644 dirsrvtests/tickets/ticket47927_test.py
git push origin master
Counting objects: 20, done.
Delta compression using up to 4 threads.
Compressing objects: 100% (11/11), done.
Writing objects: 100% (11/11), 4.01 KiB, done.
Total 11 (delta 8), reused 0 (delta 0)
cd8614a..cd9afbb master -> master
Author: Thierry Bordaz firstname.lastname@example.org
Date: Tue Apr 28 19:35:34 2015 +0200
Metadata Update from @tbordaz:
- Issue assigned to abbra
- Issue set to the milestone: 1.3.4 backlog
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 issue has been cloned to Github and is available here:
If you want to receive further updates on the issue, please navigate to the github issue
and click on subscribe button.
Thank you for understanding. We apologize for all inconvenience.
Metadata Update from @spichugi:
- Issue close_status updated to: wontfix (was: Fixed)
to comment on this ticket.