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

[Martin Kosek]

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