You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hbase.apache.org by bu...@apache.org on 2017/07/20 06:05:35 UTC
[2/4] hbase git commit: HBASE-18147 POC jenkinsfile for nightly
checks.
HBASE-18147 POC jenkinsfile for nightly checks.
* Jenkinsfile that works for all current branches.
* adds dev-support script for setting parameters for our yetus nightly invocation
* adds ruby tools to dockerfile
* adds rubocop to dockerfile
* adds ruby-lint to dockerfile
* adds perlcritic to dockerfile
Project: http://git-wip-us.apache.org/repos/asf/hbase/repo
Commit: http://git-wip-us.apache.org/repos/asf/hbase/commit/ac0543f4
Tree: http://git-wip-us.apache.org/repos/asf/hbase/tree/ac0543f4
Diff: http://git-wip-us.apache.org/repos/asf/hbase/diff/ac0543f4
Branch: refs/heads/branch-1-HBASE-18147
Commit: ac0543f4aab4815baafcd3cda676e281365c1534
Parents: cfd5b6b
Author: Sean Busbey <bu...@apache.org>
Authored: Thu Jul 20 01:01:59 2017 -0500
Committer: Sean Busbey <bu...@apache.org>
Committed: Thu Jul 20 01:03:13 2017 -0500
----------------------------------------------------------------------
dev-support/Jenkinsfile | 269 ++++++++++++++++++++++++++++++++
dev-support/docker/Dockerfile | 29 ++++
dev-support/hbase_nightly_yetus.sh | 86 ++++++++++
3 files changed, 384 insertions(+)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/hbase/blob/ac0543f4/dev-support/Jenkinsfile
----------------------------------------------------------------------
diff --git a/dev-support/Jenkinsfile b/dev-support/Jenkinsfile
new file mode 100644
index 0000000..d4aed6f
--- /dev/null
+++ b/dev-support/Jenkinsfile
@@ -0,0 +1,269 @@
+// Licensed to the Apache Software Foundation (ASF) under one
+// or more contributor license agreements. See the NOTICE file
+// distributed with this work for additional information
+// regarding copyright ownership. The ASF licenses this file
+// to you under the Apache License, Version 2.0 (the
+// "License"); you may not use this file except in compliance
+// with the License. You may obtain a copy of the License at
+//
+// http://www.apache.org/licenses/LICENSE-2.0
+//
+// Unless required by applicable law or agreed to in writing,
+// software distributed under the License is distributed on an
+// "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+// KIND, either express or implied. See the License for the
+// specific language governing permissions and limitations
+// under the License.
+pipeline {
+ agent {
+ node {
+ label 'Hadoop'
+ }
+ }
+ triggers {
+ cron('@daily')
+ }
+ options {
+ buildDiscarder(logRotator(numToKeepStr: '30'))
+ timeout (time: 6, unit: 'HOURS')
+ timestamps()
+ }
+ environment {
+ TOOLS = "${env.WORKSPACE}/tools"
+ // where we check out to across stages
+ BASEDIR = "${env.WORKSPACE}/component"
+ YETUS_RELEASE = '0.5.0'
+ // where we'll write everything from different steps.
+ OUTPUT_RELATIVE_GENERAL = 'output-general'
+ OUTPUTDIR_GENERAL = "${env.WORKSPACE}/output-general"
+ OUTPUT_RELATIVE_JDK7 = 'output-jdk7'
+ OUTPUTDIR_JDK7 = "${env.WORKSPACE}/output-jdk7"
+ OUTPUT_RELATIVE_JDK8 = 'output-jdk8'
+ OUTPUTDIR_JDK8 = "${env.WORKSPACE}/output-jdk8"
+ PROJECT = 'hbase'
+ PROJET_PERSONALITY = 'https://raw.githubusercontent.com/apache/hbase/master/dev-support/hbase-personality.sh'
+ // This section of the docs tells folks not to use the javadoc tag. older branches have our old version of the check for said tag.
+ AUTHOR_IGNORE_LIST = 'src/main/asciidoc/_chapters/developer.adoc,dev-support/test-patch.sh'
+ WHITESPACE_IGNORE_LIST = '.*/generated/.*'
+ // output from surefire; sadly the archive function in yetus only works on file names.
+ ARCHIVE_PATTERN_LIST = 'TEST-*.xml,org.apache.h*-output.txt,org.apache.h*.txt'
+ // These tests currently have known failures. Once they burn down to 0, remove from here so that new problems will cause a failure.
+ TESTS_FILTER = 'cc,checkstyle,javac,javadoc,pylint,shellcheck,whitespace,perlcritic,ruby-lint,rubocop'
+ BRANCH_SPECIFIC_DOCKERFILE = "${env.BASEDIR}/dev-support/docker/Dockerfile"
+ EXCLUDE_TESTS_URL = 'https://builds.apache.org/job/HBase-Find-Flaky-Tests/lastSuccessfulBuild/artifact/excludes/'
+ }
+ parameters {
+ booleanParam(name: 'USE_YETUS_PRERELEASE', defaultValue: false, description: '''Check to use the current HEAD of apache/yetus rather than our configured release.
+
+ Should only be used manually when e.g. there is some non-work-aroundable issue in yetus we are checking a fix for.''')
+ booleanParam(name: 'DEBUG', defaultValue: false, description: 'Produce a lot more meta-information.')
+ }
+ stages {
+ stage ('yetus install') {
+ steps {
+ sh '''#!/usr/bin/env bash
+echo "Ensure we have a copy of Apache Yetus."
+if [[ true != "${USE_YETUS_PRERELEASE}" ]]; then
+ YETUS_DIR="${WORKSPACE}/yetus-${YETUS_RELEASE}"
+ echo "Checking for Yetus ${YETUS_RELEASE} in '${YETUS_DIR}'"
+ if [ ! -d "${YETUS_DIR}" ]; then
+ echo "New download of Apache Yetus version ${YETUS_RELEASE}."
+ rm -rf "${WORKSPACE}/.gpg"
+ mkdir -p "${WORKSPACE}/.gpg"
+ chmod -R 700 "${WORKSPACE}/.gpg"
+
+ echo "install yetus project KEYS"
+ curl -L --fail -o "${WORKSPACE}/KEYS_YETUS" https://dist.apache.org/repos/dist/release/yetus/KEYS
+ gpg --homedir "${WORKSPACE}/.gpg" --import "${WORKSPACE}/KEYS_YETUS"
+
+ echo "download yetus release ${YETUS_RELEASE}"
+ curl -L --fail -O "https://dist.apache.org/repos/dist/release/yetus/${YETUS_RELEASE}/yetus-${YETUS_RELEASE}-bin.tar.gz"
+ curl -L --fail -O "https://dist.apache.org/repos/dist/release/yetus/${YETUS_RELEASE}/yetus-${YETUS_RELEASE}-bin.tar.gz.asc"
+ echo "verifying yetus release"
+ gpg --homedir "${WORKSPACE}/.gpg" --verify "yetus-${YETUS_RELEASE}-bin.tar.gz.asc"
+ mv "yetus-${YETUS_RELEASE}-bin.tar.gz" yetus.tar.gz
+ else
+ echo "Reusing cached download of Apache Yetus version ${YETUS_RELEASE}."
+ fi
+else
+ YETUS_DIR="${WORKSPACE}/yetus-git"
+ rm -rf "${YETUS_DIR}"
+ echo "downloading from github"
+ curl -L --fail https://api.github.com/repos/apache/yetus/tarball/HEAD -o yetus.tar.gz
+fi
+if [ ! -d "${YETUS_DIR}" ]; then
+ echo "unpacking yetus into '${YETUS_DIR}'"
+ mkdir -p "${YETUS_DIR}"
+ gunzip -c yetus.tar.gz | tar xpf - -C "${YETUS_DIR}" --strip-components 1
+fi
+ '''
+ dir ("${env.TOOLS}") {
+ sh """#!/usr/bin/env bash
+echo "Downloading Project personality."
+curl -L -o personality.sh "${env.PROJET_PERSONALITY}"
+ """
+ }
+ stash name: 'yetus'
+ }
+ }
+ stage ('yetus general check') {
+ environment {
+ // Should be things that work with multijdk
+ TESTS = 'all,-unit,-findbugs'
+ // on branches that don't support jdk7, this will already be JAVA_HOME, so we'll end up not
+ // doing multijdk there.
+ MULTIJDK = '/usr/lib/jvm/java-8-oracle'
+ OUTPUT_RELATIVE = "${env.OUTPUT_RELATIVE_GENERAL}"
+ OUTPUTDIR = "${env.OUTPUTDIR_GENERAL}"
+ }
+ steps {
+ checkout scm
+ unstash 'yetus'
+ // TODO should this be a download from master, similar to how the personality is?
+ sh "${env.BASEDIR}/dev-support/hbase_nightly_yetus.sh"
+ }
+ post {
+ always {
+ junit 'output-general/**/target/**/TEST-*.xml'
+ // gzip surefire reports.
+ sh '''#!/bin/bash -e
+ if [ -d "${OUTPUTDIR}/archiver" ]; then
+ count=$(find "${OUTPUTDIR}/archiver" -type f | wc -l)
+ if [[ 0 -ne ${count} ]]; then
+ echo "zipping ${count} archived files"
+ zip -m -r "${OUTPUTDIR}/test_logs.zip" "${OUTPUTDIR}/archiver"
+ else
+ echo "No archived files, skipping compressing."
+ fi
+ else
+ echo "No archiver directory, skipping compressing."
+ fi
+'''
+ // env variables don't work in archive? or this has to be relative to WORKSPACE. :(
+ archive 'output-general/*'
+ archive 'output-general/**/*'
+ publishHTML target: [
+ allowMissing: true,
+ keepAll: true,
+ alwaysLinkToLastBuild: true,
+ // has to be relative to WORKSPACE :(
+ reportDir: 'output-general',
+ reportFiles: 'console-report.html',
+ reportName: 'Nightly Build Report'
+ ]
+ }
+ }
+ }
+ stage ('yetus jdk7 checks') {
+ when {
+ branch 'branch-1*'
+ }
+ environment {
+ TESTS = 'mvninstall,unit'
+ OUTPUT_RELATIVE = "${env.OUTPUT_RELATIVE_JDK7}"
+ OUTPUTDIR = "${env.OUTPUTDIR_JDK7}"
+ // On branches where we do jdk7 checks, jdk7 will be JAVA_HOME already.
+ }
+ steps {
+ // TODO should we be doing a single checkout and then stashing that checkout?
+ checkout scm
+ unstash 'yetus'
+ sh """#!/usr/bin/env bash
+ # for branch-1.1 we don't do jdk8 findbugs, so do it here
+ if [ "${env.BRANCH_NAME}" == "branch-1.1" ]; then
+ TESTS+=",findbugs"
+ fi
+ ${env.BASEDIR}/dev-support/hbase_nightly_yetus.sh"
+ """
+ }
+ post {
+ always {
+ junit 'output-jdk7/**/target/**/TEST-*.xml'
+ // gzip surefire reports.
+ sh '''#!/bin/bash -e
+ if [ -d "${OUTPUTDIR}/archiver" ]; then
+ count=$(find "${OUTPUTDIR}/archiver" -type f | wc -l)
+ if [[ 0 -ne ${count} ]]; then
+ echo "zipping ${count} archived files"
+ zip -m -r "${OUTPUTDIR}/test_logs.zip" "${OUTPUTDIR}/archiver"
+ else
+ echo "No archived files, skipping compressing."
+ fi
+ else
+ echo "No archiver directory, skipping compressing."
+ fi
+'''
+ // env variables don't work in archive? or this has to be relative to WORKSPACE. :(
+ archive 'output-jdk7/*'
+ archive 'output-jdk7/**/*'
+ publishHTML target: [
+ allowMissing: true,
+ keepAll: true,
+ alwaysLinkToLastBuild: true,
+ // has to be relative to WORKSPACE :(
+ reportDir: 'output-jdk7',
+ reportFiles: 'console-report.html',
+ reportName: 'Nightly Build Report'
+ ]
+ }
+ }
+ }
+ stage ('yetus jdk8 checks') {
+ when {
+ not {
+ branch 'branch-1.1'
+ }
+ }
+ environment {
+ TESTS = 'mvninstall,unit,findbugs'
+ OUTPUT_RELATIVE = "${env.OUTPUT_RELATIVE_JDK8}"
+ OUTPUTDIR = "${env.OUTPUTDIR_JDK8}"
+ // This isn't strictly needed on branches that only support jdk8, but doesn't hurt
+ // and is needed on branches that do both jdk7 and jdk8
+ SET_JAVA_HOME = '/usr/lib/jvm/java-8-oracle'
+ }
+ steps {
+ checkout scm
+ unstash 'yetus'
+ sh "${env.BASEDIR}/dev-support/hbase_nightly_yetus.sh"
+ }
+ post {
+ always {
+ junit 'output-jdk8/**/target/**/TEST-*.xml'
+ // gzip surefire reports.
+ sh '''#!/bin/bash -e
+ if [ -d "${OUTPUTDIR}/archiver" ]; then
+ count=$(find "${OUTPUTDIR}/archiver" -type f | wc -l)
+ if [[ 0 -ne ${count} ]]; then
+ echo "zipping ${count} archived files"
+ zip -m -r "${OUTPUTDIR}/test_logs.zip" "${OUTPUTDIR}/archiver"
+ else
+ echo "No archived files, skipping compressing."
+ fi
+ else
+ echo "No archiver directory, skipping compressing."
+ fi
+'''
+ // env variables don't work in archive? or this has to be relative to WORKSPACE. :(
+ archive 'output-jdk8/*'
+ archive 'output-jdk8/**/*'
+ publishHTML target: [
+ allowMissing: true,
+ keepAll: true,
+ alwaysLinkToLastBuild: true,
+ // has to be relative to WORKSPACE :(
+ reportDir: 'output-jdk8',
+ reportFiles: 'console-report.html',
+ reportName: 'Nightly Build Report'
+ ]
+ }
+ }
+ }
+ }
+ post {
+ failure {
+ deleteDir()
+ }
+ }
+}
+
http://git-wip-us.apache.org/repos/asf/hbase/blob/ac0543f4/dev-support/docker/Dockerfile
----------------------------------------------------------------------
diff --git a/dev-support/docker/Dockerfile b/dev-support/docker/Dockerfile
index e83600f..9515418 100644
--- a/dev-support/docker/Dockerfile
+++ b/dev-support/docker/Dockerfile
@@ -47,6 +47,7 @@ RUN apt-get -q update && apt-get -q install --no-install-recommends -y \
libbz2-dev \
libcurl4-openssl-dev \
libfuse-dev \
+ libperl-critic-perl \
libprotobuf-dev \
libprotoc-dev \
libsnappy-dev \
@@ -127,6 +128,34 @@ RUN pip install pylint
####
RUN pip install python-dateutil
+####
+# Install Ruby 2, based on Yetus 0.4.0 dockerfile
+###
+RUN echo 'gem: --no-rdoc --no-ri' >> /root/.gemrc
+RUN apt-get -q install -y ruby2.0
+#
+# on trusty, the above installs ruby2.0 and ruby (1.9.3) exes
+# but update-alternatives is broken, so we need to do some work
+# to make 2.0 actually the default without the system flipping out
+#
+# See https://bugs.launchpad.net/ubuntu/+source/ruby2.0/+bug/1310292
+#
+RUN dpkg-divert --add --rename --divert /usr/bin/ruby.divert /usr/bin/ruby
+RUN dpkg-divert --add --rename --divert /usr/bin/gem.divert /usr/bin/gemrc
+RUN update-alternatives --install /usr/bin/ruby ruby /usr/bin/ruby2.0 1
+RUN update-alternatives --install /usr/bin/gem gem /usr/bin/gem2.0 1
+
+
+####
+# Install rubocop
+###
+RUN gem install rubocop
+
+####
+# Install ruby-lint
+###
+RUN gem install ruby-lint
+
###
# Avoid out of memory errors in builds
###
http://git-wip-us.apache.org/repos/asf/hbase/blob/ac0543f4/dev-support/hbase_nightly_yetus.sh
----------------------------------------------------------------------
diff --git a/dev-support/hbase_nightly_yetus.sh b/dev-support/hbase_nightly_yetus.sh
new file mode 100755
index 0000000..007d64a
--- /dev/null
+++ b/dev-support/hbase_nightly_yetus.sh
@@ -0,0 +1,86 @@
+#!/usr/bin/env bash
+# Licensed to the Apache Software Foundation (ASF) under one
+# or more contributor license agreements. See the NOTICE file
+# distributed with this work for additional information
+# regarding copyright ownership. The ASF licenses this file
+# to you under the Apache License, Version 2.0 (the
+# "License"); you may not use this file except in compliance
+# with the License. You may obtain a copy of the License at
+#
+# http://www.apache.org/licenses/LICENSE-2.0
+#
+# Unless required by applicable law or agreed to in writing,
+# software distributed under the License is distributed on an
+# "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+# KIND, either express or implied. See the License for the
+# specific language governing permissions and limitations
+# under the License.
+
+declare -i missing_env=0
+# Validate params
+for required_env in "TESTS" "TOOLS" "BASEDIR" "ARCHIVE_PATTERN_LIST" "OUTPUT_RELATIVE" \
+ "BRANCH_SPECIFIC_DOCKERFILE" "OUTPUTDIR" "PROJECT" "AUTHOR_IGNORE_LIST" \
+ "WHITESPACE_IGNORE_LIST" "BRANCH_NAME" "TESTS_FILTER" "DEBUG" \
+ "USE_YETUS_PRERELEASE" "WORKSPACE" "YETUS_RELEASE"; do
+ if [ -z "${!required_env}" ]; then
+ echo "[ERROR] Required environment variable '${required_env}' is not set."
+ missing_env=${missing_env}+1
+ fi
+done
+
+if [ ${missing_env} -gt 0 ]; then
+ echo "[ERROR] Please set the required environment variables before invoking. If this error is " \
+ "on Jenkins, then please file a JIRA about the error."
+ exit 1
+fi
+
+YETUS_ARGS=()
+if [[ -n "${MULTIJDK}" ]]; then
+ YETUS_ARGS=("--multijdktests=compile,javadoc" "${YETUS_ARGS[@]}")
+ YETUS_ARGS=("--multijdkdirs=${MULTIJDK}" "${YETUS_ARGS[@]}")
+fi
+
+if [[ -n "${SET_JAVA_HOME}" ]]; then
+ YETUS_ARGS=("--java-home=${SET_JAVA_HOME}" "${YETUS_ARGS[@]}")
+fi
+YETUS_ARGS=("--plugins=${TESTS}" "${YETUS_ARGS[@]}")
+YETUS_ARGS=("--personality=${TOOLS}/personality.sh" "${YETUS_ARGS[@]}")
+YETUS_ARGS=("--basedir=${BASEDIR}" "${YETUS_ARGS[@]}")
+YETUS_ARGS=("--archive-list=${ARCHIVE_PATTERN_LIST}" "${YETUS_ARGS[@]}")
+YETUS_ARGS=("--console-urls" "${YETUS_ARGS[@]}")
+# YETUS-532, repeat this twice in case the fix is to update args rather than docs
+YETUS_ARGS=("--build-url-patchdir=artifact/${OUTPUT_RELATIVE}" "${YETUS_ARGS[@]}")
+YETUS_ARGS=("--build-url-artifacts=artifact/${OUTPUT_RELATIVE}" "${YETUS_ARGS[@]}")
+YETUS_ARGS=("--docker" "${YETUS_ARGS[@]}")
+YETUS_ARGS=("--dockerfile=${BRANCH_SPECIFIC_DOCKERFILE}" "${YETUS_ARGS[@]}")
+YETUS_ARGS=("--empty-patch" "${YETUS_ARGS[@]}")
+YETUS_ARGS=("--html-report-file=${OUTPUTDIR}/console-report.html" "${YETUS_ARGS[@]}")
+YETUS_ARGS=("--jenkins" "${YETUS_ARGS[@]}")
+YETUS_ARGS=("--mvn-custom-repos" "${YETUS_ARGS[@]}")
+YETUS_ARGS=("--patch-dir=${OUTPUTDIR}" "${YETUS_ARGS[@]}")
+YETUS_ARGS=("--project=${PROJECT}" "${YETUS_ARGS[@]}")
+YETUS_ARGS=("--resetrepo" "${YETUS_ARGS[@]}")
+YETUS_ARGS=("--author-ignore-list=${AUTHOR_IGNORE_LIST}" "${YETUS_ARGS[@]}")
+YETUS_ARGS=("--whitespace-eol-ignore-list=${WHITESPACE_IGNORE_LIST}" "${YETUS_ARGS[@]}")
+YETUS_ARGS=("--whitespace-tabs-ignore-list=${WHITESPACE_IGNORE_LIST}" "${YETUS_ARGS[@]}")
+YETUS_ARGS=("--sentinel" "${YETUS_ARGS[@]}")
+YETUS_ARGS=("--branch=${BRANCH_NAME}" "${YETUS_ARGS[@]}")
+YETUS_ARGS=("--tests-filter=${TESTS_FILTER}" "${YETUS_ARGS[@]}")
+
+if [[ true == "${DEBUG}" ]]; then
+ YETUS_ARGS=("--debug" "${YETUS_ARGS[@]}")
+fi
+
+rm -rf "${OUTPUTDIR}"
+mkdir -p "${OUTPUTDIR}"
+if [[ true != "${USE_YETUS_PRERELEASE}" ]]; then
+ YETUS_ARGS=("--shelldocs=${WORKSPACE}/yetus-${YETUS_RELEASE}/bin/shelldocs" "${YETUS_ARGS[@]}")
+ TESTPATCHBIN="${WORKSPACE}/yetus-${YETUS_RELEASE}/bin/test-patch"
+else
+ YETUS_ARGS=("--shelldocs=${WORKSPACE}/yetus-git/shelldocs/shelldocs.py" "${YETUS_ARGS[@]}")
+ TESTPATCHBIN="${WORKSPACE}/yetus-git/precommit/test-patch.sh"
+fi
+echo "Launching yetus with command line:"
+echo "${TESTPATCHBIN} ${YETUS_ARGS[*]}"
+
+/usr/bin/env bash "${TESTPATCHBIN}" "${YETUS_ARGS[@]}"