Skip to content

feat: Enhance ClusterManager with dynamic scaling, restart policies, …#3

Merged
jkyberneees merged 5 commits intomainfrom
enhance-cluster-manager-implementation
Aug 24, 2025
Merged

feat: Enhance ClusterManager with dynamic scaling, restart policies, …#3
jkyberneees merged 5 commits intomainfrom
enhance-cluster-manager-implementation

Conversation

@jkyberneees
Copy link
Contributor

No description provided.

@jkyberneees jkyberneees requested a review from Copilot August 24, 2025 21:14
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR enhances the ClusterManager with dynamic scaling capabilities, improved restart policies, and better lifecycle management. The changes add support for rolling restarts, exponential backoff with jitter, and configurable exit behavior for test environments.

  • Added dynamic scaling methods (scaleTo, scaleUp, scaleDown) for runtime worker adjustment
  • Implemented rolling restart strategy that spawns replacements before terminating old workers
  • Enhanced restart policy with sliding window throttling and exponential backoff with jitter

Reviewed Changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated 6 comments.

Show a summary per file
File Description
src/interfaces/gateway.ts Adds exitOnShutdown configuration option for embedded environments
src/cluster/cluster-manager.ts Major enhancements including dynamic scaling, rolling restarts, and improved restart policies
test/cluster/fixtures/worker.ts Minimal test worker script that responds to SIGTERM
test/cluster/cluster-manager.basic.test.ts Basic functionality tests for startup and scaling operations
test/cluster/cluster-manager.restart-policy.test.ts Tests for restart throttling and rolling restart behavior

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@jkyberneees jkyberneees merged commit ba915fd into main Aug 24, 2025
5 of 6 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.

1 participant