update-gerrit-groups: check that group is not already included

Change-Id: I1a3643d86cacdd7357e908a126a2ce15bf397e09

1 files changed, 5 insertions, 1 deletions

diff --git a/update-gerrit-groups.py b/update-gerrit-groups.py
index a5ecaf3..f90d7bf 100755
--- a/update-gerrit-groups.py
+++ b/update-gerrit-groups.py
@@ -37,4 +37,8 @@ for group, ldapmembers in result.iteritems():
         gerrit.add_group_member(group, member)
     for member in members_to_remove:
         log.warn("Wants to remove %s from group %s", member, group)
-    gerrit.add_included_group(group, "ldap/%s" % group)
+
+    included_groups = gerrit.list_included_groups(group)
+    if "ldap/%s" % group not in included_groups:
+        log.debug("adding ldap/%s to included groups")
+        gerrit.add_included_group(group, "ldap/%s" % group)