You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@geode.apache.org by GitBox <gi...@apache.org> on 2019/01/11 15:29:06 UTC

[GitHub] pivotal-jbarrett commented on a change in pull request #38: Update script for running tests on AWS

pivotal-jbarrett commented on a change in pull request #38: Update script for running tests on AWS
URL: https://github.com/apache/geode-benchmarks/pull/38#discussion_r247154251
 
 

 ##########
 File path: infrastructure/scripts/aws/run_tests.sh
 ##########
 @@ -39,11 +39,17 @@ echo "HOSTS=${HOSTS}"
 ssh ${SSH_OPTIONS} geode@$FIRST_INSTANCE "\
   rm -rf geode-benchmarks geode && \
   git clone https://github.com/apache/geode geode && \
-  (pushd geode; git checkout ${BRANCH}) && \
-  (pushd geode; ./gradlew pTML -PversionNumber=${DATE} -PreleaseType="-BENCHMARKBUILD") && \
+  (pushd geode && git checkout ${BRANCH}) && \
+  (pushd geode && ./gradlew install installDist)"
+
+
+VERSION=$(ssh ${SSH_OPTIONS} geode@$FIRST_INSTANCE geode/geode-assembly/build/install/apache-geode/bin/gfsh version)
 
 Review comment:
   Readability. The version variable would need to escaped so that it is evaluated after the gfsh command executed just prior to the second gradlew.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services