This commit is contained in:
Eric 2025-02-06 18:07:33 +11:00
parent 54fb0d7172
commit e1bb35c86d
No known key found for this signature in database
2 changed files with 4 additions and 3 deletions

View File

@ -77,7 +77,7 @@ jobs:
## Part 3 Tests ##
- name: Integration tests
if: matrix.tests == 'integration' || matrix.tests == 'all'
run: make -j${ncpu} ENABLE_PARALLEL_TESTS=0 testIntegration
run: make -j${ncpu} PARALLEL=0 testIntegration
- name: Parallel integration tests
if: matrix.tests == 'integration-parallel'

View File

@ -320,11 +320,12 @@ proc buildCommand(
raiseTestManagerError "bad file name, testFile: " & test.config.testFile, parent
withLock(test.manager.codexPortLock):
# Increase the port by 100 to allow each test to run 100 codex nodes
# Increase the port by 1000 to allow each test to run 1000 codex nodes
# (clients, SPs, validators) giving a good chance the port will be free. We
# cannot rely on `nextFreePort` in multinodes entirely as there could be a
# concurrency issue where the port is determined free in mulitiple tests and
# then there is a clash during the run.
# then there is a clash during the run. Windows, in particular, does not
# like giving up ports.
let apiPort = await nextFreePort(test.manager.lastCodexApiPort + 1000)
test.manager.lastCodexApiPort = apiPort
let discPort = await nextFreePort(test.manager.lastCodexDiscPort + 1000)