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

eze pushed a commit to branch 9.2.x
in repository https://gitbox.apache.org/repos/asf/trafficserver.git


The following commit(s) were added to refs/heads/9.2.x by this push:
     new 00bb0393be Reduce ram_cache regression test scope for CI stability 
(#12827)
00bb0393be is described below

commit 00bb0393be8b7df506d7e5f3ed350333b5f12083
Author: Brian Neradt <[email protected]>
AuthorDate: Mon Jan 26 19:15:41 2026 -0600

    Reduce ram_cache regression test scope for CI stability (#12827)
    
    The ram_cache test was failing in CI when run with -R 3 due to the 256MB
    cache size test running over 4 million iterations. This caused timeouts
    or OOM kills in slower CI environments. Reduced the maximum cache size
    from 256MB to 16MB and lowered the minimum regression level from
    EXTENDED to NIGHTLY.
    
    Fixes: #12728
---
 iocore/cache/CacheTest.cc | 10 +++++++---
 1 file changed, 7 insertions(+), 3 deletions(-)

diff --git a/iocore/cache/CacheTest.cc b/iocore/cache/CacheTest.cc
index c1ea6fb4cf..bb18c62c21 100644
--- a/iocore/cache/CacheTest.cc
+++ b/iocore/cache/CacheTest.cc
@@ -655,8 +655,7 @@ test_RamCache(RegressionTest *t, RamCache *cache, const 
char *name, int64_t cach
 
 REGRESSION_TEST(ram_cache)(RegressionTest *t, int level, int *pstatus)
 {
-  // Run with -R 3 for now to trigger this check, until we figure out the CI
-  if (REGRESSION_TEST_EXTENDED > level) {
+  if (REGRESSION_TEST_NIGHTLY > level) {
     *pstatus = REGRESSION_TEST_PASSED;
     return;
   }
@@ -666,7 +665,12 @@ REGRESSION_TEST(ram_cache)(RegressionTest *t, int level, 
int *pstatus)
     *pstatus = REGRESSION_TEST_FAILED;
     return;
   }
-  for (int s = 20; s <= 28; s += 4) {
+
+  // Test cache sizes from 1MB to 16MB. The sample_size is cache_size >> 6, so
+  // the 16MB test runs 262K iterations which completes in reasonable time. 
This
+  // used to run with 256MB (s=28) at 4M+ iterations, which is too much for CI
+  // resulting in failures, either due to timeout or OOM issues.
+  for (int s = 20; s <= 24; s += 4) {
     int64_t cache_size = 1LL << s;
     *pstatus           = REGRESSION_TEST_PASSED;
     if (!test_RamCache(t, new_RamCacheLRU(), "LRU", cache_size) || 
!test_RamCache(t, new_RamCacheCLFUS(), "CLFUS", cache_size)) {

Reply via email to