Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: use allowed cpus from target container #256

Merged
merged 1 commit into from
Feb 5, 2025
Merged

Conversation

joshiste
Copy link
Member

@joshiste joshiste commented Feb 5, 2025

if a process is limited to a certain cpuset or
some of the configured processors are not online, we need to pass the number of workers to stress-ng as otherwise stress-ng will always use the configured number of processors.

@joshiste joshiste force-pushed the fix/repect-cpu-limit branch 5 times, most recently from a6c4343 to 4b61872 Compare February 5, 2025 15:56
@joshiste joshiste enabled auto-merge February 5, 2025 15:59
if a process is limited to a certain cpuset or
some of the configured processors are not online, we
need to pass the number of workers to stress-ng as
otherwise stress-ng will always use the configured
number of processors.
@joshiste joshiste force-pushed the fix/repect-cpu-limit branch from 4b61872 to 4689b68 Compare February 5, 2025 16:16
@joshiste joshiste merged commit 1f72800 into main Feb 5, 2025
14 checks passed
@github-actions github-actions bot locked and limited conversation to collaborators Feb 5, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant