[Freeipa-devel] [PATCH 0019] handle cleanRUV in the topology plugin
thierry bordaz
tbordaz at redhat.com
Fri Oct 23 10:38:12 UTC 2015
On 10/23/2015 12:39 PM, Ludwig Krispenz wrote:
>
> 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,
>>
> I will adress the points raised, thank you
>> 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.
> there are two potential problems with the cleanallruv task:
> 1] the rid could come back if not all servers were in sync, but with
> cleaning the changelog as part of cleanallruv I think the ris is low now
> 2] the cleanallruv is stuck on waiting for the task to complete on
> other servers even if they cannot be reached
>
> I want to avoid 2] therefore I choose this setting
Oh yes I absolutely agree.
What surprised me is that I thought IPA CLI already implemented that
flag in cleanRUV. But I was wrong.
thanks
thierry
>> 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/3b625e15/attachment.htm>
More information about the Freeipa-devel
mailing list