(kudu) branch master updated: KUDU-3605 [TestSecurity] Start Tservers during test

[abukor]

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

abukor pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/kudu.git


The following commit(s) were added to refs/heads/master by this push:
     new f644e373b KUDU-3605 [TestSecurity] Start Tservers during test
f644e373b is described below

commit f644e373b9adee8ee73987f23ae4d6abe7bb98a6
Author: Ádám Bakai <aba...@cloudera.com>
AuthorDate: Tue Oct 22 15:12:05 2024 +0200

    KUDU-3605 [TestSecurity] Start Tservers during test
    
    testExternallyProvidedSubjectRefreshedExternally failed approximately
    4% of the time due to a missing consensus meta file. Running master
    servers without tserver can lead to undefined behavior. Adding the
    option to start tservers resolved the issue. This was tested on
    dist-test by running hundreds of times.
    
    
    Change-Id: Ie9389110f2bcaaec1743fff8f6e859295b30735c
    Reviewed-on: http://gerrit.cloudera.org:8080/21965
    Reviewed-by: Attila Bukor <abu...@apache.org>
    Tested-by: Attila Bukor <abu...@apache.org>
---
 java/kudu-client/src/test/java/org/apache/kudu/client/TestSecurity.java | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git 
a/java/kudu-client/src/test/java/org/apache/kudu/client/TestSecurity.java 
b/java/kudu-client/src/test/java/org/apache/kudu/client/TestSecurity.java
index cbe84d426..cd181bc0c 100644
--- a/java/kudu-client/src/test/java/org/apache/kudu/client/TestSecurity.java
+++ b/java/kudu-client/src/test/java/org/apache/kudu/client/TestSecurity.java
@@ -430,7 +430,7 @@ public class TestSecurity {
    */
   @Test(timeout = 300000)
   public void testExternallyProvidedSubjectRefreshedExternally() throws 
Exception {
-    startCluster(ImmutableSet.of(Option.SHORT_TOKENS_AND_TICKETS));
+    startCluster(ImmutableSet.of(Option.SHORT_TOKENS_AND_TICKETS, 
Option.START_TSERVERS));
 
     Subject subject = SecurityUtil.getSubjectFromTicketCacheOrNull();
     Assert.assertNotNull(subject);