vrr/Pulse
2
0
Fork 0
mirror of https://github.com/rcourtman/Pulse.git synced 2026-05-20 01:01:20 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 2

fix: increase pulse-sensor-proxy rate limits for multi-node support

Browse source 
- 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:
rcourtman 2025-10-21 11:21:12 +00:00
parent 5f5d91b87c
commit 46b8b8d08f
1 changed files with 2 additions and 2 deletions
Download patch file Download diff file Expand all files Collapse all files

4
cmd/pulse-sensor-proxy/throttle.go
View file

@ -38,13 +38,13 @@ type rateLimiter struct {
} }
const ( const (
defaultPerPeerBurst = 2 defaultPerPeerBurst = 5 // Allow burst of 5 requests for multi-node polling
defaultPerPeerConcurrency = 2 defaultPerPeerConcurrency = 2
defaultGlobalConcurrency = 8 defaultGlobalConcurrency = 8
) )
var ( 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 defaultPenaltyDuration = 2 * time.Second
defaultPerPeerLimit = rate.Every(defaultPerPeerRateInterval) defaultPerPeerLimit = rate.Every(defaultPerPeerRateInterval)
) )