From 08510b608e01ee82a08b47b2e18ff32d15c773aa Mon Sep 17 00:00:00 2001
From: Spencer <email@spalger.com>
Date: Fri, 12 Jun 2020 16:37:38 -0700
Subject: [PATCH] include ci-stats metrics in pr comment (#68563)

Co-authored-by: spalger <spalger@users.noreply.github.com>
Co-authored-by: Elastic Machine <elasticmachine@users.noreply.github.com>
---
 Jenkinsfile          | 12 ++++++------
 vars/githubPr.groovy |  1 +
 2 files changed, 7 insertions(+), 6 deletions(-)

diff --git a/Jenkinsfile b/Jenkinsfile
index b6a36c79f877d..763ee95ddde99 100644
--- a/Jenkinsfile
+++ b/Jenkinsfile
@@ -4,8 +4,8 @@ library 'kibana-pipeline-library'
 kibanaLibrary.load()
 
 kibanaPipeline(timeoutMinutes: 155, checkPrChanges: true) {
-  ciStats.trackBuild {
-    githubPr.withDefaultPrComments {
+  githubPr.withDefaultPrComments {
+    ciStats.trackBuild {
       catchError {
         retryable.enable()
         parallel([
@@ -53,10 +53,10 @@ kibanaPipeline(timeoutMinutes: 155, checkPrChanges: true) {
         ])
       }
     }
+  }
 
-    if (params.NOTIFY_ON_FAILURE) {
-      slackNotifications.onFailure()
-      kibanaPipeline.sendMail()
-    }
+  if (params.NOTIFY_ON_FAILURE) {
+    slackNotifications.onFailure()
+    kibanaPipeline.sendMail()
   }
 }
diff --git a/vars/githubPr.groovy b/vars/githubPr.groovy
index 965fb1d4e108e..9dad2f88de2ec 100644
--- a/vars/githubPr.groovy
+++ b/vars/githubPr.groovy
@@ -186,6 +186,7 @@ def getNextCommentMessage(previousCommentInfo = [:]) {
   }
 
   messages << getTestFailuresMessage()
+  messages << ciStats.getMetricsReport()
 
   if (info.builds && info.builds.size() > 0) {
     messages << getHistoryText(info.builds)