rmetzger commented on a change in pull request #11222: [FLINK-15834] Set up 
nightly builds in Azure & various CI improvements
URL: https://github.com/apache/flink/pull/11222#discussion_r384407405
 
 

 ##########
 File path: tools/azure-pipelines/prepare_precommit.sh
 ##########
 @@ -32,8 +32,17 @@ find . -type f -name '*.timestamp' | xargs touch
 export M2_HOME=/home/vsts/maven_cache/apache-maven-3.2.5/ 
 export PATH=/home/vsts/maven_cache/apache-maven-3.2.5/bin:$PATH
 mvn -version
-mvn install --settings ./tools/azure-pipelines/google-mirror-settings.xml 
-DskipTests -Drat.skip
+MVN_CALL="mvn install --settings 
./tools/azure-pipelines/google-mirror-settings.xml -DskipTests -Drat.skip 
$PROFILE"
+echo "Invoking Maven: '$MVN_CALL'"
+$MVN_CALL
+EXIT_CODE=$?
 
+if [ $EXIT_CODE != 0 ]; then
+       echo 
"=============================================================================="
+       echo "Build error. Exit code: $EXIT_CODE. Failing build"
+       echo 
"=============================================================================="
+       exit $EXIT_CODE
+fi
 
 chmod -R +x build-target
 chmod -R +x flink-end-to-end-tests
 
 Review comment:
   I left this file in, even though it is currently not used. I consider always 
running the e2e tests an experiment. If we need to undo it, this file is still 
around.
   If the experiment is successful, I will remove it (a cleanup PR is needed 
anyways once we remove travis support).

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


With regards,
Apache Git Services

Reply via email to