Fix ldap check in gerrit create 51/16151/1
authorAric Gardner <agardner@linuxfoundation.org>
Wed, 10 Jul 2019 18:10:26 +0000 (14:10 -0400)
committerAric Gardner <agardner@linuxfoundation.org>
Wed, 10 Jul 2019 18:11:08 +0000 (14:11 -0400)
ldap csv is not needed as we have lfidapi

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

shell/gerrit_create

index 6cc4e41..6500629 100755 (executable)
@@ -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