Fix flaky retry-count assertion in ingress_request_rate_limiter_scatter_gather_error_propagation.js

XMLWordPrintableJSON

    • Type: Bug
    • Resolution: Fixed
    • Priority: Major - P3
    • 9.1.0-rc0, 9.0.0-rc1
    • Affects Version/s: None
    • Component/s: None
    • None
    • Cluster Scalability
    • Fully Compatible
    • ALL
    • v9.0
    • Cluster Scalability Priorities
    • 200
    • None
    • None
    • None
    • None
    • None
    • None
    • None

      In the "both shards overloaded > retry cap" case, the test asserted numRetriesDueToOverloadAttempted == min(numOverloadErrorsReceived, kRetryAttempts) per shard. This fails when one shard hits its cap first and calls stopRetrying(), cutting off the other shard after it received an error but before it attempted the corresponding retry (observed 2 retries vs 3 errors). The terminal error is also always counted without a retry. Fix: assert retries is between errors - 1 and errors, capped at kRetryAttempts. Latent since the test was introduced in SERVER-127629.

            Assignee:
            Anja Kalaba
            Reporter:
            Anja Kalaba
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

              Created:
              Updated:
              Resolved: