[Freeipa-devel] [PATCH 0186-0191] Replace LDAP cache with RBTDB

[Petr Spacek]

On 14.8.2013 16:42, Petr Spacek wrote:
> On 14.8.2013 16:25, Petr Spacek wrote:
>> On 1.8.2013 15:57, Petr Spacek wrote:
>>> Hello,
>>>
>>> attached monster patches replace our internal cache/database with RBTDB
>>> implementation. See commit messages and comments inside.
>>>
>>> This patch set provides very basic functionality (including DNS support for
>>> updates). Error handling definitely needs more love, but it should be enough
>>> for rapid DNSSEC prototyping.
>>
>> Patch 186 v2: The code now applies incremental changes in LDAP to the
>> in-memory database. Commit message was modified to mention that wildcards are
>> now supported.
>>
>> Patch 187 v2: The code was re-worked and now it respects serial_autoincrement
>> option.
>>
>> Patch 188 v2: Minor comment clean-up and rebase on top of patch 187 v2.
>>
>> Patch 189 v2: Call to deleterdataset() nested in substractrdataset() was
>> deleted. This code was meant only for testing purposes.
>>
>> These patch set is now ready for review. Please see commit messages! Some
>> functionality is missing intentionally, but it will be fixed by separate
>> patches.
>
> It would be too easy!
>
> Patch 186 v3: Commit message was extended with information that LDAP MODRDN
> operation is not supported at the moment.
>
> Patch 187 v3: Missing file ldap_driver.h was added.

This extended patch set handles correctly object deletion from LDAP.

Patches 186-189 contain very minor changes, only moving code from one place to 
the other.

See commit messages for patches 190 and 191.

This should be testable. I would recommend to test the whole patch set at 
once, most probably it doesn't make much sense to test patches separately.

-- 
Petr^2 Spacek
-------------- next part --------------
A non-text attachment was scrubbed...
Name: bind-dyndb-ldap-pspacek-0186-4-Use-RBTDB-instead-of-internal-LDAP-cache.patch
Type: text/x-patch
Size: 87972 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/freeipa-devel/attachments/20130913/8f714050/attachment.bin>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: bind-dyndb-ldap-pspacek-0187-4-Update-serial-after-each-change-in-zone.patch
Type: text/x-patch
Size: 8513 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/freeipa-devel/attachments/20130913/8f714050/attachment-0001.bin>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: bind-dyndb-ldap-pspacek-0188-3-Drop-old-LDAP-cache.patch
Type: text/x-patch
Size: 19653 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/freeipa-devel/attachments/20130913/8f714050/attachment-0002.bin>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: bind-dyndb-ldap-pspacek-0189-3-Add-support-for-write-back-to-LDAP-after-a-DNS-updat.patch
Type: text/x-patch
Size: 11102 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/freeipa-devel/attachments/20130913/8f714050/attachment-0003.bin>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: bind-dyndb-ldap-pspacek-0190-Improve-logging-of-LDAP-entries-without-a-supported.patch
Type: text/x-patch
Size: 951 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/freeipa-devel/attachments/20130913/8f714050/attachment-0004.bin>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: bind-dyndb-ldap-pspacek-0191-Handle-object-deletion.patch
Type: text/x-patch
Size: 10376 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/freeipa-devel/attachments/20130913/8f714050/attachment-0005.bin>