Skip to content

Commit

Permalink
Make test-migrations-via-try-runtime CI job depend on live-sync-credi…
Browse files Browse the repository at this point in the history
…tcoin

this should reuse the database which is already on-disk and not cause a
second sync, hopefully reducing the overall execution time for this CI
job.

Also specify missing `--pruning archive` option to deal with errors
like:

Error = Call(Custom(ErrorObject { code: ServerError(-32000), message: "Client error: UnknownBlock: State already discarded for 0x7a6052272f51d086f3c0f89ced306b845ba93465591026107c6e236174d993d4", data: None }))
  • Loading branch information
atodorov committed Nov 9, 2023
1 parent f517168 commit e0e04e5
Showing 1 changed file with 5 additions and 1 deletion.
6 changes: 5 additions & 1 deletion .github/workflows/runtime-upgrade.yml
Original file line number Diff line number Diff line change
Expand Up @@ -631,6 +631,7 @@ jobs:
- build-sut
- setup
- setup-self-hosted
- live-sync-creditcoin
- deploy-github-runner
steps:
- uses: actions/checkout@v4
Expand Down Expand Up @@ -663,14 +664,17 @@ jobs:
echo "INFO: IP_ADDRESS=$IP_ADDRESS"
./target/release/creditcoin-node --version
# node should be reusing the database from the
# live-sync-creditcoin CI job listed as dependency above
./target/release/creditcoin-node \
--name "test-node-$GITHUB_RUN_ID-attempt-$GITHUB_RUN_ATTEMPT" \
--chain ${{ needs.setup.outputs.target_chain }} \
--validator --mining-key 5DkPYq8hFiCeGxFBkz6DAwnTrvKevAJfTYrzFtr9hpDsEAU1 \
--bootnodes "${{ needs.setup.outputs.boot_node }}" \
--rpc-max-request-size 200000 \
--rpc-max-response-size 200000 \
--prometheus-external \
--prometheus-external --pruning archive \
--telemetry-url "wss://telemetry.creditcoin.network/submit/ 0" \
--base-path /mnt \
--public-addr "/dns4/$IP_ADDRESS/tcp/30333" >creditcoin-node-used-for-try-runtime.log 2>&1 &
Expand Down

0 comments on commit e0e04e5

Please sign in to comment.