[Freeipa-devel] [PATCH] 921 fix existing replication agreements

[Rob Crittenden]

Martin Kosek wrote:
> On Mon, 2012-01-30 at 16:28 +0200, Alexander Bokovoy wrote:
>> On Mon, 30 Jan 2012, Martin Kosek wrote:
>>> My main concern is if it is conceptually OK to create a separate script
>>> for one-time fixes like this one. What if we find another problem with
>>> replica agreements where we would need to update existing agreements?
>>> Would we create another fix tool or enhance ipa-fixreplica? I am just
>>> afraid that in time we would "pollute" our ipa-* tool collection with
>>> one time fixes.
>>>
>>> Maybe we could move this agreement fix to a plugin in ipa-ldap-updater
>>> or create a more general tool for one-time fixes like this one. I am
>>> thinking about something like this:
>>> ipa-server-remedy [-l/--list]  [-u/--unit] [-d/--debug] [-t/--test]
>>> When run without options it would run all remedy "plugins" to fix all
>>> possible one-time errors. There would be also these options:
>>> --list: list all remedy "plugins". In this case there would be just this
>>> one
>>> --unit: run just the chosen remedy "plugin"
>>> --debug, --test: the same functionality as in your patch
>> We do have ipa-ldap-updater which now supports plugins and may serve
>> the same purpose without introducing new utility...
>>
>
> Yes, this was my first suggestion. I mentioned this general utility as a
> second option if we need to implement this onetime fix as a separate
> tool for some reason.
>
> Martin
>

This is for FreeIPA 2.1 which doesn't support update plugins.

rob