Skip to content

Reduce max Pulp workers to 12#2253

Merged
cityofships merged 1 commit intostackhpc/2025.1from
reduce-seed-pulp-workers
Apr 2, 2026
Merged

Reduce max Pulp workers to 12#2253
cityofships merged 1 commit intostackhpc/2025.1from
reduce-seed-pulp-workers

Conversation

@m-bull
Copy link
Copy Markdown
Contributor

@m-bull m-bull commented Apr 2, 2026

On seed hosts with more than 32 CPU threads, PULP_CONTENT_WORKERS and PULP_API_WORKERS were set to 32, which was causing the number of connections to the bundled Postgres instance to exceed the default max_connections.

Reducing the maximum number of PULP_CONTENT_WORKERS and PULP_API_WORKERS will prevent Postgres from running out of connection slots.

@m-bull m-bull requested a review from a team as a code owner April 2, 2026 18:38
Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request reduces the maximum number of Pulp content and API workers from 32 to 12 to prevent connection starvation on seed hosts. Feedback suggests updating the release note to accurately state that this issue affects hosts with 16 or more CPU threads, rather than 32, based on the worker calculation logic.

On seed hosts with more than 32 CPU threads, PULP_CONTENT_WORKERS
and PULP_API_WORKERS were set to 32, which was causing the number
of connections to the bundled Postgres instance to exceed the
default max_connections.

Reducing the maximum number of PULP_CONTENT_WORKERS and PULP_API_WORKERS
will prevent Postgres from running out of connection slots.
@m-bull m-bull force-pushed the reduce-seed-pulp-workers branch from 05aa447 to bd670de Compare April 2, 2026 18:46
@cityofships cityofships merged commit 1bb897f into stackhpc/2025.1 Apr 2, 2026
21 of 22 checks passed
@cityofships cityofships deleted the reduce-seed-pulp-workers branch April 2, 2026 21:42
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.

3 participants