Skip to content

Limit connections to 12 and fix application connection and deletion logic - #219

Merged
ravjotbrar merged 12 commits into
mainfrom
limit-connections
Feb 13, 2026
Merged

ravjotbrar merged 12 commits into
mainfrom
limit-connections

Conversation

@ravjotbrar

@ravjotbrar ravjotbrar commented Feb 11, 2026

Copy link
Copy Markdown
Collaborator

Description

  • Start one metrics server for only connected nodes, instead of all cluster nodes
  • Close metrics server when we disconnect/delete a connection by adding a delete endpoint
  • Limit active connections to a maximum of 12
  • Change tap in connection epic to mergeMap so returns aren't ignored
  • Fix delete epic to close and delete connections appropriately
  • Reuse GlideClusterClient for all nodes belonging to the same cluster
  • Add clusterId to clients map so we know when to close glide cluster client
  • Add clusterNodesMap to keep track of connected nodes per cluster

Change Visualization

Limited to 1 for testing purposes

Screen.Recording.2026-02-11.at.2.19.03.PM.mov

Ravjot Brar and others added 5 commits February 10, 2026 22:24
Comment thread apps/frontend/src/components/cluster-topology/cluster-node.tsx Outdated
Comment thread apps/frontend/src/components/ui/connection-form.tsx Outdated
Comment thread apps/frontend/src/state/valkey-features/connection/connectionSelectors.ts Outdated
Comment thread apps/metrics/src/index.js Outdated
Comment thread apps/server/src/actions/config.ts
Comment thread apps/server/src/actions/connection.ts Outdated
Comment thread apps/server/src/connection.ts
Comment thread apps/server/src/connection.ts Outdated
Comment thread apps/server/src/connection.ts
Comment thread apps/server/src/connection.ts Outdated
Comment thread apps/server/src/connection.ts Outdated
Comment thread apps/server/src/connection.ts Outdated
Comment thread common/src/constants.ts Outdated
Comment thread apps/metrics/src/index.js
@ravjotbrar
ravjotbrar merged commit 2046463 into main Feb 13, 2026
7 checks passed
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.

2 participants