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/09/04 01:11:47 UTC

[hbase] branch branch-1.4 updated: HBASE-22724 Add a emoji on the vote table for pre commit result on github (#568)

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

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


The following commit(s) were added to refs/heads/branch-1.4 by this push:
     new 8b3536a  HBASE-22724 Add a emoji on the vote table for pre commit result on github (#568)
8b3536a is described below

commit 8b3536a7a06affa2655227019305e0bde8701465
Author: Duo Zhang <zh...@apache.org>
AuthorDate: Wed Sep 4 08:52:28 2019 +0800

    HBASE-22724 Add a emoji on the vote table for pre commit result on github (#568)
    
    Signed-off-by: Sean Busbey <bu...@apache.org>
    Signed-off-by: Stack <st...@apache.org>
---
 dev-support/Jenkinsfile          | 2 +-
 dev-support/Jenkinsfile_GitHub   | 7 ++++---
 dev-support/hbase-personality.sh | 4 ++++
 3 files changed, 9 insertions(+), 4 deletions(-)

diff --git a/dev-support/Jenkinsfile b/dev-support/Jenkinsfile
index 705649c..48e53d6 100644
--- a/dev-support/Jenkinsfile
+++ b/dev-support/Jenkinsfile
@@ -30,7 +30,7 @@ pipeline {
     skipDefaultCheckout()
   }
   environment {
-    YETUS_RELEASE = '0.9.0'
+    YETUS_RELEASE = '0.11.0'
     // where we'll write everything from different steps. Need a copy here so the final step can check for success/failure.
     OUTPUT_DIR_RELATIVE_GENERAL = 'output-general'
     OUTPUT_DIR_RELATIVE_JDK7 = 'output-jdk7'
diff --git a/dev-support/Jenkinsfile_GitHub b/dev-support/Jenkinsfile_GitHub
index 8bba063..0b47058 100644
--- a/dev-support/Jenkinsfile_GitHub
+++ b/dev-support/Jenkinsfile_GitHub
@@ -37,7 +37,7 @@ 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.10.0'
+        YETUS_VERSION='rel/0.11.0'
     }
 
     parameters {
@@ -151,7 +151,6 @@ pipeline {
                         YETUS_ARGS+=("--docker")
                         YETUS_ARGS+=("--dockerfile=${DOCKERFILE}")
                         YETUS_ARGS+=("--mvn-custom-repos")
-                        YETUS_ARGS+=("--jenkins")
                         YETUS_ARGS+=("--multijdkdirs=/usr/lib/jvm/java-8-openjdk-amd64")
                         YETUS_ARGS+=("--findbugs-home=/usr")
                         YETUS_ARGS+=("--whitespace-eol-ignore-list=.*/generated/.*")
@@ -159,9 +158,11 @@ pipeline {
                         YETUS_ARGS+=("--personality=${SOURCEDIR}/dev-support/hbase-personality.sh")
                         YETUS_ARGS+=("--quick-hadoopcheck")
                         # effectively treat dev-support as a custom maven module
-                        YETUS_ARGS+=("--skip-dir=dev-support")
+                        YETUS_ARGS+=("--skip-dirs=dev-support")
                         # help keep the ASF boxes clean
                         YETUS_ARGS+=("--sentinel")
+                        # use emoji vote so it is easier to find the broken line
+                        YETUS_ARGS+=("--github-use-emoji-vote")
                         "${TESTPATCHBIN}" "${YETUS_ARGS[@]}"
                         '''
                 }
diff --git a/dev-support/hbase-personality.sh b/dev-support/hbase-personality.sh
index 7294819..0e2645a 100755
--- a/dev-support/hbase-personality.sh
+++ b/dev-support/hbase-personality.sh
@@ -86,12 +86,15 @@ function personality_parse_args
   for i in "$@"; do
     case ${i} in
       --exclude-tests-url=*)
+        delete_parameter "${i}"
         EXCLUDE_TESTS_URL=${i#*=}
       ;;
       --include-tests-url=*)
+        delete_parameter "${i}"
         INCLUDE_TESTS_URL=${i#*=}
       ;;
       --hadoop-profile=*)
+        delete_parameter "${i}"
         HADOOP_PROFILE=${i#*=}
       ;;
     esac
@@ -446,6 +449,7 @@ function hadoopcheck_parse_args
   for i in "$@"; do
     case ${i} in
       --quick-hadoopcheck)
+        delete_parameter "${i}"
         QUICK_HADOOPCHECK=true
       ;;
     esac