[Freeipa-devel] [PATCH 0020-0021] some topology plugin fixes
Ludwig Krispenz
lkrispen at redhat.com
Fri Oct 30 08:57:48 UTC 2015
On 10/29/2015 01:28 PM, thierry bordaz wrote:
> On 10/23/2015 10:44 AM, Ludwig Krispenz wrote:
>> Hi,
>> the attached two patches address issues I found when testing ca
>> management in the topology plugin
>>
>> Thanks for review,
>> Ludwig
>>
>>
> Hi Ludwig,
>
> Patch 20 is good to me. I have one remark, you call
> ipa_topo_cfg_host_find with lock flag. So that the replica config is
> not updated during the test.
> Now the lock protects each call separately. The risk is very low that
> the target host could become unmanaged by the time we test the source
> host.
yes, and if two paralle operations do related things like adding an
agreement and making a host managed/unmanaged there is a race for the
lock. The lock itself cannot prevent these things, it only can protect
the data structures from being read while modified.
Also with two separate locked calls the second call has a chance to be
aware of parallel changes
> ACK.
>
> Patch 21 is also good. Just in ipa_topo_util_init_hosts, why not
> calling ipa_topo_cfg_host_add to not duplicate the source ?
no reason, revised patch is attached, thanks for noticing
>
> thanks
> thierry
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://listman.redhat.com/archives/freeipa-devel/attachments/20151030/8c1b4797/attachment.htm>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: lkrispen-freeipa-0021-1-update-list-of-managed-servers-when-a-suffix-becomes.patch
Type: text/x-patch
Size: 6816 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/freeipa-devel/attachments/20151030/8c1b4797/attachment.bin>
More information about the Freeipa-devel
mailing list