Re: [Freeipa-devel] [PATCH 0022] migrate-ds: proper treatment of unsuccessful migrations

2015-03-23 Thread Petr Vobornik
On 03/23/2015 12:55 PM, Martin Babinsky wrote: On 03/23/2015 09:52 AM, Martin Babinsky wrote: On 03/18/2015 10:54 AM, Martin Babinsky wrote: This is a proper fix to both https://fedorahosted.org/freeipa/ticket/4846 and https://fedorahosted.org/freeipa/ticket/4952. To do this I had to throw out

Re: [Freeipa-devel] [PATCH 0022] migrate-ds: proper treatment of unsuccessful migrations

2015-03-23 Thread Martin Babinsky
On 03/23/2015 09:52 AM, Martin Babinsky wrote: On 03/18/2015 10:54 AM, Martin Babinsky wrote: This is a proper fix to both https://fedorahosted.org/freeipa/ticket/4846 and https://fedorahosted.org/freeipa/ticket/4952. To do this I had to throw out some unused parameters from _update_default_gro

Re: [Freeipa-devel] [PATCH 0022] migrate-ds: proper treatment of unsuccessful migrations

2015-03-23 Thread Martin Babinsky
On 03/18/2015 10:54 AM, Martin Babinsky wrote: This is a proper fix to both https://fedorahosted.org/freeipa/ticket/4846 and https://fedorahosted.org/freeipa/ticket/4952. To do this I had to throw out some unused parameters from _update_default_group function (particularly the pesky pkey causing

[Freeipa-devel] [PATCH 0022] migrate-ds: proper treatment of unsuccessful migrations

2015-03-18 Thread Martin Babinsky
This is a proper fix to both https://fedorahosted.org/freeipa/ticket/4846 and https://fedorahosted.org/freeipa/ticket/4952. To do this I had to throw out some unused parameters from _update_default_group function (particularly the pesky pkey causing bug #4846 to pop out). I did not test the