You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@kudu.apache.org by to...@apache.org on 2016/02/02 07:55:29 UTC
[6/9] incubator-kudu git commit: Do not report tests that fail due to
NTP issues
Do not report tests that fail due to NTP issues
Occasionally we have tests that fail due to machines with broken
NTP synchronization. In those cases, we don't want to report them
to the flaky test infrastructure, since it inflates our flakiness
count on every test which uses NTP.
Change-Id: Ia103868de0e6fd3ef95ced72035de7f67ccfb8db
Reviewed-on: http://gerrit.cloudera.org:8080/1980
Reviewed-by: Jean-Daniel Cryans
Tested-by: Todd Lipcon <to...@apache.org>
Project: http://git-wip-us.apache.org/repos/asf/incubator-kudu/repo
Commit: http://git-wip-us.apache.org/repos/asf/incubator-kudu/commit/64bd51c5
Tree: http://git-wip-us.apache.org/repos/asf/incubator-kudu/tree/64bd51c5
Diff: http://git-wip-us.apache.org/repos/asf/incubator-kudu/diff/64bd51c5
Branch: refs/heads/master
Commit: 64bd51c5795ac9726734f29b4060e9fae9dd2b72
Parents: cecdb47
Author: Todd Lipcon <to...@apache.org>
Authored: Mon Feb 1 15:28:32 2016 -0800
Committer: Todd Lipcon <to...@apache.org>
Committed: Tue Feb 2 02:00:19 2016 +0000
----------------------------------------------------------------------
build-support/report-test.sh | 7 +++++++
1 file changed, 7 insertions(+)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/incubator-kudu/blob/64bd51c5/build-support/report-test.sh
----------------------------------------------------------------------
diff --git a/build-support/report-test.sh b/build-support/report-test.sh
index 34872c3..b0d9426 100755
--- a/build-support/report-test.sh
+++ b/build-support/report-test.sh
@@ -73,6 +73,13 @@ if [ -n "$HEAPCHECK" ]; then
BUILD_CONFIG="$BUILD_CONFIG heapcheck"
fi
+# We sometimes have flaky infrastructure where NTP is broken. In that case
+# do not report it as a failed test.
+if grep -q 'Clock considered unsynchronized' $LOGFILE ; then
+ echo Not reporting test that failed due to NTP issues.
+ exit 1
+fi
+
# Only upload a log if the test failed.
# This saves some space on S3, network bandwidth, etc, and we don't
# have a lot of use for the logs of successful tests anyway.