Skip to content

fix(ng-dev): limit the amount of CPUs used by workers - #436

Closed
alan-agius4 wants to merge 1 commit into
angular:mainfrom
alan-agius4:killed-worker
Closed

alan-agius4 wants to merge 1 commit into
angular:mainfrom
alan-agius4:killed-worker

Conversation

@alan-agius4

Copy link
Copy Markdown
Contributor

Some environments, like CircleCI which use Docker report a number of CPUs by the host and not the count of available.

This causes the task to be killed when formatting a large number of files due lack of resources.

https://app.circleci.com/pipelines/github/angular/angular-cli/20921/workflows/1f32e2ff-e726-41bc-a47c-9926f8b02f1d/jobs/290488

@alan-agius4

Copy link
Copy Markdown
Contributor Author

Some environments, like CircleCI which use Docker report a number of CPUs by the host and not the count of available.

This causes the task to be killed when formatting a large number of files due lack of resources.

https://app.circleci.com/pipelines/github/angular/angular-cli/20921/workflows/1f32e2ff-e726-41bc-a47c-9926f8b02f1d/jobs/290488

@josephperrott josephperrott left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM

@josephperrott josephperrott added the action: merge The PR is ready for merge by the caretaker label Feb 28, 2022
@josephperrott

Copy link
Copy Markdown
Member

This PR was merged into the repository by commit 104c49a.

@angular-automatic-lock-bot

Copy link
Copy Markdown

This issue has been automatically locked due to inactivity.
Please file a new issue if you are encountering a similar or related problem.

Read more about our automatic conversation locking policy.

This action has been performed automatically by a bot.

@angular-automatic-lock-bot angular-automatic-lock-bot Bot locked and limited conversation to collaborators Mar 31, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

action: merge The PR is ready for merge by the caretaker domain: ci

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants