You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hbase.apache.org by ap...@apache.org on 2019/05/16 23:48:04 UTC

[hbase] branch branch-1.3 updated (9b3bbf3 -> 53b4126)

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

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


    from 9b3bbf3  HBASE-22421 addendum remove bats
     new 70e24e6  HBASE-22429 hbase-vote download step requires URL to end with '/' (#243)
     new 53b4126  HBASE-22430 hbase-vote should tee build and test output to console

The 2 revisions listed above as "new" are entirely new to this
repository and will be described in separate emails.  The revisions
listed as "add" were already present in the repository and have only
been added to this reference.


Summary of changes:
 dev-support/hbase-vote.sh | 17 +++++++++--------
 1 file changed, 9 insertions(+), 8 deletions(-)


[hbase] 02/02: HBASE-22430 hbase-vote should tee build and test output to console

Posted by ap...@apache.org.
This is an automated email from the ASF dual-hosted git repository.

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

commit 53b4126e4bb5e6420df9dd59bfa01842a7adf10e
Author: Andrew Purtell <ap...@apache.org>
AuthorDate: Thu May 16 16:45:53 2019 -0700

    HBASE-22430 hbase-vote should tee build and test output to console
---
 dev-support/hbase-vote.sh | 12 ++++++------
 1 file changed, 6 insertions(+), 6 deletions(-)

diff --git a/dev-support/hbase-vote.sh b/dev-support/hbase-vote.sh
index 0a48861..0d6c316 100755
--- a/dev-support/hbase-vote.sh
+++ b/dev-support/hbase-vote.sh
@@ -16,7 +16,7 @@
 # specific language governing permissions and limitations
 # under the License.
 
-set -e
+set -e -o pipefail
 
 usage() {
   SCRIPT=$(basename "${BASH_SOURCE[@]}")
@@ -113,7 +113,7 @@ function download_release_candidate () {
 function verify_signatures() {
     rm -f "${OUTPUT_PATH_PREFIX}"_verify_signatures
     for file in *.tar.gz; do
-        gpg --verify "${file}".asc "${file}" >> "${OUTPUT_PATH_PREFIX}"_verify_signatures 2>&1 && SIGNATURE_PASSED=1 || SIGNATURE_PASSED=0
+        gpg --verify "${file}".asc "${file}" 2>&1 | tee -a "${OUTPUT_PATH_PREFIX}"_verify_signatures && SIGNATURE_PASSED=1 || SIGNATURE_PASSED=0
     done
 }
 
@@ -122,7 +122,7 @@ function verify_checksums() {
     SHA_EXT=$(find . -name "*.sha*" | awk -F '.' '{ print $NF }' | head -n 1)
     for file in *.tar.gz; do
         gpg --print-md SHA512 "${file}" > "${file}"."${SHA_EXT}".tmp
-        diff "${file}"."${SHA_EXT}".tmp "${file}"."${SHA_EXT}" >> "${OUTPUT_PATH_PREFIX}"_verify_checksums 2>&1 && CHECKSUM_PASSED=1 || CHECKSUM_PASSED=0
+        diff "${file}"."${SHA_EXT}".tmp "${file}"."${SHA_EXT}" 2>&1 | tee -a "${OUTPUT_PATH_PREFIX}"_verify_checksums && CHECKSUM_PASSED=1 || CHECKSUM_PASSED=0
         rm -f "${file}"."${SHA_EXT}".tmp
     done
 }
@@ -134,17 +134,17 @@ function unzip_from_source() {
 
 function rat_test() {
     rm -f "${OUTPUT_PATH_PREFIX}"_rat_test
-    mvn clean apache-rat:check > "${OUTPUT_PATH_PREFIX}"_rat_test 2>&1 && RAT_CHECK_PASSED=1
+    mvn clean apache-rat:check 2>&1 | tee "${OUTPUT_PATH_PREFIX}"_rat_test && RAT_CHECK_PASSED=1
 }
 
 function build_from_source() {
     rm -f "${OUTPUT_PATH_PREFIX}"_build_from_source
-    mvn clean install -DskipTests > "${OUTPUT_PATH_PREFIX}"_build_from_source 2>&1 && BUILD_FROM_SOURCE_PASSED=1
+    mvn clean install -DskipTests 2>&1 | tee "${OUTPUT_PATH_PREFIX}"_build_from_source && BUILD_FROM_SOURCE_PASSED=1
 }
 
 function run_all_tests() {
     rm -f "${OUTPUT_PATH_PREFIX}"_run_all_tests
-    mvn test -P runAllTests > "${OUTPUT_PATH_PREFIX}"_run_all_tests 2>&1 && UNIT_TEST_PASSED=1
+    mvn test -fae -P runAllTests -Dsurefire.rerunFailingTestsCount=3 2>&1 | tee "${OUTPUT_PATH_PREFIX}"_run_all_tests && UNIT_TEST_PASSED=1
 }
 
 function execute() {


[hbase] 01/02: HBASE-22429 hbase-vote download step requires URL to end with '/' (#243)

Posted by ap...@apache.org.
This is an automated email from the ASF dual-hosted git repository.

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

commit 70e24e649db7ab32c03fcc56eacbfaac38e42599
Author: Tak Lon (Stephen) Wu <wu...@amazon.com>
AuthorDate: Thu May 16 16:43:27 2019 -0700

    HBASE-22429 hbase-vote download step requires URL to end with '/' (#243)
---
 dev-support/hbase-vote.sh | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/dev-support/hbase-vote.sh b/dev-support/hbase-vote.sh
index 4bcd680..0a48861 100755
--- a/dev-support/hbase-vote.sh
+++ b/dev-support/hbase-vote.sh
@@ -74,6 +74,7 @@ operator may still consider to verify the following manually
 3. Other concerns if any
 __EOF
 
+[[ "${SOURCE_URL}" != */ ]] && SOURCE_URL="${SOURCE_URL}/"
 HBASE_RC_VERSION=$(tr "/" "\n" <<< "${SOURCE_URL}" | tail -n2)
 HBASE_VERSION=$(echo "${HBASE_RC_VERSION}" | sed -e 's/RC[0-9]//g' | sed -e 's/hbase-//g')
 JAVA_VERSION=$(java -version 2>&1 | cut -f3 -d' ' | head -n1 | sed -e 's/"//g')
@@ -106,7 +107,7 @@ function download_and_import_keys() {
 
 function download_release_candidate () {
     # get all files from release candidate repo
-    wget --recursive --no-parent "${SOURCE_URL}"
+    wget -r -np -nH --cut-dirs 4 "${SOURCE_URL}"
 }
 
 function verify_signatures() {
@@ -169,7 +170,7 @@ __EOF
 
 download_and_import_keys
 download_release_candidate
-pushd dist.apache.org/repos/dist/dev/hbase/"${HBASE_RC_VERSION}"
+pushd "${HBASE_RC_VERSION}"
 
 execute verify_signatures
 execute verify_checksums