[Freeipa-devel] [PATCH] [WIP] ipa-replica-manage del with managed topology

Petr Vobornik pvoborni at redhat.com
Thu Jun 11 16:34:46 UTC 2015


Attaching a wip patch for `ipa-replica-manage del` to work with managed 
topology.

There are two prerequisite patches, they add following commands. All 
commands has NO_CLI flag which means they are hidden in CLI.
- server-del
- serverservice-add, mod, del, show, find

serverservice is object name for server "services" in cn=masters. I 
don't like the "service" name much but it's already been used in general 
discussions.

The main patch introduces two distinct methods for deleting servers, one 
for managed topology another for the old method. They share some code.

There are some differences in behavior.

1. the original 'del' worked also with winsync agreements. I'm not sure 
why is that. Shouldn't 'disconnect' be used for winsync agreements? At 
least man page says that.

2. options --clean and --force aren't used in the new method. I don't 
think that they are required. They serve for deleting the server entry 
in cn=masters.  The new method is build around this deletion so that 
it's always done which also means the cleanup is done.

3. Clean RUV task is run after deleting server entry and related 
cleanup. I don't think it works well. From observing the changes, it 
looks like it's executed before topology plugin manages to delete the 
agreements. This task then doesn't want to end and it reports that it 
has not finished somewhere. It finishes successfully if dirsrv is 
restarted. Agreements are then removed as well and all is fine.

Ludwig, should the clean RUV step be done differently? E.g. somewhere 
else or after something finishes?
-- 
Petr Vobornik
-------------- next part --------------
A non-text attachment was scrubbed...
Name: wip-freeipa-pvoborni-0748-ipa-replica-manage-adjust-del-to-work-with-managed-t.patch
Type: text/x-patch
Size: 10472 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/freeipa-devel/attachments/20150611/c75c74ae/attachment.bin>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: wip-freeipa-pvoborni-0747-add-serverservice-internal-commands.patch
Type: text/x-patch
Size: 7927 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/freeipa-devel/attachments/20150611/c75c74ae/attachment-0001.bin>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: wip-freeipa-pvoborni-0746-server-add-del-command.patch
Type: text/x-patch
Size: 1818 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/freeipa-devel/attachments/20150611/c75c74ae/attachment-0002.bin>


More information about the Freeipa-devel mailing list