Skip to content

Commit

Permalink
Don't sync twice b/c target chain is using ParityDB by default now
Browse files Browse the repository at this point in the history
when testing previously the live-sync-creditcoin job would use
RocksDB and then test-against-disconnected-live-node would resync using
ParityDB before we can disconnect it and restart the node!

Since v2.3 Testnet and Mainnet are already using ParityDB so this isn't
necessary.
  • Loading branch information
atodorov committed Sep 21, 2023
1 parent 3f31d9c commit eb37921
Showing 1 changed file with 5 additions and 41 deletions.
46 changes: 5 additions & 41 deletions .github/workflows/runtime-upgrade.yml
Original file line number Diff line number Diff line change
Expand Up @@ -424,47 +424,6 @@ jobs:
name: creditcoin-node
path: target/release

- name: Start creditcoin-node from current PR
run: |
# see https://opensource.com/article/18/5/how-find-ip-address-linux
IP_ADDRESS=$(curl https://ifconfig.me)
echo "INFO: IP_ADDRESS=$IP_ADDRESS"
chmod a+x ./target/release/creditcoin-node
./target/release/creditcoin-node --version
./target/release/creditcoin-node \
--name "test-node-$GITHUB_RUN_ID-attempt-$GITHUB_RUN_ATTEMPT" \
--chain ${{ needs.setup.outputs.target_chain }} \
--pruning archive \
--validator --mining-key 5DkPYq8hFiCeGxFBkz6DAwnTrvKevAJfTYrzFtr9hpDsEAU1 \
--bootnodes "${{ needs.setup.outputs.boot_node }}" \
--prometheus-external \
--telemetry-url "wss://telemetry.creditcoin.network/submit/ 0" \
--public-addr "/dns4/$IP_ADDRESS/tcp/50555" \
--port 50555 >creditcoin-node-disconnected-paritydb-sync.log 2>&1 &
- name: Wait for blockchain to start
run: |
./integration-tests/wait-for-creditcoin.sh
# wait again because of RocksDB -> ParityDB switch !
- name: Wait for creditcoin-node to sync past block number ${{ needs.setup.outputs.last_block_number }}
run: |
./scripts/wait-for-sync.sh ${{ needs.setup.outputs.last_block_number }}
- name: TERM creditcoin-node
continue-on-error: true
run: |
killall -TERM creditcoin-node
sleep 30
- name: KILL creditcoin-node
continue-on-error: true
run: |
killall -KILL creditcoin-node
sleep 30
- name: Start a disconnected creditcoin-node
run: |
chmod a+x ./target/release/creditcoin-node
Expand All @@ -486,6 +445,11 @@ jobs:
run: |
./integration-tests/wait-for-creditcoin.sh
# check this eventhough it should be true unless live-sync screws up!
- name: Check if creditcoin-node is past block number ${{ needs.setup.outputs.last_block_number }}
run: |
./scripts/wait-for-sync.sh ${{ needs.setup.outputs.last_block_number }}
- name: Install Docker engine
run: |
./scripts/install-docker-engine-from-upstream.sh
Expand Down

0 comments on commit eb37921

Please sign in to comment.