[Freeipa-devel] [PATCH] 0079 Enable Retro Changelog and Content Synchronization DS plugins

[Petr Viktorin]

On 01/14/2014 10:40 AM, Petr Spacek wrote:
> On 28.11.2013 10:33, Ana Krivokapic wrote:
>> On 10/29/2013 06:57 PM, Ana Krivokapic wrote:
>>> On 10/29/2013 12:46 PM, Martin Kosek wrote:
>>>> On 10/25/2013 05:03 PM, Ana Krivokapic wrote:
>>>>> Hello,
>>>>>
>>>>> This patch addresses ticket
>>>>> https://fedorahosted.org/freeipa/ticket/3967.
>>>>>
>>>> NACK. I do not think this will work well with the case when DNS is
>>>> not used. As
>>>> bind-dyndb-ldap is not required component, FreeIPA could be
>>>> installed on a
>>>> machine without bind installed. And in that case, /var/named/ won't
>>>> be there.
>>>>
>>>> I think that this directory will follow similar pattern as
>>>> %{_localstatedir}/lib/ipa/pki-ca/publish
>>>> and be just %ghost and be created in when bind-dyndb-ldap is being
>>>> configured
>>>> in bindinstance.py.
>>>>
>>>> Martin
>>> Fixed, updated patch attached.
>>>
>> The patch needed a rebase.
>
> There was a copy&paste error in install/updates/20-syncrepl.update ,
> attached patch fixes it.
>
> I have tested the install & upgrade scenarios without and with one
> replica and it works for me, functional ACK.
>
> Please push it to master if nobody objects against the code.

Thanks! No objections, pushed: 689382dc833e687d30349b10a8fd7dc740d54d08

-- 
Petr³