Project

General

Profile

Feature #8266

pulp worker count needs a maximum

Added by Eric Helms over 6 years ago. Updated almost 3 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
Installer
Target version:
Difficulty:
Triaged:
Yes
Bugzilla link:
Pull request:
Fixed in Releases:
Found in Releases:

Description

Cloned from https://bugzilla.redhat.com/show_bug.cgi?id=1120903
Description of problem:

currently if you install a satellite on systems with a large number of cpus (16, 32, etc..) the number of pulp workers is very large. In reality we only need around 8 workers max. So we should default the config file in /etc/defaults/ to 6 or 8 or the cpu count if its lower than 6 or 8

Version-Release number of selected component (if applicable):

How reproducible:

Steps to Reproduce:
1. Install satellite 6
2. Check the number of workers ps aux | grep pulp_workers (then divide by 2)

Actual results:
Does not limit itself to any number.

Expected results:
For a large cpu system should limit itself to 6 or 8

Additional info:

Associated revisions

Revision b7efa752 (diff)
Added by Eric Helms over 6 years ago

Fixes #8266: Allow setting the number of Pulp workers to be used.

This also adds logic to set the Pulp workers based on processor count
but to cap the maximum at 8 unless overriden by a user.

Revision c16eb78d
Added by Eric D Helms over 6 years ago

Merge pull request #30 from ehelms/fixes-8266

Fixes #8266: Allow setting the number of Pulp workers to be used.

History

#1 Updated by Eric Helms over 6 years ago

  • Target version set to 62
  • Legacy Backlogs Release (now unused) set to 14
  • Triaged changed from No to Yes

#2 Updated by Eric Helms over 6 years ago

  • Status changed from New to Ready For Testing

#3 Updated by Eric Helms over 6 years ago

  • Target version changed from 62 to 63

#4 Updated by Eric Helms over 6 years ago

  • Legacy Backlogs Release (now unused) changed from 14 to 23

#5 Updated by Eric Helms over 6 years ago

  • Target version changed from 63 to 66

#6 Updated by Eric Helms over 6 years ago

  • Target version changed from 66 to 67

#7 Updated by Eric Helms over 6 years ago

  • Status changed from Ready For Testing to Closed
  • Target version changed from 67 to 66

Also available in: Atom PDF