Re: [Freeipa-devel] [PATCH 0016] clear start attr from segment after initialization

2015-06-30 Thread Tomas Babej
On 06/30/2015 12:45 PM, thierry bordaz wrote: > On 06/30/2015 12:05 PM, Ludwig Krispenz wrote: >> new patch with comments attached >> >> On 06/30/2015 10:43 AM, thierry bordaz wrote: >>> On 06/30/2015 09:19 AM, Ludwig Krispenz wrote: On 06/26/2015 02:14 PM, thierry bordaz wrote: > O

Re: [Freeipa-devel] [PATCH 0016] clear start attr from segment after initialization

2015-06-30 Thread thierry bordaz
On 06/30/2015 12:05 PM, Ludwig Krispenz wrote: new patch with comments attached On 06/30/2015 10:43 AM, thierry bordaz wrote: On 06/30/2015 09:19 AM, Ludwig Krispenz wrote: On 06/26/2015 02:14 PM, thierry bordaz wrote: On 06/22/2015 11:35 AM, Ludwig Krispenz wrote: fix for ticket #5065, rem

Re: [Freeipa-devel] [PATCH 0016] clear start attr from segment after initialization

2015-06-30 Thread Ludwig Krispenz
new patch with comments attached On 06/30/2015 10:43 AM, thierry bordaz wrote: On 06/30/2015 09:19 AM, Ludwig Krispenz wrote: On 06/26/2015 02:14 PM, thierry bordaz wrote: On 06/22/2015 11:35 AM, Ludwig Krispenz wrote: fix for ticket #5065, removing start - after online init copmpleted - add

Re: [Freeipa-devel] [PATCH 0016] clear start attr from segment after initialization

2015-06-30 Thread thierry bordaz
On 06/30/2015 09:19 AM, Ludwig Krispenz wrote: On 06/26/2015 02:14 PM, thierry bordaz wrote: On 06/22/2015 11:35 AM, Ludwig Krispenz wrote: fix for ticket #5065, removing start - after online init copmpleted - additionally check after startup Hi Ludwig, The fix looks good to me. I have jus

Re: [Freeipa-devel] [PATCH 0016] clear start attr from segment after initialization

2015-06-30 Thread Ludwig Krispenz
On 06/26/2015 02:14 PM, thierry bordaz wrote: On 06/22/2015 11:35 AM, Ludwig Krispenz wrote: fix for ticket #5065, removing start - after online init copmpleted - additionally check after startup Hi Ludwig, The fix looks good to me. I have just a clarification regarding ipa_topo_util_reset_

Re: [Freeipa-devel] [PATCH 0016] clear start attr from segment after initialization

2015-06-26 Thread thierry bordaz
On 06/22/2015 11:35 AM, Ludwig Krispenz wrote: fix for ticket #5065, removing start - after online init copmpleted - additionally check after startup Hi Ludwig, The fix looks good to me. I have just a clarification regarding ipa_topo_util_reset_init. It resets 'nsds5BeginReplicaRefresh' at t

Re: [Freeipa-devel] [PATCH 0016] clear start attr from segment after initialization

2015-06-22 Thread Oleg Fayans
Oops, you are right! That fixed the issue On 06/22/2015 12:09 PM, Ludwig Krispenz wrote: did you apply 0014 and 0015 before ? On 06/22/2015 11:46 AM, Oleg Fayans wrote: applying freeipa-lkrispen-0016-clear-start-attr-from-segment-after-initialization.patch error: patch failed: daemons/ipa-sla

Re: [Freeipa-devel] [PATCH 0016] clear start attr from segment after initialization

2015-06-22 Thread Ludwig Krispenz
did you apply 0014 and 0015 before ? On 06/22/2015 11:46 AM, Oleg Fayans wrote: applying freeipa-lkrispen-0016-clear-start-attr-from-segment-after-initialization.patch error: patch failed: daemons/ipa-slapi-plugins/topology/topology_util.c:471 error: daemons/ipa-slapi-plugins/topology/topology_

Re: [Freeipa-devel] [PATCH 0016] clear start attr from segment after initialization

2015-06-22 Thread Oleg Fayans
applying freeipa-lkrispen-0016-clear-start-attr-from-segment-after-initialization.patch error: patch failed: daemons/ipa-slapi-plugins/topology/topology_util.c:471 error: daemons/ipa-slapi-plugins/topology/topology_util.c: patch does not apply On 06/22/2015 11:35 AM, Ludwig Krispenz wrote: f