Use longer timeout when waiting for all threads to queue in ConcurrentTokenAcquisitionWithQueueing test

XMLWordPrintableJSON

    • Type: Bug
    • Resolution: Fixed
    • Priority: Major - P3
    • 8.2.0-rc0, 8.0.12
    • Affects Version/s: None
    • Component/s: None
    • None
    • Networking & Observability
    • ALL
    • v8.1, v8.0, v7.0
    • N&O 2025-07-07
    • 0
    • None
    • 3
    • TBD
    • None
    • None
    • None
    • None
    • None
    • None
    • None

      We wait for all threads to queue in the unit test, but it looks like on very slow machines the test may not wait long enough, and we hit the assertion right before the last request queues:

      [2025/06/15 12:22:14.723] {"t":{"$date":"2025-06-15T15:40:30.982+00:00"},"s":"D4", "c":"-",        "id":10550200,"ctx":"thread21","msg":"Going to sleep waiting for token acquisition","attr":{"rateLimiterName":"ConcurrentTokenAcquisitionWithQueueing","napTimeMillis":"249 ms"}}
      [2025/06/15 12:22:14.723] {"t":{"$date":"2025-06-15T15:40:30.982+00:00"},"s":"D4", "c":"-",        "id":10550200,"ctx":"thread22","msg":"Going to sleep waiting for token acquisition","attr":{"rateLimiterName":"ConcurrentTokenAcquisitionWithQueueing","napTimeMillis":"499 ms"}}
      [2025/06/15 12:22:14.723] {"t":{"$date":"2025-06-15T15:40:31.010+00:00"},"s":"D4", "c":"-",        "id":10550200,"ctx":"thread23","msg":"Going to sleep waiting for token acquisition","attr":{"rateLimiterName":"ConcurrentTokenAcquisitionWithQueueing","napTimeMillis":"721 ms"}}
      [2025/06/15 12:22:14.723] {"t":{"$date":"2025-06-15T15:40:31.010+00:00"},"s":"D4", "c":"-",        "id":10550200,"ctx":"thread24","msg":"Going to sleep waiting for token acquisition","attr":{"rateLimiterName":"ConcurrentTokenAcquisitionWithQueueing","napTimeMillis":"971 ms"}}
      [2025/06/15 12:22:14.723] {"t":{"$date":"2025-06-15T15:40:31.010+00:00"},"s":"D4", "c":"-",        "id":10550200,"ctx":"thread25","msg":"Going to sleep waiting for token acquisition","attr":{"rateLimiterName":"ConcurrentTokenAcquisitionWithQueueing","napTimeMillis":"1221 ms"}}
      [2025/06/15 12:22:14.723] {"t":{"$date":"2025-06-15T15:40:31.041+00:00"},"s":"D4", "c":"-",        "id":10550200,"ctx":"thread26","msg":"Going to sleep waiting for token acquisition","attr":{"rateLimiterName":"ConcurrentTokenAcquisitionWithQueueing","napTimeMillis":"1440 ms"}}
      [2025/06/15 12:22:14.723] {"t":{"$date":"2025-06-15T15:40:31.041+00:00"},"s":"D4", "c":"-",        "id":10550200,"ctx":"thread27","msg":"Going to sleep waiting for token acquisition","attr":{"rateLimiterName":"ConcurrentTokenAcquisitionWithQueueing","napTimeMillis":"1690 ms"}}
      [2025/06/15 12:22:14.723] {"t":{"$date":"2025-06-15T15:40:31.310+00:00"},"s":"E",  "c":"TEST",     "id":23070,   "ctx":"thread28","msg":"Throwing exception","attr":{"exception":"Expected rateLimiter.queued() == numThreads - maxTokens (7 == 8) @src/mongo/db/admission/rate_limiter_util_test.cpp:442"}}
      [2025/06/15 12:22:14.723] {"t":{"$date":"2025-06-15T15:40:31.386+00:00"},"s":"D4", "c":"-",        "id":10550200,"ctx":"thread29","msg":"Going to sleep waiting for token acquisition","attr":{"rateLimiterName":"ConcurrentTokenAcquisitionWithQueueing","napTimeMillis":"1594 ms"}}
      
      

      We can increase the timeout to avoid this failure.

            Assignee:
            Erin McNulty
            Reporter:
            Erin McNulty
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

              Created:
              Updated:
              Resolved: