You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@spark.apache.org by pw...@apache.org on 2014/09/02 22:08:56 UTC
git commit: [Build] merge changes to run-tests-jenkins from master
branch
Repository: spark
Updated Branches:
refs/heads/branch-1.1 ccf35202e -> dff42a77b
[Build] merge changes to run-tests-jenkins from master branch
Author: Nicholas Chammas <ni...@gmail.com>
Author: nchammas <ni...@gmail.com>
Closes #2237 from nchammas/branch-1.1 and squashes the following commits:
39bdd5e [Nicholas Chammas] merge updates from master
f5aa841 [nchammas] Merge pull request #3 from apache/branch-1.1
Project: http://git-wip-us.apache.org/repos/asf/spark/repo
Commit: http://git-wip-us.apache.org/repos/asf/spark/commit/dff42a77
Tree: http://git-wip-us.apache.org/repos/asf/spark/tree/dff42a77
Diff: http://git-wip-us.apache.org/repos/asf/spark/diff/dff42a77
Branch: refs/heads/branch-1.1
Commit: dff42a77ba85799ebe1faa5ec96fbe24e833e2e0
Parents: ccf3520
Author: Nicholas Chammas <ni...@gmail.com>
Authored: Tue Sep 2 13:08:52 2014 -0700
Committer: Patrick Wendell <pw...@gmail.com>
Committed: Tue Sep 2 13:08:52 2014 -0700
----------------------------------------------------------------------
dev/run-tests-jenkins | 183 +++++++++++++++++++++++++++++++++------------
1 file changed, 134 insertions(+), 49 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/spark/blob/dff42a77/dev/run-tests-jenkins
----------------------------------------------------------------------
diff --git a/dev/run-tests-jenkins b/dev/run-tests-jenkins
index 3076eb8..06c3781 100755
--- a/dev/run-tests-jenkins
+++ b/dev/run-tests-jenkins
@@ -19,67 +19,152 @@
# Wrapper script that runs the Spark tests then reports QA results
# to github via its API.
+# Environment variables are populated by the code here:
+#+ https://github.com/jenkinsci/ghprb-plugin/blob/master/src/main/java/org/jenkinsci/plugins/ghprb/GhprbTrigger.java#L139
# Go to the Spark project root directory
FWDIR="$(cd `dirname $0`/..; pwd)"
cd "$FWDIR"
COMMENTS_URL="https://api.github.com/repos/apache/spark/issues/$ghprbPullId/comments"
+PULL_REQUEST_URL="https://github.com/apache/spark/pull/$ghprbPullId"
-function post_message {
- message=$1
- data="{\"body\": \"$message\"}"
- echo "Attempting to post to Github:"
- echo "$data"
+COMMIT_URL="https://github.com/apache/spark/commit/${ghprbActualCommit}"
+# GitHub doesn't auto-link short hashes when submitted via the API, unfortunately. :(
+SHORT_COMMIT_HASH="${ghprbActualCommit:0:7}"
- curl -D- -u x-oauth-basic:$GITHUB_OAUTH_KEY -X POST --data "$data" -H \
- "Content-Type: application/json" \
- $COMMENTS_URL | head -n 8
+TESTS_TIMEOUT="120m" # format: http://linux.die.net/man/1/timeout
+
+function post_message () {
+ local message=$1
+ local data="{\"body\": \"$message\"}"
+ local HTTP_CODE_HEADER="HTTP Response Code: "
+
+ echo "Attempting to post to Github..."
+
+ local curl_output=$(
+ curl `#--dump-header -` \
+ --silent \
+ --user x-oauth-basic:$GITHUB_OAUTH_KEY \
+ --request POST \
+ --data "$data" \
+ --write-out "${HTTP_CODE_HEADER}%{http_code}\n" \
+ --header "Content-Type: application/json" \
+ "$COMMENTS_URL" #> /dev/null #| "$FWDIR/dev/jq" .id #| head -n 8
+ )
+ local curl_status=${PIPESTATUS[0]}
+
+ if [ "$curl_status" -ne 0 ]; then
+ echo "Failed to post message to GitHub." >&2
+ echo " > curl_status: ${curl_status}" >&2
+ echo " > curl_output: ${curl_output}" >&2
+ echo " > data: ${data}" >&2
+ # exit $curl_status
+ fi
+
+ local api_response=$(
+ echo "${curl_output}" \
+ | grep -v -e "^${HTTP_CODE_HEADER}"
+ )
+
+ local http_code=$(
+ echo "${curl_output}" \
+ | grep -e "^${HTTP_CODE_HEADER}" \
+ | sed -r -e "s/^${HTTP_CODE_HEADER}//g"
+ )
+
+ if [ -n "$http_code" ] && [ "$http_code" -ne "201" ]; then
+ echo " > http_code: ${http_code}." >&2
+ echo " > api_response: ${api_response}" >&2
+ echo " > data: ${data}" >&2
+ fi
+
+ if [ "$curl_status" -eq 0 ] && [ "$http_code" -eq "201" ]; then
+ echo " > Post successful."
+ fi
}
-start_message="QA tests have started for PR $ghprbPullId."
-if [ "$sha1" == "$ghprbActualCommit" ]; then
- start_message="$start_message This patch DID NOT merge cleanly! "
-else
- start_message="$start_message This patch merges cleanly. "
-fi
-start_message="$start_message<br>View progress: "
-start_message="$start_message${BUILD_URL}consoleFull"
-
-post_message "$start_message"
-
-./dev/run-tests
-test_result="$?"
-
-result_message="QA results for PR $ghprbPullId:<br>"
-
-if [ "$test_result" -eq "0" ]; then
- result_message="$result_message- This patch PASSES unit tests.<br>"
-else
- result_message="$result_message- This patch FAILED unit tests.<br>"
-fi
-
-if [ "$sha1" != "$ghprbActualCommit" ]; then
- result_message="$result_message- This patch merges cleanly<br>"
- non_test_files=$(git diff master --name-only | grep -v "\/test" | tr "\n" " ")
- new_public_classes=$(git diff master $non_test_files \
- | grep -e "trait " -e "class " \
- | grep -e "{" -e "(" \
- | grep -v -e \@\@ -e private \
- | grep \+ \
- | sed "s/\+ *//" \
- | tr "\n" "~" \
- | sed "s/~/<br>/g")
- if [ "$new_public_classes" == "" ]; then
- result_message="$result_message- This patch adds no public classes<br>"
+# check PR merge-ability and check for new public classes
+{
+ if [ "$sha1" == "$ghprbActualCommit" ]; then
+ merge_note=" * This patch **does not** merge cleanly!"
else
- result_message="$result_message- This patch adds the following public classes (experimental):<br>"
- result_message="$result_message$new_public_classes"
+ merge_note=" * This patch merges cleanly."
+
+ source_files=$(
+ git diff master --name-only \
+ | grep -v -e "\/test" `# ignore files in test directories` \
+ | grep -e "\.py$" -e "\.java$" -e "\.scala$" `# include only code files` \
+ | tr "\n" " "
+ )
+ new_public_classes=$(
+ git diff master ${source_files} `# diff this patch against master and...` \
+ | grep "^\+" `# filter in only added lines` \
+ | sed -r -e "s/^\+//g" `# remove the leading +` \
+ | grep -e "trait " -e "class " `# filter in lines with these key words` \
+ | grep -e "{" -e "(" `# filter in lines with these key words, too` \
+ | grep -v -e "\@\@" -e "private" `# exclude lines with these words` \
+ | grep -v -e "^// " -e "^/\*" -e "^ \* " `# exclude comment lines` \
+ | sed -r -e "s/\{.*//g" `# remove from the { onwards` \
+ | sed -r -e "s/\}//g" `# just in case, remove }; they mess the JSON` \
+ | sed -r -e "s/\"/\\\\\"/g" `# escape double quotes; they mess the JSON` \
+ | sed -r -e "s/^(.*)$/\`\1\`/g" `# surround with backticks for style` \
+ | sed -r -e "s/^/ \* /g" `# prepend ' *' to start of line` \
+ | sed -r -e "s/$/\\\n/g" `# append newline to end of line` \
+ | tr -d "\n" `# remove actual LF characters`
+ )
+
+ if [ "$new_public_classes" == "" ]; then
+ public_classes_note=" * This patch adds no public classes."
+ else
+ public_classes_note=" * This patch adds the following public classes _(experimental)_:"
+ public_classes_note="${public_classes_note}\n${new_public_classes}"
+ fi
fi
-fi
-result_message="${result_message}<br>For more information see test ouptut:"
-result_message="${result_message}<br>${BUILD_URL}consoleFull"
+}
-post_message "$result_message"
+# post start message
+{
+ start_message="\
+ [QA tests have started](${BUILD_URL}consoleFull) for \
+ PR $ghprbPullId at commit [\`${SHORT_COMMIT_HASH}\`](${COMMIT_URL})."
+
+ start_message="${start_message}\n${merge_note}"
+ # start_message="${start_message}\n${public_classes_note}"
+
+ post_message "$start_message"
+}
+
+# run tests
+{
+ timeout "${TESTS_TIMEOUT}" ./dev/run-tests
+ test_result="$?"
+
+ if [ "$test_result" -eq "124" ]; then
+ fail_message="**[Tests timed out](${BUILD_URL}consoleFull)** after \
+ a configured wait of \`${TESTS_TIMEOUT}\`."
+ post_message "$fail_message"
+ exit $test_result
+ else
+ if [ "$test_result" -eq "0" ]; then
+ test_result_note=" * This patch **passes** unit tests."
+ else
+ test_result_note=" * This patch **fails** unit tests."
+ fi
+ fi
+}
+
+# post end message
+{
+ result_message="\
+ [QA tests have finished](${BUILD_URL}consoleFull) for \
+ PR $ghprbPullId at commit [\`${SHORT_COMMIT_HASH}\`](${COMMIT_URL})."
+
+ result_message="${result_message}\n${test_result_note}"
+ result_message="${result_message}\n${merge_note}"
+ result_message="${result_message}\n${public_classes_note}"
+
+ post_message "$result_message"
+}
exit $test_result
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@spark.apache.org
For additional commands, e-mail: commits-help@spark.apache.org