[Freeipa-devel] [PATCH 0153-0158] move ipa-replica-manage del functionality into server-del

[Martin Babinsky]

On 06/15/2016 10:30 AM, Jan Cholasta wrote:
> Hi,
>
> On 12.6.2016 17:31, Martin Babinsky wrote:
>> On 06/09/2016 08:12 PM, Martin Babinsky wrote:
>>> These patches expand `server_del` to a full fledged IPA master killer in
>>> domain level 1.
>>>
>>> Due to 'server uninstallation removed master from topology' use case,
>>> the individual steps are not in the same order as in the original code
>>> to facilitate self-removal from topology without introducing an array of
>>> permissions for master to remove itself.
>>>
>>> I had no opportunity to test out the CI test suite because of technical
>>> problems so it would be nice if our upstream QE could give it a spin and
>>> report errors.
>>>
>>> http://www.freeipa.org/page/V4/Manage_replication_topology_4_4
>>> https://fedorahosted.org/freeipa/ticket/5181
>>>
>>>
>>>
>> Attaching rebased patches and bumping for review.
>>
>> Please note that they depend on 'Server Roles v2' patchset.
>
> Patch 0153:
>
> Should be an ipaserver module, unless it is required on clients as well,
> in which case it should be an ipalib module.
>
>
> Patch 0154: LGTM
>
>
> Patch 0155:
>
> In LDAPDelete subclasses, the primary key argument is multivalue, so I'm
> guessing your post_callback won't work correctly.
>
> Also, since this is *server*-del, s/master/server/ where applicable.
>
>
> Patch 0156: LGTM
>
>
> Patch 0157:
>
> This looks suspicious:
>
> +    result = server_del_cmd(hostname, version=api_version, **options)
>
> Version is automatically filled in in Command.__call__(), why do you add
> it manually here?
>
>
> Patch 0158: LGTM
>
>
> Honza
>

Attaching updated patches.

-- 
Martin^3 Babinsky
-------------- next part --------------
A non-text attachment was scrubbed...
Name: freeipa-mbabinsk-0153-3-ipaserver-module-for-working-with-managed-topology.patch
Type: text/x-patch
Size: 9954 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/freeipa-devel/attachments/20160615/256a3104/attachment.bin>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: freeipa-mbabinsk-0154-3-delegate-removal-of-master-DNS-record-and-replica-ke.patch
Type: text/x-patch
Size: 3115 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/freeipa-devel/attachments/20160615/256a3104/attachment-0001.bin>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: freeipa-mbabinsk-0155-3-server-del-perform-full-master-removal-in-managed-to.patch
Type: text/x-patch
Size: 20918 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/freeipa-devel/attachments/20160615/256a3104/attachment-0002.bin>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: freeipa-mbabinsk-0156-3-CI-test-suite-for-server-del.patch
Type: text/x-patch
Size: 15681 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/freeipa-devel/attachments/20160615/256a3104/attachment-0003.bin>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: freeipa-mbabinsk-0157-3-ipa-replica-manage-use-server_del-when-removing-doma.patch
Type: text/x-patch
Size: 12944 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/freeipa-devel/attachments/20160615/256a3104/attachment-0004.bin>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: freeipa-mbabinsk-0158-3-remove-the-master-from-managed-topology-during-unins.patch
Type: text/x-patch
Size: 10643 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/freeipa-devel/attachments/20160615/256a3104/attachment-0005.bin>