You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@kudu.apache.org by al...@apache.org on 2020/01/28 18:01:56 UTC
[kudu] branch master updated (3ba5ec5 -> 6b7355d)
This is an automated email from the ASF dual-hosted git repository.
alexey pushed a change to branch master
in repository https://gitbox.apache.org/repos/asf/kudu.git.
from 3ba5ec5 [tests] simpler wait-for-crash in TestFillMultipleDisks
new d014360 [tests] fix flakiness in RaftConsensusNonVoterITest
new 6b7355d [docs] curl in the list of packages for build-from-source
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:
docs/installation.adoc | 2 +-
src/kudu/integration-tests/raft_consensus_nonvoter-itest.cc | 9 +++++++--
2 files changed, 8 insertions(+), 3 deletions(-)
[kudu] 02/02: [docs] curl in the list of packages for
build-from-source
Posted by al...@apache.org.
This is an automated email from the ASF dual-hosted git repository.
alexey pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/kudu.git
commit 6b7355d12c1ce10eb1e361003ebd6b87ce2d7009
Author: Alexey Serbin <al...@apache.org>
AuthorDate: Mon Jan 27 23:41:18 2020 -0800
[docs] curl in the list of packages for build-from-source
Added curl into the list of packages necessary to build Kudu from source
on RHEL/CentOS. For some reason, it was omitted in that list while
present in package list of other Linux flavors.
Change-Id: I61b80750e4ea2fbcfdd578e0c5aaa00802f632f6
Reviewed-on: http://gerrit.cloudera.org:8080/15114
Tested-by: Kudu Jenkins
Reviewed-by: Adar Dembo <ad...@cloudera.com>
---
docs/installation.adoc | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/docs/installation.adoc b/docs/installation.adoc
index 3497501..08c5c34 100644
--- a/docs/installation.adoc
+++ b/docs/installation.adoc
@@ -225,7 +225,7 @@ automated deployment scenario. It skips the steps marked *Optional* above.
----
#!/bin/bash
-sudo yum -y install autoconf automake cyrus-sasl-devel cyrus-sasl-gssapi \
+sudo yum -y install autoconf automake curl cyrus-sasl-devel cyrus-sasl-gssapi \
cyrus-sasl-plain flex gcc gcc-c++ gdb git java-1.8.0-openjdk-devel \
krb5-server krb5-workstation libtool make openssl-devel patch pkgconfig \
redhat-lsb-core rsync unzip vim-common which
[kudu] 01/02: [tests] fix flakiness in RaftConsensusNonVoterITest
Posted by al...@apache.org.
This is an automated email from the ASF dual-hosted git repository.
alexey pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/kudu.git
commit d014360c66cf3b35708c0eb794ade9f454dcf687
Author: Alexey Serbin <al...@apache.org>
AuthorDate: Tue Jan 28 00:27:43 2020 -0800
[tests] fix flakiness in RaftConsensusNonVoterITest
Fixed flakiness in PromoteAndDemote scenario of the
RaftConsensusNonVoterITest test.
Due to the short heartbeat interval and concurrent activity, sometimes
it takes a bit more time for a tserver to respond to
RequestConsensusVote call than the Raft timeout.
Prior to this fix, the scenario failed for ASAN builds in rate of
1 in ~100 runs. With the fix, I could not see it failing after
multiple 1K runs.
Change-Id: Ia76e4ea57f2f9d4251ad3cbed47267266fee7e28
Reviewed-on: http://gerrit.cloudera.org:8080/15115
Tested-by: Kudu Jenkins
Reviewed-by: Adar Dembo <ad...@cloudera.com>
---
src/kudu/integration-tests/raft_consensus_nonvoter-itest.cc | 9 +++++++--
1 file changed, 7 insertions(+), 2 deletions(-)
diff --git a/src/kudu/integration-tests/raft_consensus_nonvoter-itest.cc b/src/kudu/integration-tests/raft_consensus_nonvoter-itest.cc
index 5f508d9..c4d9c86 100644
--- a/src/kudu/integration-tests/raft_consensus_nonvoter-itest.cc
+++ b/src/kudu/integration-tests/raft_consensus_nonvoter-itest.cc
@@ -858,8 +858,13 @@ TEST_F(RaftConsensusNonVoterITest, PromoteAndDemote) {
FLAGS_num_replicas = kInitialReplicasNum;
NO_FATALS(BuildAndStart(kTserverFlags, kMasterFlags));
ASSERT_FALSE(tablet_replicas_.empty());
- ASSERT_OK(StartElection(tablet_replicas_.begin()->second, tablet_id_, kTimeout));
- ASSERT_OK(WaitUntilLeader(tablet_replicas_.begin()->second, tablet_id_, kTimeout));
+ ASSERT_EVENTUALLY([&]() {
+ const MonoDelta kElectionTimeout = MonoDelta::FromSeconds(3);
+ ASSERT_OK(StartElection(
+ tablet_replicas_.begin()->second, tablet_id_, kElectionTimeout));
+ ASSERT_OK(WaitUntilLeader(
+ tablet_replicas_.begin()->second, tablet_id_, kElectionTimeout));
+ });
ASSERT_EQ(4, tablet_servers_.size());
ASSERT_EQ(kInitialReplicasNum, tablet_replicas_.size());