mirror of
https://github.com/getnora-io/nora.git
synced 2026-04-12 16:10:31 +00:00
Increase upload rate limits for Docker parallel requests
Docker client sends many parallel requests when pushing layers. Increased upload rate limiter from 10 req/s to 50 req/s and burst from 20 to 100.
This commit is contained in:
@@ -28,12 +28,12 @@ pub struct RateLimitConfig {
|
|||||||
impl Default for RateLimitConfig {
|
impl Default for RateLimitConfig {
|
||||||
fn default() -> Self {
|
fn default() -> Self {
|
||||||
Self {
|
Self {
|
||||||
auth_rps: 1, // 1 req/sec for auth (strict)
|
auth_rps: 1, // 1 req/sec for auth (strict)
|
||||||
auth_burst: 5, // Allow burst of 5
|
auth_burst: 5, // Allow burst of 5
|
||||||
upload_rps: 10, // 10 req/sec for uploads
|
upload_rps: 50, // 50 req/sec for uploads (Docker needs parallel)
|
||||||
upload_burst: 20, // Allow burst of 20
|
upload_burst: 100, // Allow burst of 100
|
||||||
general_rps: 100, // 100 req/sec general
|
general_rps: 100, // 100 req/sec general
|
||||||
general_burst: 200, // Allow burst of 200
|
general_burst: 200, // Allow burst of 200
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
@@ -58,15 +58,16 @@ pub fn auth_rate_limiter() -> tower_governor::GovernorLayer<
|
|||||||
|
|
||||||
/// Create rate limiter layer for upload endpoints
|
/// Create rate limiter layer for upload endpoints
|
||||||
///
|
///
|
||||||
/// Default: 10 requests per second, burst of 20
|
/// Default: 50 requests per second, burst of 100
|
||||||
|
/// Higher limits to accommodate Docker client's parallel layer uploads
|
||||||
pub fn upload_rate_limiter() -> tower_governor::GovernorLayer<
|
pub fn upload_rate_limiter() -> tower_governor::GovernorLayer<
|
||||||
tower_governor::key_extractor::PeerIpKeyExtractor,
|
tower_governor::key_extractor::PeerIpKeyExtractor,
|
||||||
governor::middleware::StateInformationMiddleware,
|
governor::middleware::StateInformationMiddleware,
|
||||||
axum::body::Body,
|
axum::body::Body,
|
||||||
> {
|
> {
|
||||||
let config = GovernorConfigBuilder::default()
|
let config = GovernorConfigBuilder::default()
|
||||||
.per_second(10)
|
.per_second(50)
|
||||||
.burst_size(20)
|
.burst_size(100)
|
||||||
.use_headers()
|
.use_headers()
|
||||||
.finish()
|
.finish()
|
||||||
.unwrap();
|
.unwrap();
|
||||||
@@ -101,7 +102,7 @@ mod tests {
|
|||||||
let config = RateLimitConfig::default();
|
let config = RateLimitConfig::default();
|
||||||
assert_eq!(config.auth_rps, 1);
|
assert_eq!(config.auth_rps, 1);
|
||||||
assert_eq!(config.auth_burst, 5);
|
assert_eq!(config.auth_burst, 5);
|
||||||
assert_eq!(config.upload_rps, 10);
|
assert_eq!(config.upload_rps, 50);
|
||||||
assert_eq!(config.general_rps, 100);
|
assert_eq!(config.general_rps, 100);
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|||||||
Reference in New Issue
Block a user