#3448 Some update plugins do ldap.update_entry directly
Closed: Fixed None by sbose. Opened 5 years ago by sbose.

Instead of modifing LDAP objects directly the update plugins should return a dict with the needs changes. The changes are then applied by the caller in updateclient.py which respects the -t option of ipa-ldap-updater and adds lots of useful debugging information when running with -v

To be honest I liked the original description better, to honor the test flag.

I think we'd be quite restrictive to always have to return a dict of changes, which is why I provided for both methods.

Part of upgrade refactoring

Test option was removed in commit:

Upgrade plugins are allowed to use ldap and API by design.

Closing ticket as fixed because --test option was removed.

Metadata Update from @sbose:
- Issue assigned to mbasti
- Issue set to the milestone: FreeIPA 4.2

2 years ago

Login to comment on this ticket.