[logging-pipelines] branch master updated: Notify on test failure (unstable builds)

[ma...@apache.org]

This is an automated email from the ASF dual-hosted git repository.

mattsicker pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/logging-pipelines.git


The following commit(s) were added to refs/heads/master by this push:
     new 178d4cf  Notify on test failure (unstable builds)
178d4cf is described below

commit 178d4cf5d155bd901ff182d861509d867c38a169
Author: Matt Sicker <bo...@gmail.com>
AuthorDate: Sun Aug 23 13:20:48 2020 -0500

    Notify on test failure (unstable builds)
    
    Now that failing tests only mark a build as unstable rather than as
    failure, we haven't been getting notified of unstable builds. This uses
    a nice email template to include contextual info about what failed.
---
 audit/Jenkinsfile               | 3 +++
 chainsaw/Jenkinsfile            | 3 +++
 kotlin/Jenkinsfile              | 3 +++
 log4j/Jenkinsfile               | 4 ++--
 vars/notifyBuildUnstable.groovy | 6 ++++++
 5 files changed, 17 insertions(+), 2 deletions(-)

diff --git a/audit/Jenkinsfile b/audit/Jenkinsfile
index 94f8c98..7cbaadd 100644
--- a/audit/Jenkinsfile
+++ b/audit/Jenkinsfile
@@ -73,6 +73,9 @@ pipeline {
                 fixed {
                     notifyBuildFixed()
                 }
+                unstable {
+                    notifyBuildUnstable()
+                }
                 failure {
                     notifyBuildFailed()
                 }
diff --git a/chainsaw/Jenkinsfile b/chainsaw/Jenkinsfile
index eda1825..c66846d 100644
--- a/chainsaw/Jenkinsfile
+++ b/chainsaw/Jenkinsfile
@@ -54,6 +54,9 @@ pipeline {
                 fixed {
                     notifyBuildFixed()
                 }
+                unstable {
+                    notifyBuildUnstable()
+                }
                 failure {
                     notifyBuildFailed()
                 }
diff --git a/kotlin/Jenkinsfile b/kotlin/Jenkinsfile
index 94f8c98..7cbaadd 100644
--- a/kotlin/Jenkinsfile
+++ b/kotlin/Jenkinsfile
@@ -73,6 +73,9 @@ pipeline {
                 fixed {
                     notifyBuildFixed()
                 }
+                unstable {
+                    notifyBuildUnstable()
+                }
                 failure {
                     notifyBuildFailed()
                 }
diff --git a/log4j/Jenkinsfile b/log4j/Jenkinsfile
index 78fe04d..9bcafcd 100644
--- a/log4j/Jenkinsfile
+++ b/log4j/Jenkinsfile
@@ -101,8 +101,8 @@ pipeline {
                 fixed {
                     notifyBuildFixed()
                 }
-                regression {
-                    notifyBuildFailed()
+                unstable {
+                    notifyBuildUnstable()
                 }
                 failure {
                     notifyBuildFailed()
diff --git a/vars/notifyBuildUnstable.groovy b/vars/notifyBuildUnstable.groovy
new file mode 100644
index 0000000..84f6cf3
--- /dev/null
+++ b/vars/notifyBuildUnstable.groovy
@@ -0,0 +1,6 @@
+def call() {
+    emailext to: 'notifications@logging.apache.org',
+        from: 'Mr. Jenkins <je...@ci-builds.apache.org>',
+        subject: "[CI][UNSTABLE] ${env.JOB_NAME}#${env.BUILD_NUMBER} has test failures",
+        body: '${SCRIPT, template="text.jelly"}'
+}