Skip to content

Commit 48d8dba

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 48d8dba

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

.github/workflows/famedly-tests.yml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -234,10 +234,10 @@ jobs:
234234
# Normally, this Github runner has 4 cores(vCPUs) available to it.
235235
# Using only one trial test runner does not saturate the cpu. Rough experimentation in the past led to having 3
236236
# 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
237+
# to scale back a little. This should be 6 trial test runners/jobs. Adding in
238238
# --debug=core will display which measurement core is in use. For python 3.14+
239239
# this is educational.
240-
- run: poetry run coverage run --debug=core -m twisted.trial -j$((`nproc` * 2 )) tests
240+
- run: poetry run coverage run --debug=core -m twisted.trial -j6 tests
241241
env:
242242
SYNAPSE_POSTGRES: ${{ matrix.job.database == 'postgres' || '' }}
243243
SYNAPSE_POSTGRES_HOST: /var/run/postgresql

0 commit comments

Comments
 (0)