Skip to content

Commit 6916797

Browse files
committed
chore: Adjust unit test job runners to be hardcoded to 6 instead of 8, reducing I/O load and hopefully deflaking some tests
1 parent f836368 commit 6916797

File tree

1 file changed

+2
-4
lines changed

1 file changed

+2
-4
lines changed

.github/workflows/famedly-tests.yml

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -232,12 +232,10 @@ jobs:
232232
# coverage is already installed from the pyproject.toml file
233233
- run: poetry run pip install coverage-enable-subprocess
234234
# Normally, this Github runner has 4 cores(vCPUs) available to it.
235-
# Using only one trial test runner does not saturate the cpu. Rough experimentation in the past led to having 3
236-
# of these runners per core as optimal, but now coverage is required and there may be the need
237-
# to scale back a little. This should be 8 trial test runners/jobs. Adding in
235+
# Just use that number to define the number of runners. Adding in
238236
# --debug=core will display which measurement core is in use. For python 3.14+
239237
# this is educational.
240-
- run: poetry run coverage run --debug=core -m twisted.trial -j$((`nproc` * 2 )) tests
238+
- run: poetry run coverage run --debug=core -m twisted.trial -j$((`nproc`)) tests
241239
env:
242240
SYNAPSE_POSTGRES: ${{ matrix.job.database == 'postgres' || '' }}
243241
SYNAPSE_POSTGRES_HOST: /var/run/postgresql

0 commit comments

Comments
 (0)