commit aff1c05ed78e52f41c8b1e7c03b3c57f08368130
author Matt Kwong <mattkwong@google.com> Thu Mar 09 15:08:01 2017 -0800
committer Matt Kwong <mattkwong@google.com> Thu Mar 16 17:04:33 2017 -0700
tree 854974c3366ba2239253f431d9a05940252ac1f5
parent 4166dc3536986c5295131a6e7c5920baaae487b3

Make Jenkins post microbenchmarking diff in a comment

tools/jenkins/comment_on_pr.sh

diff --git a/tools/jenkins/comment_on_pr.sh b/tools/jenkins/comment_on_pr.sh
old mode 100644
new mode 100755
index 79c54c2..85f33aa
--- a/tools/jenkins/comment_on_pr.sh
+++ b/tools/jenkins/comment_on_pr.sh
@@ -33,7 +33,7 @@
 
 set -e
 
-if [ -z $1 ] || [ -z $JENKINS_OAUTH_TOKEN ] || [ -z $ghprbPullId ]; then
+if [ -z "$1" ] || [ -z $JENKINS_OAUTH_TOKEN ] || [ -z $ghprbPullId ]; then
   echo "Insufficient arguments or environment variables provided."
   exit 1
 fi

tools/run_tests/run_microbenchmark.py

diff --git a/tools/run_tests/run_microbenchmark.py b/tools/run_tests/run_microbenchmark.py
index 57b2636..12d9815 100755
--- a/tools/run_tests/run_microbenchmark.py
+++ b/tools/run_tests/run_microbenchmark.py
@@ -229,6 +229,8 @@
                   type=int,
                   help='Minimum time to run benchmarks for the summary collection')
 args = argp.parse_args()
+if args.diff_perf:
+  git_comment = ''
 
 try:
   for collect in args.collect:
@@ -262,7 +264,14 @@
       if diff:
         heading('Performance diff: %s' % bm_name)
         text(diff)
+        git_comment += '```\\nPerformance diff: %s\\n%s\\n```\\n' % (bm_name, diff.replace('\n', '\\n'))
 finally:
+  if args.diff_perf:
+    subprocess.call(['tools/jenkins/comment_on_pr.sh "%s"' % git_comment.replace('`', '\`')],
+                    stdout=subprocess.PIPE,
+                    shell=True)
+  if not os.path.exists('reports'):
+    os.makedirs('reports')
   index_html += "</body>\n</html>\n"
   with open('reports/index.html', 'w') as f:
     f.write(index_html)