Skip to content

Run tests on all logical cores - #849

Merged
nkiryanov merged 2 commits into
masterfrom
test-all-logical-cores
Jul 25, 2026
Merged

Run tests on all logical cores#849
nkiryanov merged 2 commits into
masterfrom
test-all-logical-cores

Conversation

@nkiryanov

Copy link
Copy Markdown
Contributor

logical and not autoauto counts physical cores, which GitHub runners report as 1.
It needs psutil, until now pulled in only as an ipython dependency, so pytest-xdist gets the extra.

More workers need more connections: the compose Postgres cap goes 10 → 100.

@nkiryanov

Copy link
Copy Markdown
Contributor Author

@f213, @kazqvaizer что думаете чтоб не брать логические процессоры по умолчанию?

Я использую где-то последний месяц и везде норм: и в ci и локально определяет как нужно.

ports:
- 5432:5432
command: --autovacuum=off --fsync=off --synchronous_commit=off --full_page_writes=off --work_mem=12MB --max-connections=10 --max_wal_senders=0
command: --autovacuum=off --fsync=off --synchronous_commit=off --full_page_writes=off --work_mem=12MB --max-connections=100 --max_wal_senders=0

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

тут побольше, т.к. 10 уже может не хватить


lint-mypy:
uv run mypy -n 4 src
uv run mypy --num-workers 4 src

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

это заодно: с длинными флагами не приходится гадать что за аргумент

@nkiryanov
nkiryanov requested review from f213 and kazqvaizer July 25, 2026 13:38
@nkiryanov
nkiryanov force-pushed the test-all-logical-cores branch from 326726b to f05139e Compare July 25, 2026 14:25
@nkiryanov
nkiryanov enabled auto-merge (squash) July 25, 2026 14:25
@nkiryanov
nkiryanov merged commit d07afd0 into master Jul 25, 2026
3 checks passed
@nkiryanov
nkiryanov deleted the test-all-logical-cores branch July 25, 2026 14:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants