bazel: update tcmalloc (#23671)

keith · web-flow · commit 1c68be22da99 · 2022-11-01T18:29:46.000-07:00
Fixes #23580 Full diff: google/tcmalloc@5940033...e33c7bc Signed-off-by: Keith Smiley <keithbsmiley@gmail.com>
diff --git a/bazel/repository_locations.bzl b/bazel/repository_locations.bzl
@@ -336,12 +336,12 @@ REPOSITORY_LOCATIONS_SPEC = dict(
         project_name = "tcmalloc",
         project_desc = "Fast, multi-threaded malloc implementation",
         project_url = "https://github.com/google/tcmalloc",
-        version = "59400332b9cff9920b6a1da203ac1575272a9f44",
-        sha256 = "3e0a0c135318fa69e748b140d32cb24a64d885bedfeb5f23fa01cc0d7859bbf0",
+        version = "e33c7bc60415127c104006d3301c96902f98d42a",
+        sha256 = "14a2c91b71d6719558768a79671408c9acd8284b418e80386c5888047e2c15aa",
         strip_prefix = "tcmalloc-{version}",
         urls = ["https://github.com/google/tcmalloc/archive/{version}.tar.gz"],
         use_category = ["dataplane_core", "controlplane"],
-        release_date = "2022-08-06",
+        release_date = "2022-10-24",
         cpe = "N/A",
         license = "Apache-2.0",
         license_url = "https://github.com/google/tcmalloc/blob/{version}/LICENSE",
diff --git a/test/common/stats/thread_local_store_test.cc b/test/common/stats/thread_local_store_test.cc
@@ -1381,7 +1381,7 @@ TEST_F(StatsThreadLocalStoreTestNoFixture, MemoryWithoutTlsRealSymbolTable) {
   TestUtil::forEachSampleStat(
       100, true, [this](absl::string_view name) { store_.counterFromString(std::string(name)); });
   EXPECT_MEMORY_EQ(memory_test.consumedBytes(), 688080); // July 2, 2020
-  EXPECT_MEMORY_LE(memory_test.consumedBytes(), 0.75 * million_);
+  EXPECT_MEMORY_LE(memory_test.consumedBytes(), 0.85 * million_);
 }
 
 TEST_F(StatsThreadLocalStoreTestNoFixture, MemoryWithTlsRealSymbolTable) {
@@ -1390,7 +1390,7 @@ TEST_F(StatsThreadLocalStoreTestNoFixture, MemoryWithTlsRealSymbolTable) {
   TestUtil::forEachSampleStat(
       100, true, [this](absl::string_view name) { store_.counterFromString(std::string(name)); });
   EXPECT_MEMORY_EQ(memory_test.consumedBytes(), 827616); // Sep 25, 2020
-  EXPECT_MEMORY_LE(memory_test.consumedBytes(), 0.9 * million_);
+  EXPECT_MEMORY_LE(memory_test.consumedBytes(), 0.97 * million_);
 }
 
 TEST_F(StatsThreadLocalStoreTest, ShuttingDown) {
diff --git a/test/integration/stats_integration_test.cc b/test/integration/stats_integration_test.cc
@@ -378,6 +378,7 @@ TEST_P(ClusterMemoryTestRunner, MemoryLargeClusterSize) {
   //                                          searching
   // 2021/08/18  13176    40577       40700   Support slow start mode
   // 2022/03/14                       42000   Fix test flakes
+  // 2022/10/27                       44000   Update tcmalloc
 
   // Note: when adjusting this value: EXPECT_MEMORY_EQ is active only in CI
   // 'release' builds, where we control the platform and tool-chain. So you
@@ -398,7 +399,7 @@ TEST_P(ClusterMemoryTestRunner, MemoryLargeClusterSize) {
     // https://github.com/envoyproxy/envoy/issues/12209
     // EXPECT_MEMORY_EQ(m_per_cluster, 37061);
   }
-  EXPECT_MEMORY_LE(m_per_cluster, 42000); // Round up to allow platform variations.
+  EXPECT_MEMORY_LE(m_per_cluster, 44000); // Round up to allow platform variations.
 }
 
 TEST_P(ClusterMemoryTestRunner, MemoryLargeHostSizeWithStats) {

Original file line number	Diff line number	Diff line change
`@@ -378,6 +378,7 @@ TEST_P(ClusterMemoryTestRunner, MemoryLargeClusterSize) {`
`378`	`378`	`// searching`
`379`	`379`	`// 2021/08/18 13176 40577 40700 Support slow start mode`
`380`	`380`	`// 2022/03/14 42000 Fix test flakes`
	`381`	`+ // 2022/10/27 44000 Update tcmalloc`
`381`	`382`
`382`	`383`	`// Note: when adjusting this value: EXPECT_MEMORY_EQ is active only in CI`
`383`	`384`	`// 'release' builds, where we control the platform and tool-chain. So you`
`@@ -398,7 +399,7 @@ TEST_P(ClusterMemoryTestRunner, MemoryLargeClusterSize) {`
`398`	`399`	`// https://github.com/envoyproxy/envoy/issues/12209`
`399`	`400`	`// EXPECT_MEMORY_EQ(m_per_cluster, 37061);`
`400`	`401`	`}`
`401`		`- EXPECT_MEMORY_LE(m_per_cluster, 42000); // Round up to allow platform variations.`
	`402`	`+ EXPECT_MEMORY_LE(m_per_cluster, 44000); // Round up to allow platform variations.`
`402`	`403`	`}`
`403`	`404`
`404`	`405`	`TEST_P(ClusterMemoryTestRunner, MemoryLargeHostSizeWithStats) {`