You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hbase.apache.org by zh...@apache.org on 2021/10/08 04:05:59 UTC

[hbase] 01/01: HBASE-26339 SshPublisher will skip uploading artifacts if the build is failure

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

zhangduo pushed a commit to branch HBASE-26339
in repository https://gitbox.apache.org/repos/asf/hbase.git

commit a01573e03f6531d07c6e16f81cbdad36b1078a00
Author: Duo Zhang <zh...@apache.org>
AuthorDate: Fri Oct 8 12:04:12 2021 +0800

    HBASE-26339 SshPublisher will skip uploading artifacts if the build is failure
---
 dev-support/Jenkinsfile_GitHub | 39 +++++++++++++++++++++++++++++++++------
 1 file changed, 33 insertions(+), 6 deletions(-)

diff --git a/dev-support/Jenkinsfile_GitHub b/dev-support/Jenkinsfile_GitHub
index 56ba855..888fb9e 100644
--- a/dev-support/Jenkinsfile_GitHub
+++ b/dev-support/Jenkinsfile_GitHub
@@ -102,12 +102,21 @@ pipeline {
                                   passwordVariable: 'GITHUB_PASSWORD',
                                   usernameVariable: 'GITHUB_USER'
                                 )]) {
-                                sh label: 'test-patch', script: '''#!/bin/bash -e
+                                def ret = sh(
+                                  label: 'test-patch',
+                                  returnStatus: true,
+                                  script: '''#!/bin/bash -e
                                     hostname -a ; pwd ; ls -la
                                     printenv 2>&1 | sort
                                     echo "[INFO] Launching Yetus via ${YETUS_DRIVER}"
                                     "${YETUS_DRIVER}"
-                                '''
+                                  '''
+                                )
+                                if (ret != 0) {
+                                  // mark the build as UNSTABLE instead of FAILURE, to avoid skipping the later publish of
+                                  // test output. See HBASE-26339 for more details.
+                                  currentBuild.result = 'UNSTABLE'
+                                }
                             }
                         }
                     }
@@ -189,12 +198,21 @@ pipeline {
                                 passwordVariable: 'GITHUB_PASSWORD',
                                 usernameVariable: 'GITHUB_USER'
                               )]) {
-                                sh label: 'test-patch', script: '''#!/bin/bash -e
+                                def ret = sh(
+                                  label: 'test-patch',
+                                  returnStatus: true,
+                                  script: '''#!/bin/bash -e
                                     hostname -a ; pwd ; ls -la
                                     printenv 2>&1 | sort
                                     echo "[INFO] Launching Yetus via ${YETUS_DRIVER}"
                                     "${YETUS_DRIVER}"
-                                '''
+                                  '''
+                                )
+                                if (ret != 0) {
+                                  // mark the build as UNSTABLE instead of FAILURE, to avoid skipping the later publish of
+                                  // test output. See HBASE-26339 for more details.
+                                  currentBuild.result = 'UNSTABLE'
+                                }
                             }
                         }
                     }
@@ -309,12 +327,21 @@ pipeline {
                                 passwordVariable: 'GITHUB_PASSWORD',
                                 usernameVariable: 'GITHUB_USER'
                               )]) {
-                                sh label: 'test-patch', script: '''#!/bin/bash -e
+                                def ret = sh(
+                                  label: 'test-patch',
+                                  returnStatus: true,
+                                  script: '''#!/bin/bash -e
                                     hostname -a ; pwd ; ls -la
                                     printenv 2>&1 | sort
                                     echo "[INFO] Launching Yetus via ${YETUS_DRIVER}"
                                     "${YETUS_DRIVER}"
-                                '''
+                                  '''
+                                )
+                                if (ret != 0) {
+                                  // mark the build as UNSTABLE instead of FAILURE, to avoid skipping the later publish of
+                                  // test output. See HBASE-26339 for more details.
+                                  currentBuild.result = 'UNSTABLE'
+                                }
                             }
                         }
                     }