From: Aric Gardner <agardner@linuxfoundation.org>
Date: Wed, 10 Jul 2019 18:10:26 +0000 (-0400)
Subject: Fix ldap check in gerrit create
X-Git-Tag: v0.25.3~3^2
X-Git-Url: https://gerrit.linuxfoundation.org/infra/gitweb?a=commitdiff_plain;h=d11c28e4e5739b01b857e2f04994d281f68a6f67;p=releng%2Flftools.git

Fix ldap check in gerrit create

ldap csv is not needed as we have lfidapi

Signed-off-by: Aric Gardner <agardner@linuxfoundation.org>
Change-Id: Ie3c29138ebd5562155afd43958da94bf95714652
---

diff --git a/shell/gerrit_create b/shell/gerrit_create
index 6cc4e41c..65006294 100755
--- a/shell/gerrit_create
+++ b/shell/gerrit_create
@@ -33,9 +33,9 @@ sanity_checks () {
   fi
 
   #Exit if ldap group does not exist
-  #To run from your workstation you must be on the vpn
+  #lftools must be configured and lfservices_releng member must be added to group.
   echo "Checking that ldapgroup=$ldapgroup exists"
-  check_ldap="$(lftools ldap csv "$ldapgroup")"
+  check_ldap="$(lftools lfidapi search-members "$ldapgroup")"
   if [[ -z "$check_ldap" ]]; then
     echo "ldap group is empty or does not exist."
     exit 1