kudu-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From t...@apache.org
Subject [1/3] kudu git commit: [security-faults-itest] fixes on test flakiness
Date Fri, 19 May 2017 00:32:46 GMT
Repository: kudu
Updated Branches:
  refs/heads/master 733afeb82 -> 65c3aec9a


[security-faults-itest] fixes on test flakiness

Couple of updates to make the test less flaky and have more coverage:

  * Increased TTL of Kerberos ticket to 120 seconds: in case of inferior
    VMs the test sometimes could not make a couple of smoke test
    iterations in ~60 seconds.

  * Enabled newly introduced --rpc_reopen_outbound_connections flag
    to make servers authenticate the client upon every RPC call.

Test results prior to this fix with --stress_cpu_threads=8
(DEBUG build, 18 out of 1024 fail):
  http://dist-test.cloudera.org//job?job_id=aserbin.1494950598.5510

Test results after this fix --stress_cpu_threads=8
(DEBUG build,  0 out of 1024 fail):
  http://dist-test.cloudera.org//job?job_id=aserbin.1494948803.23142

Change-Id: I4f3ff1c65cd695a367e52804dae951e1b962452d
Reviewed-on: http://gerrit.cloudera.org:8080/6895
Tested-by: Kudu Jenkins
Reviewed-by: Todd Lipcon <todd@apache.org>


Project: http://git-wip-us.apache.org/repos/asf/kudu/repo
Commit: http://git-wip-us.apache.org/repos/asf/kudu/commit/068767b8
Tree: http://git-wip-us.apache.org/repos/asf/kudu/tree/068767b8
Diff: http://git-wip-us.apache.org/repos/asf/kudu/diff/068767b8

Branch: refs/heads/master
Commit: 068767b815ee75f988a406bfe4541e9602e0d831
Parents: 733afeb
Author: Alexey Serbin <aserbin@cloudera.com>
Authored: Mon May 15 23:36:03 2017 -0700
Committer: Todd Lipcon <todd@apache.org>
Committed: Thu May 18 17:10:49 2017 +0000

----------------------------------------------------------------------
 src/kudu/integration-tests/security-faults-itest.cc | 7 ++++++-
 1 file changed, 6 insertions(+), 1 deletion(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/kudu/blob/068767b8/src/kudu/integration-tests/security-faults-itest.cc
----------------------------------------------------------------------
diff --git a/src/kudu/integration-tests/security-faults-itest.cc b/src/kudu/integration-tests/security-faults-itest.cc
index f68f21d..93a9566 100644
--- a/src/kudu/integration-tests/security-faults-itest.cc
+++ b/src/kudu/integration-tests/security-faults-itest.cc
@@ -30,6 +30,7 @@
 #include "kudu/tablet/key_value_test_schema.h"
 #include "kudu/util/test_util.h"
 
+DECLARE_bool(rpc_reopen_outbound_connections);
 DECLARE_bool(rpc_trace_negotiation);
 
 using kudu::client::KuduClient;
@@ -52,10 +53,14 @@ namespace kudu {
 class SecurityComponentsFaultsITest : public KuduTest {
  public:
   SecurityComponentsFaultsITest()
-      : krb_lifetime_seconds_(64),
+      : krb_lifetime_seconds_(120),
         num_masters_(3),
         num_tservers_(3) {
 
+    // Reopen client-->server connections on every RPC. This is to make sure the
+    // servers authenticate the client on every RPC call.
+    FLAGS_rpc_reopen_outbound_connections = true;
+
     // Want to run with Kerberos enabled.
     cluster_opts_.enable_kerberos = true;
 


Mime
View raw message