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

[Petr Spacek]

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.

-- 
Petr^2 Spacek
-------------- next part --------------
A non-text attachment was scrubbed...
Name: freeipa-akrivoka-0079-04-Enable-Retro-Changelog-and-Content-Synchronization-D.patch
Type: text/x-patch
Size: 4400 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/freeipa-devel/attachments/20140114/cfff3331/attachment.bin>