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 2019/07/22 06:11:37 UTC

[hbase] branch branch-1.3 updated: HBASE-22719 Add debug support for github PR pre commit job

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

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


The following commit(s) were added to refs/heads/branch-1.3 by this push:
     new 4aadaf3  HBASE-22719 Add debug support for github PR pre commit job
4aadaf3 is described below

commit 4aadaf32cd527058e9b0e9a28e2696d2af3a8b8c
Author: Duo Zhang <zh...@apache.org>
AuthorDate: Mon Jul 22 08:37:45 2019 +0800

    HBASE-22719 Add debug support for github PR pre commit job
---
 dev-support/Jenkinsfile_GitHub | 21 +++++++++++----------
 1 file changed, 11 insertions(+), 10 deletions(-)

diff --git a/dev-support/Jenkinsfile_GitHub b/dev-support/Jenkinsfile_GitHub
index 987c218..4ecd3e9 100644
--- a/dev-support/Jenkinsfile_GitHub
+++ b/dev-support/Jenkinsfile_GitHub
@@ -35,13 +35,13 @@ pipeline {
         DOCKERFILE = "${SOURCEDIR}/dev-support/docker/Dockerfile"
         YETUS='yetus'
         // Branch or tag name.  Yetus release tags are 'rel/X.Y.Z'
-        YETUS_VERSION='rel/0.9.0'
+        YETUS_VERSION='rel/0.10.0'
     }
 
     parameters {
-        string(name: 'JIRA_ISSUE_KEY',
-               defaultValue: '',
-               description: 'The JIRA issue that has a patch needing pre-commit testing. Example: HBASE-1234')
+        booleanParam(name: 'DEBUG',
+               defaultValue: false,
+               description: 'Print extra outputs for debugging the jenkins job and yetus')
     }
 
     stages {
@@ -96,17 +96,18 @@ pipeline {
                         ulimit -a >"${PATCHDIR}/machine/ulimit-a" 2>&1 || true
                         ## /H*
 
-                        # if given a JIRA issue, process it. If CHANGE_URL is set
-                        # (e.g., Github Branch Source plugin), process it.
-                        # otherwise exit, because we don't want HBase to do a
+                        # If CHANGE_URL is set (e.g., Github Branch Source plugin), process it.
+                        # Otherwise exit, because we don't want HBase to do a
                         # full build.  We wouldn't normally do this check for smaller
                         # projects. :)
-                        if [[ -n "${JIRA_ISSUE_KEY}" ]]; then
-                            YETUS_ARGS+=("${JIRA_ISSUE_KEY}")
-                        elif [[ -z "${CHANGE_URL}" ]]; then
+                        if [[ -z "${CHANGE_URL}" ]]; then
                             echo "Full build skipped" > "${WORKSPACE}/${PATCHDIR}/report.html"
                             exit 0
                         fi
+                        # enable debug output for yetus
+                        if [[ "true" = "${DEBUG}" ]]; then
+                            YETUS_ARGS+=("--debug")
+                        fi
                         YETUS_ARGS+=("--patch-dir=${WORKSPACE}/${PATCHDIR}")
                         # where the source is located
                         YETUS_ARGS+=("--basedir=${WORKSPACE}/${SOURCEDIR}")