commit 8e137e524a1b1da7bbf4603662876d5719563b57
author Craig Tiller <ctiller@google.com> Mon Jun 12 15:03:33 2023 -0700
committer GitHub <noreply@github.com> Mon Jun 12 15:03:33 2023 -0700
tree a5265214d5511c2af9c3c1cc7d289b123eeab48f
parent fa32eb36b3c39ce596f075d471c35eb28a344aab

[end2end] Reduce likelihood of two tests opening the same UDS socket (#33410)

test/core/end2end/end2end_test_suites.cc

diff --git a/test/core/end2end/end2end_test_suites.cc b/test/core/end2end/end2end_test_suites.cc
index 527b9db..9ab5603 100644
--- a/test/core/end2end/end2end_test_suites.cc
+++ b/test/core/end2end/end2end_test_suites.cc
@@ -21,6 +21,7 @@
 #include <initializer_list>
 #include <map>
 #include <memory>
+#include <random>
 #include <regex>
 #include <string>
 #include <utility>
@@ -85,7 +86,7 @@
 
 namespace {
 
-std::atomic<int> unique{0};
+std::atomic<uint64_t> unique{std::random_device()()};
 
 void ProcessAuthFailure(void* state, grpc_auth_context* /*ctx*/,
                         const grpc_metadata* /*md*/, size_t /*md_count*/,
@@ -568,7 +569,7 @@
               return std::make_unique<LocalTestFixture>(
                   absl::StrFormat(
                       "unix-abstract:grpc_fullstack_test.%%00.%d.%" PRId64
-                      ".%" PRId32 ".%d",
+                      ".%" PRId32 ".%" PRId64,
                       getpid(), now.tv_sec, now.tv_nsec,
                       unique.fetch_add(1, std::memory_order_relaxed)),
                   UDS);
@@ -611,7 +612,7 @@
               return std::make_unique<LocalTestFixture>(
                   absl::StrFormat(
                       "unix:/tmp/grpc_fullstack_test.%%25.%d.%" PRId64
-                      ".%" PRId32 ".%d",
+                      ".%" PRId32 ".%" PRId64,
                       getpid(), now.tv_sec, now.tv_nsec,
                       unique.fetch_add(1, std::memory_order_relaxed)),
                   UDS);
@@ -628,7 +629,7 @@
               return std::make_unique<LocalTestFixture>(
                   absl::StrFormat(
                       "unix:/tmp/grpc_fullstack_test.%d.%" PRId64 ".%" PRId32
-                      ".%d",
+                      ".%" PRId64,
                       getpid(), now.tv_sec, now.tv_nsec,
                       unique.fetch_add(1, std::memory_order_relaxed)),
                   UDS);
@@ -846,7 +847,7 @@
               gpr_timespec now = gpr_now(GPR_CLOCK_REALTIME);
               return std::make_unique<InsecureFixture>(absl::StrFormat(
                   "unix-abstract:grpc_fullstack_test.%d.%" PRId64 ".%" PRId32
-                  ".%d",
+                  ".%" PRId64,
                   getpid(), now.tv_sec, now.tv_nsec,
                   unique.fetch_add(1, std::memory_order_relaxed)));
             }},
@@ -860,7 +861,8 @@
             [](const ChannelArgs&, const ChannelArgs&) {
               gpr_timespec now = gpr_now(GPR_CLOCK_REALTIME);
               return std::make_unique<InsecureFixture>(absl::StrFormat(
-                  "unix:/tmp/grpc_fullstack_test.%d.%" PRId64 ".%" PRId32 ".%d",
+                  "unix:/tmp/grpc_fullstack_test.%d.%" PRId64 ".%" PRId32
+                  ".%" PRId64,
                   getpid(), now.tv_sec, now.tv_nsec,
                   unique.fetch_add(1, std::memory_order_relaxed)));
             }},