[Freeipa-devel] [PATCH 0488-0489] Perfomance: membership processing related patches

[Martin Babinsky]

On 05/28/2016 01:17 PM, Martin Basti wrote:
> https://fedorahosted.org/freeipa/ticket/4995
>
> Patches attached
>
>
>

Hi,

PATCH 0488: LGTM

PATCH 0489:

@@ -996,10 +997,10 @@ def check_deleted_segments(hostname, masters, 
topo_errors, starting_host):
          i = 0
          while True:
              left = api.Command.topologysegment_find(
-                suffix_name, iparepltoposegmentleftnode=hostname, 
sizelimit=0
+                suffix_name, iparepltoposegmentleftnode=hostname, 
sizelimit=0,
              )['result']
              right = api.Command.topologysegment_find(
-                suffix_name, iparepltoposegmentrightnode=hostname, 
sizelimit=0
+                suffix_name, iparepltoposegmentrightnode=hostname, 
sizelimit=0,
              )['result']

it seems that you added 'no_members=True' there and then removed it 
because reasons. Please revert the this part to the original code so 
that it does not stick out.

-- 
Martin^3 Babinsky