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
Linked to Bugzilla bug: https://bugzilla.redhat.com/show_bug.cgi?id=1199520 (Red Hat Enterprise Linux 7)
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
to comment on this ticket.