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/8f06993d Tree: http://git-wip-us.apache.org/repos/asf/hbase/tree/8f06993d Diff: http://git-wip-us.apache.org/repos/asf/hbase/diff/8f06993d Branch: refs/heads/HBASE-18147 Commit: 8f06993df515e928c33fc3a9dc0422e19cc67998 Parents: 01db60d Author: Sean Busbey <sean.bus...@gmail.com> Authored: Tue Jul 4 15:12:38 2017 -0400 Committer: Sean Busbey <bus...@apache.org> Committed: Thu Jul 20 00:37:32 2017 -0500 ---------------------------------------------------------------------- dev-support/Jenkinsfile | 269 ++++++++++++++++++++++++++++++++ dev-support/docker/Dockerfile | 29 ++++ dev-support/hbase_nightly_yetus.sh | 85 ++++++++++ 3 files changed, 383 insertions(+) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/hbase/blob/8f06993d/dev-support/Jenkinsfile ---------------------------------------------------------------------- diff --git a/dev-support/Jenkinsfile b/dev-support/Jenkinsfile new file mode 100644 index 0000000..5ff3d82 --- /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/8f06993d/dev-support/docker/Dockerfile ---------------------------------------------------------------------- diff --git a/dev-support/docker/Dockerfile b/dev-support/docker/Dockerfile index 2ecc42e..c654ded 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 \ @@ -126,6 +127,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/8f06993d/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..8b56bc7 --- /dev/null +++ b/dev-support/hbase_nightly_yetus.sh @@ -0,0 +1,85 @@ +#!/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[@]}"