kudu git commit: [security-faults-itest] longer TTL for Kerberos tickets

[da...@apache.org]

Repository: kudu
Updated Branches:
  refs/heads/master 206a3f1f5 -> 16dd6e4aa


[security-faults-itest] longer TTL for Kerberos tickets

Increased TTL for Kerberos tickets -- as it turned out, starting up
external Kudu mini-cluster on inferior VMs takes a long time.

Change-Id: Ife65aef745ca82a70e3e79e7beccd2821bb48f2e
Reviewed-on: http://gerrit.cloudera.org:8080/6413
Tested-by: Kudu Jenkins
Reviewed-by: Dan Burkert <da...@apache.org>


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

Branch: refs/heads/master
Commit: 16dd6e4aa1c660120c331245bdefe2498a425957
Parents: 206a3f1
Author: Alexey Serbin <as...@cloudera.com>
Authored: Thu Mar 16 11:18:46 2017 -0700
Committer: Dan Burkert <da...@apache.org>
Committed: Fri Mar 17 00:00:48 2017 +0000

----------------------------------------------------------------------
 src/kudu/integration-tests/security-faults-itest.cc | 16 ++++++++++++++--
 1 file changed, 14 insertions(+), 2 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/kudu/blob/16dd6e4a/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 12309d5..6651e85 100644
--- a/src/kudu/integration-tests/security-faults-itest.cc
+++ b/src/kudu/integration-tests/security-faults-itest.cc
@@ -53,9 +53,9 @@ class SecurityComponentsFaultsITest : public KuduTest {
  public:
   SecurityComponentsFaultsITest()
 #if defined(ADDRESS_SANITIZER) || defined(THREAD_SANITIZER)
-      : krb_lifetime_seconds_(15),
+      : krb_lifetime_seconds_(64),
 #else
-      : krb_lifetime_seconds_(5),
+      : krb_lifetime_seconds_(16),
 #endif
         num_masters_(3),
         num_tservers_(3) {
@@ -128,6 +128,8 @@ class SecurityComponentsFaultsITest : public KuduTest {
   }
 
  protected:
+  // The ticket lifetime should be long enough to start the cluster and run a
+  // single iteration of smoke test workload.
   const int krb_lifetime_seconds_;
   const int num_masters_;
   const int num_tservers_;
@@ -138,6 +140,11 @@ class SecurityComponentsFaultsITest : public KuduTest {
 // Check how the system behaves when KDC is not available upon start-up
 // of Kudu server-side components.
 TEST_F(SecurityComponentsFaultsITest, NoKdcOnStart) {
+  if (!AllowSlowTests()) {
+    LOG(WARNING) << "test is skipped; set KUDU_ALLOW_SLOW_TESTS=1 to run";
+    return;
+  }
+
   // Start with the KDC first: let's generate generate keytabs, get initial
   // kerberos tickets, etc.
   ASSERT_OK(StartCluster());
@@ -174,6 +181,11 @@ TEST_F(SecurityComponentsFaultsITest, NoKdcOnStart) {
 // Check that restarting KDC does not affect running master and tablet servers:
 // they are able to operate with no issues past ticket TTL once KDC is back.
 TEST_F(SecurityComponentsFaultsITest, KdcRestartsInTheMiddle) {
+  if (!AllowSlowTests()) {
+    LOG(WARNING) << "test is skipped; set KUDU_ALLOW_SLOW_TESTS=1 to run";
+    return;
+  }
+
   // Enable KRPC negotiation tracing for the Kudu client running smoke test
   // workload.
   FLAGS_rpc_trace_negotiation = true;