Skip to content

Cap HTTP server worker counts to bound open file descriptors#242

Merged
mcharytoniuk merged 2 commits into
mainfrom
limit-http-server-workers
Jun 5, 2026
Merged

Cap HTTP server worker counts to bound open file descriptors#242
mcharytoniuk merged 2 commits into
mainfrom
limit-http-server-workers

Conversation

@mcharytoniuk
Copy link
Copy Markdown
Contributor

No description provided.

Copilot AI review requested due to automatic review settings June 5, 2026 13:20
Copy link
Copy Markdown
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 aims to bound open file descriptor usage by capping the number of Actix HttpServer worker threads used by the balancer’s HTTP-facing services.

Changes:

  • Introduced per-service HTTP_WORKERS constants to define an upper bound for worker counts.
  • Applied .workers(HTTP_WORKERS) to Actix HttpServer builders across web admin, management, inference, and OpenAI-compat services.

Reviewed changes

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

File Description
paddler_balancer/src/web_admin_panel_service/mod.rs Adds a worker cap constant and applies it to the web admin panel HttpServer.
paddler_balancer/src/management_service/mod.rs Adds a worker cap constant and applies it to the management HttpServer.
paddler_balancer/src/inference_service/mod.rs Adds a worker cap constant and applies it to the inference HttpServer.
paddler_balancer/src/compatibility/openai_service/mod.rs Adds a worker cap constant and applies it to the OpenAI-compat HttpServer.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread paddler_balancer/src/inference_service/mod.rs
Comment thread paddler_balancer/src/compatibility/openai_service/mod.rs
Comment thread paddler_balancer/src/management_service/mod.rs
Comment thread paddler_balancer/src/web_admin_panel_service/mod.rs
@mcharytoniuk mcharytoniuk merged commit b347079 into main Jun 5, 2026
6 checks passed
@mcharytoniuk mcharytoniuk deleted the limit-http-server-workers branch June 5, 2026 15:20
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