Skip to content

Commit 300f9a4

Browse files
fix: align memory stress tests with backpressure behavior
- Make BACKPRESSURE_TIMEOUT configurable via S3PROXY_BACKPRESSURE_TIMEOUT env var - Replace test_concurrent_uploads_bounded with test_backpressure_queues_concurrent_uploads (tests that queued requests succeed instead of asserting immediate 503) - Replace test_rejection_is_fast_no_body_read with test_rejection_after_backpressure_timeout (uses 1s timeout to actually trigger rejections) - Remove retries>0 assertions from memory-bounded tests (backpressure queues requests instead of rejecting, so retries don't happen with 30s timeout)
1 parent 72b69dc commit 300f9a4

File tree

2 files changed

+146
-234
lines changed

2 files changed

+146
-234
lines changed

s3proxy/concurrency.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -42,7 +42,7 @@ def _create_malloc_release() -> Callable[[], int] | None:
4242
_malloc_release = _create_malloc_release()
4343

4444

45-
BACKPRESSURE_TIMEOUT = 30 # seconds to wait before rejecting
45+
BACKPRESSURE_TIMEOUT = int(os.environ.get("S3PROXY_BACKPRESSURE_TIMEOUT", "30"))
4646

4747

4848
class ConcurrencyLimiter:

0 commit comments

Comments
 (0)