[Freeipa-devel] [PATCH] 0069 Add/Remove DNS records for replicas

[Jakub Hrozek]

On 01/25/2011 06:05 PM, Simo Sorce wrote:
> On Tue, 25 Jan 2011 11:59:22 -0500
> Simo Sorce<ssorce at redhat.com>  wrote:
>
>> On Tue, 25 Jan 2011 17:47:49 +0100
>> Jakub Hrozek<jhrozek at redhat.com>  wrote:
>>
>>> On Tue, Jan 25, 2011 at 11:09:46AM -0500, Simo Sorce wrote:
>>>> On Tue, 25 Jan 2011 10:36:49 -0500
>>>> Simo Sorce<ssorce at redhat.com>  wrote:
>>>>
>>>>> On Tue, 25 Jan 2011 15:40:26 +0100
>>>>> Jakub Hrozek<jhrozek at redhat.com>  wrote:
>>>>>
>>>>>> On Tue, Jan 25, 2011 at 08:57:57AM -0500, Simo Sorce wrote:
>>>>>>> On Mon, 24 Jan 2011 23:06:17 +0100
>>>>>>> Jakub Hrozek<jhrozek at redhat.com>  wrote:
>>>>>>>
>>>>>>>> On 01/23/2011 02:09 AM, Simo Sorce wrote:
>>>>>>>>>
>>>>>>>>> Do it always when the dns tree is available, even if the
>>>>>>>>> replica being installed doesn't provide dns service
>>>>>>>>> itself.
>>>>>>>>>
>>>>>>>>> Ticket #824
>>>>>>>>>
>>>>>>>>> Simo.
>>>>>>>>>
>>>>>>>>
>>>>>>>> I tried applying this on top of both origin/master and 068
>>>>>>>> but did not succeed. Can you rebase, please?
>>>>>>>
>>>>>>> Rebased on top of the new 0068
>>>>>>>
>>>>>>> Simo.
>>>>>>>
>>>>>>
>>>>>> I think you attached the wrong patch - the attachment is
>>>>>> 68-02, not 69.
>>>>>
>>>>> Sigh.
>>>>> Right one attached now.
>>>>>
>>>>> Simo.
>>>>>
>>>>
>>>> Rebased once again leaving only 0068 on top of master.
>>>>
>>>> Simo.
>>>>
>>>> --
>>>> Simo Sorce * Red Hat, Inc * New York
>>>
>>> Adding DNS records works fine with or without Bind, but removing
>>> does not work - you need to import api from ipalib  - currently
>>> it's just undefined symbol in ipa-replica-manage. I suspect you'll
>>> also need to call api.bootstrap() and api.finalize() for the LDAP
>>> module to work.
>>
>> Ha now, I know what happend, the other patch I had in the tree is what
>> adds api and all, and that's why I guess you couldn't apply.
>> All my tests about removal passed but I had that other patch in the
>> tree too.
>>
>> Let me rebase and steal those changes from my other patch and
>> resubmit.
>>
>> Simo.
>>
>
> New patch attached.
>
> Simo.
>

Ack