mirror of
https://github.com/rcourtman/Pulse.git
synced 2026-02-18 00:17:39 +01:00
fix: increase pulse-sensor-proxy rate limits for multi-node support
- Increase rate limit from 1 req/5sec to 1 req/sec (60/min) - Increase burst from 2 to 5 requests - Fixes temperature collection failures when monitoring 3+ nodes - All requests from containerized Pulse use same UID, causing rate limiting - New limits support 5-10 node deployments comfortably Resolves issue where adding standalone nodes broke temperature monitoring for all nodes due to aggressive rate limiting.
This commit is contained in:
@@ -38,13 +38,13 @@ type rateLimiter struct {
|
||||
}
|
||||
|
||||
const (
|
||||
defaultPerPeerBurst = 2
|
||||
defaultPerPeerBurst = 5 // Allow burst of 5 requests for multi-node polling
|
||||
defaultPerPeerConcurrency = 2
|
||||
defaultGlobalConcurrency = 8
|
||||
)
|
||||
|
||||
var (
|
||||
defaultPerPeerRateInterval = 5 * time.Second // 0.2 qps (~12/min)
|
||||
defaultPerPeerRateInterval = 1 * time.Second // 1 qps (60/min) - supports 5-10 node deployments
|
||||
defaultPenaltyDuration = 2 * time.Second
|
||||
defaultPerPeerLimit = rate.Every(defaultPerPeerRateInterval)
|
||||
)
|
||||
|
||||
Reference in New Issue
Block a user