[Freeipa-devel] [PATCH 0019] handle cleanRUV in the topology plugin
thierry bordaz
tbordaz at redhat.com
Fri Oct 23 13:34:05 UTC 2015
On 10/23/2015 03:38 PM, Ludwig Krispenz wrote:
>
> On 10/23/2015 03:19 PM, thierry bordaz wrote:
>> Hi Ludwig,
>>
>> Thanks for the patch.
>> Yes it is looking good to me. Just a minor change about the message
>> logged (if case of failure to add the cleanallruv task), you may
>> recommend to the administrator the exact command to run.
> no, also we do not know why it failed, there would be the option to do
> it via ipa-replica-manage or ldapmodify, the logging that creating the
> task failed should be be enough to start investigation and do a manual
> cleanallruv
Ok. I agree. Thanks
>>
>> ACK
>>
>> thanks
>> thierry
>> On 10/23/2015 02:27 PM, Ludwig Krispenz wrote:
>>> Hi Thierry,
>>>
>>> hope this addresses your concerns
>>>
>>> Ludwig
>>>
>>> On 10/23/2015 11:24 AM, thierry bordaz wrote:
>>>> On 10/23/2015 11:00 AM, thierry bordaz wrote:
>>>>> On 10/12/2015 01:17 PM, Ludwig Krispenz wrote:
>>>>>>
>>>>>> On 10/12/2015 12:44 PM, Martin Basti wrote:
>>>>>>>
>>>>>>>
>>>>>>> On 23.07.2015 10:46, Ludwig Krispenz wrote:
>>>>>>>> The attached patch moves the cleaning of the RUV into the
>>>>>>>> topology plugin.
>>>>>>>>
>>>>>>>> I encountered a problem when removing a replica, which
>>>>>>>> disconnects the topology, but it was fixed with my WIP for #5072.
>>>>>>>>
>>>>>>>> I want to keep these issues separate, so please review and test
>>>>>>>> the patch and let me know about issues found
>>>>>>>>
>>>>>>>> Ludwig
>>>>>>>>
>>>>>>>>
>>>>>>>
>>>>>>> Is this patch still valid and pending review?
>>>>>> it should be still valid, waiting for review, wanted to rebase
>>>>>> after topology/promotion patches have been checked in and resend
>>>>>>
>>>>>>
>>>>>>
>>>>> Hello Ludwig,
>>>>>
>>>>> The patch looks good. I have few minor remarks:
>>>>>
>>>>> * Are the hostname in ruv always fqdn ? to retrieve the RUV
>>>>> element of a given host you use 'strstr'.
>>>>> If you have host vm-11 and vm-112, I wonder if it could pickup
>>>>> the wrong RUV element
>>>>> * In ipa_topo_util_cleanruv_element you need a pblock_done/free
>>>>> (or destroy)
>>>>> * In it fails to add the clearn-ruv task, you should log a
>>>>> message so that the admin knows what to do.
>>>>>
>>>>> thanks
>>>>> thierry
>>>>>
>>>>>
>>>>>
>>>> Hi Ludwig,
>>>>
>>>> Additional question. cleanruv is done with 'replica-force-cleaning:
>>>> yes'. Currently ipa-replica-manage does not implement this flag.
>>>> Why do you use it in topology plugin.
>>>> My concern is that if we delete a host before all the updates from
>>>> that host has been received, could we receive a late update that
>>>> will recreate the ruv element ?
>>>>
>>>> thanks
>>>> thierry
>>>
>>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://listman.redhat.com/archives/freeipa-devel/attachments/20151023/6b1e472c/attachment.htm>
More information about the Freeipa-devel
mailing list