Skip to content

cleanup connection handling#1704

Merged
luke-lombardi merged 1 commit into
mainfrom
ll/cleanup-connection-handling
Jun 21, 2026
Merged

cleanup connection handling#1704
luke-lombardi merged 1 commit into
mainfrom
ll/cleanup-connection-handling

Conversation

@luke-lombardi

@luke-lombardi luke-lombardi commented Jun 21, 2026

Copy link
Copy Markdown
Contributor

Summary by cubic

Simplified pod connection tracking by removing Redis aggregates and relying only on per-proxy snapshot counters with TTL. This avoids stale counts and ensures accurate shared totals across proxies for autoscaling.

  • Refactors
    • Removed aggregate keys and fallbacks (Keys.podTotalConnections, Keys.podContainerConnections); totals are now computed by summing snapshot keys (Keys.podProxyConnections*).
    • flushConnectionState publishes only snapshot counters with TTL; aggregate refresh path deleted.
    • Updated autoscaler and proxy tests to read shared counts from snapshots and verify stale aggregates are ignored; removed the unused podContainerConnectionTimeout.

Written for commit 6298c7d. Summary will update on new commits.

Review in cubic

@luke-lombardi luke-lombardi merged commit ee287f4 into main Jun 21, 2026
4 checks passed
@luke-lombardi luke-lombardi deleted the ll/cleanup-connection-handling branch June 21, 2026 04:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant