From d790a290d8ae75bd3e37750e6e2116f5e4cc15c2 Mon Sep 17 00:00:00 2001
From: Jeffrey Regan <jregan@google.com>
Date: Wed, 16 May 2018 13:13:04 -0700
Subject: [PATCH] tweaking integration test

---
 demos/integration_tests.sh     | 32 ++++++++++++++++++++++++--------
 demos/ldap/integration_test.sh | 31 +++++++++++++------------------
 2 files changed, 37 insertions(+), 26 deletions(-)

diff --git a/demos/integration_tests.sh b/demos/integration_tests.sh
index 4501242341..1791109182 100755
--- a/demos/integration_tests.sh
+++ b/demos/integration_tests.sh
@@ -15,9 +15,16 @@
 # limitations under the License.
 
 
-# This script run periodically by kubernetes test-infra.
-# At time of writing, it's 'call point' was in
+# This script is run periodically by kubernetes test-infra.
+#
+# It's meant to use kustomized configurations against a live cluster.
+#
+# At time of writing, its 'call point' was in
 # https://github.com/kubernetes/test-infra/blob/master/jobs/config.json
+#
+# The script is written to assume that the process running it
+# has checked out kubernetes-sigs/kustomize repo, and has
+# changed the current directory to
 
 function exit_with {
   local msg=$1
@@ -25,15 +32,22 @@ function exit_with {
   exit 1
 }
 
-base_dir="$( cd "$(dirname "$0")/../../.." && pwd )"
-cd "$base_dir" || {
-  exit_with "Cannot cd to ${base_dir}. Aborting."
-}
+repo=kubernetes-sigs/kustomize
+if [[ `pwd` != */$repo ]]; then
+  exit_with "Script must be run from $repo"
+fi
+
+go install . || \
+  { exit_with "Failed to install kustomize."; }
 
-go install github.com/kubernetes-sigs/kustomize || \
-  { exit_with "Failed to install kustomize"; }
 export PATH=$GOPATH/bin:$PATH
 
+command -v kustomize >/dev/null 2>&1 || \
+  { exit_with "Require kustomize but it's not installed."; }
+
+command -v kubectl >/dev/null 2>&1 || \
+  { exit_with "Require kubectl but it's not installed."; }
+
 function runTest {
   local script=$1
   shift
@@ -50,4 +64,6 @@ function runTest {
   echo "$script passed."
 }
 
+pushd demos
 runTest ldap/integration_test.sh ldap/base
+popd
diff --git a/demos/ldap/integration_test.sh b/demos/ldap/integration_test.sh
index f7b0c42f4b..44dcff7b60 100755
--- a/demos/ldap/integration_test.sh
+++ b/demos/ldap/integration_test.sh
@@ -28,28 +28,23 @@
 
 set -x
 
-function exit_with {
-  local msg=$1
-  echo >&2 ${msg}
-  exit 1
-}
-
-# make sure kustomize and kubectl are available
-command -v kustomize >/dev/null 2>&1 || \
-  { exit_with "Require kustomize but it's not installed.  Aborting."; }
-command -v kubectl >/dev/null 2>&1 || \
-  { exit_with "Require kubectl but it's not installed.  Aborting."; }
-
-# set namespace to default
-kubectl config set-context $(kubectl config current-context) --namespace=default
-
-echo Kustomizing \"$1\"
-ls $1
-kustomize build $1 > generatedResources.yaml
+target=$1
+
+echo pwd is `pwd`
+echo Kustomizing \"$target\"
+ls $target
+kustomize build $target > generatedResources.yaml
 [[ $? -eq 0 ]] || { exit_with "Failed to kustomize build"; }
+
 cat generatedResources.yaml
+
+# Setting the namespace this way is a test-infra thing?
+kubectl config set-context \
+  $(kubectl config current-context) --namespace=default
+
 kubectl apply -f generatedResources.yaml
 [[ $? -eq 0 ]] || { exit_with "Failed to run kubectl apply"; }
+
 sleep 20
 
 # get the pod and namespace