You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@beam.apache.org by ma...@apache.org on 2019/06/24 20:23:22 UTC

[beam] branch master updated: [BEAM-7598] Do not build Python tar file in run_integration_test.sh

This is an automated email from the ASF dual-hosted git repository.

markliu pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/beam.git


The following commit(s) were added to refs/heads/master by this push:
     new 2d28efe  [BEAM-7598] Do not build Python tar file in run_integration_test.sh
     new 5b91cb4  Merge pull request #8916 from markflyhigh/fix-run-it-script
2d28efe is described below

commit 2d28efedaceecb35e5d83bb80702aefcef317fcb
Author: Mark Liu <ma...@markliu-macbookpro2.roam.corp.google.com>
AuthorDate: Thu Jun 20 08:33:03 2019 -0700

    [BEAM-7598] Do not build Python tar file in run_integration_test.sh
---
 sdks/python/scripts/run_integration_test.sh | 7 +++----
 1 file changed, 3 insertions(+), 4 deletions(-)

diff --git a/sdks/python/scripts/run_integration_test.sh b/sdks/python/scripts/run_integration_test.sh
index 6e7b3ca..f4e38df 100755
--- a/sdks/python/scripts/run_integration_test.sh
+++ b/sdks/python/scripts/run_integration_test.sh
@@ -171,12 +171,11 @@ fi
 
 if [[ -z $PIPELINE_OPTS ]]; then
 
-  # Create a tarball if not exists
+  # Get tar ball path
   if [[ $(find ${SDK_LOCATION} 2> /dev/null) ]]; then
-    SDK_LOCATION=$(find ${SDK_LOCATION})
+    SDK_LOCATION=$(find ${SDK_LOCATION} | tail -n1)
   else
-    python setup.py -q sdist
-    SDK_LOCATION=$(ls dist/apache-beam-*.tar.gz | tail -n1)
+    echo "[WARNING] Could not find SDK tarball in SDK_LOCATION: $SDK_LOCATION."
   fi
 
   # Install test dependencies for ValidatesRunner tests.