From fd14dc21b4dc52b3168f32a644a4167cc55ab919 Mon Sep 17 00:00:00 2001 From: Lucas Di Pentima Date: Wed, 27 Sep 2017 00:44:47 -0300 Subject: [PATCH] 12018: Avoid making duplicate api calls to add an already added user when the input file has the same line more than once. Arvados-DCO-1.1-Signed-off-by: Lucas Di Pentima --- sdk/python/arvados/commands/sync_groups.py | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/sdk/python/arvados/commands/sync_groups.py b/sdk/python/arvados/commands/sync_groups.py index 2bc08e23d5..a1125de1b6 100644 --- a/sdk/python/arvados/commands/sync_groups.py +++ b/sdk/python/arvados/commands/sync_groups.py @@ -150,7 +150,8 @@ def main(arguments=None, stdout=sys.stdout, stderr=sys.stderr): groups_created += 1 # Both group & user exist, check if user is a member g_uuid = group_name_to_uuid[group] - if not user in remote_groups[g_uuid]['previous_members']: + if not (user in remote_groups[g_uuid]['previous_members'] or + user in remote_groups[g_uuid]['current_members']): # User wasn't a member, but should. api_client.links().create(body={ 'link_class': 'permission', -- 2.30.2