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 2023/06/05 16:34:37 UTC
[kudu] 06/06: [jwt] Fix flaky jwt tests
This is an automated email from the ASF dual-hosted git repository.
alexey pushed a commit to branch branch-1.17.x
in repository https://gitbox.apache.org/repos/asf/kudu.git
commit 06bd4cbc1c78c407625bcb2a8ca93671d63af1ed
Author: Marton Greber <gr...@gmail.com>
AuthorDate: Wed Apr 19 13:29:01 2023 +0000
[jwt] Fix flaky jwt tests
Previously the jwt tests in security-itest.cc were flaky. To fix them
this patch increases kLifetimeMs from 1000 to 5000.
The following command has been looped to check for stability:
KUDU_ALLOW_SLOW_TESTS=1 ./bin/security-itest \
--gtest_filter='*Jwt*' --stress_cpu_threads=32
Change-Id: I049ecd2330c6aca59bf2edc96af8a0866fd869d4
Reviewed-on: http://gerrit.cloudera.org:8080/19769
Tested-by: Kudu Jenkins
Reviewed-by: Zoltan Chovan <zc...@cloudera.com>
Reviewed-by: Attila Bukor <ab...@apache.org>
(cherry picked from commit bcac764985821091383f6f2519125dce7a4f37b5)
Reviewed-on: http://gerrit.cloudera.org:8080/19976
Reviewed-by: Alexey Serbin <al...@apache.org>
---
src/kudu/integration-tests/security-itest.cc | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/src/kudu/integration-tests/security-itest.cc b/src/kudu/integration-tests/security-itest.cc
index 8987292dd..045309e88 100644
--- a/src/kudu/integration-tests/security-itest.cc
+++ b/src/kudu/integration-tests/security-itest.cc
@@ -523,7 +523,7 @@ TEST_F(SecurityITest, TestJwtMiniCluster) {
MiniOidcOptions oidc_opts;
const auto* const kValidAccount = "valid";
const auto* const kInvalidAccount = "invalid";
- const uint64_t kLifetimeMs = 1000;
+ const uint64_t kLifetimeMs = 5000;
oidc_opts.account_ids = {
{ kValidAccount, true },
{ kInvalidAccount, false },
@@ -609,7 +609,7 @@ TEST_F(SecurityITest, TestJwtMiniClusterWithInvalidCert) {
cluster_opts_.enable_client_jwt = true;
MiniOidcOptions oidc_opts;
const auto* const kValidAccount = "valid";
- const uint64_t kLifetimeMs = 1000;
+ const uint64_t kLifetimeMs = 5000;
oidc_opts.account_ids = {
{ kValidAccount, true }
};
@@ -666,7 +666,7 @@ TEST_F(SecurityITest, TestJwtMiniClusterWithUntrustedCert) {
cluster_opts_.enable_client_jwt = true;
MiniOidcOptions oidc_opts;
const auto* const kValidAccount = "valid";
- const uint64_t kLifetimeMs = 1000;
+ const uint64_t kLifetimeMs = 5000;
oidc_opts.account_ids = {
{ kValidAccount, true }
};