Skip to content

Reduce max Pulp workers to 12#2287

Open
priteau wants to merge 1 commit intostackhpc/2024.1from
reduce-seed-pulp-workers-caracal
Open

Reduce max Pulp workers to 12#2287
priteau wants to merge 1 commit intostackhpc/2024.1from
reduce-seed-pulp-workers-caracal

Conversation

@priteau
Copy link
Copy Markdown
Member

@priteau priteau commented Apr 27, 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.

(cherry picked from commit 1bb897f)

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.

(cherry picked from commit 1bb897f)
@priteau priteau self-assigned this Apr 27, 2026
@priteau priteau requested a review from a team as a code owner April 27, 2026 08:25
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 to 12 to prevent connection starvation on seed hosts. Feedback suggests also capping task workers to avoid similar issues and correcting inconsistencies in the release notes regarding CPU thread counts.

Comment thread etc/kayobe/seed.yml
Comment thread releasenotes/notes/reduce-seed-pulp-workers-b085c86bfe7ab5b6.yaml
@Alex-Welsh
Copy link
Copy Markdown
Member

recheck

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