ci(test): parallelize slow provider tests in CI workflow #697
Conversation
|
graham-chainlink
force-pushed
the
ggoh/reduce-canton-ctf-validators
branch
from
cc509ee to
d3b4a13
Compare
graham-chainlink
changed the title
| giveSelector: chainsel.CANTON_LOCALNET.Selector, | ||
| giveConfig: CTFChainProviderConfig{ | ||
| NumberOfValidators: 5, | ||
| NumberOfValidators: 1, |
There was a problem hiding this comment.
this can be 1 instead of spinning up 5 validators to save some setup time.
There was a problem hiding this comment.
Pull request overview
This PR optimizes CI performance by parallelizing slow provider tests that involve Docker container startup overhead.
Changes:
- Separated Aptos, Canton, TON, and TRON provider tests into dedicated jobs with appropriate timeouts (5-10 minutes)
- Reduced Canton test configuration from 5 validators to 1 for faster execution
- Configured remaining fast provider tests to run together with exclusion patterns
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
| chain/canton/provider/ctf_provider_test.go | Reduced number of validators from 5 to 1 to speed up Canton tests |
| .github/workflows/pull-request-main.yml | Split single provider test job into five parallel jobs with specific timeout configurations |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Split provider tests into separate jobs to improve CI performance: - Aptos, Canton, TON, and TRON providers now run in dedicated jobs - Canton gets 10min timeout (takes ~4min), others get 5min - Remaining fast providers run together with exclusion pattern Also reduced Canton test validators from 5 to 1 for faster execution.
graham-chainlink
force-pushed
the
ggoh/reduce-canton-ctf-validators
branch
from
d3b4a13 to
d26c572
Compare
|
| # disable the checkptr runtime check due a false positive in github.com/xssnick/tonutils-go | ||
| # causing tests in ci to fail "fatal error: checkptr: pointer arithmetic result points to invalid allocation" | ||
| # https://github.com/xssnick/tonutils-go/issues/310 | ||
| go-test-cmd: go test -race -gcflags=all=-d=checkptr=0 -coverprofile=coverage.txt ./chain/ton/provider/... |
There was a problem hiding this comment.
This command has extra gcflags was that intentional?
There was a problem hiding this comment.
yeah ton needs the extra flag, refer to the comment # disable the checkptr runtime check due a false positive in github.com/xssnick/tonutils-go
There was a problem hiding this comment.
this existed in previous change
3 participants
Split provider tests into separate jobs to improve CI performance:
Also reduced Canton test validators from 5 to 1 for faster execution.