From 5e7addf8f0475e630184edb97bc38fcd62ccc4fd Mon Sep 17 00:00:00 2001 From: Liam Aharon Date: Tue, 25 Jul 2023 01:10:55 +1000 Subject: [PATCH] ci: always run runtime migration checks (#7422) * add rococo test and run ci every pr * remove debug logging from ci output * fix run command * Update scripts/ci/gitlab/pipeline/check.yml Co-authored-by: Alexander Samusev <41779041+alvicsam@users.noreply.github.com> * make --checks explicit * fix checks * kick ci * kick ci * kick ci * kick ci --------- Co-authored-by: Alexander Samusev <41779041+alvicsam@users.noreply.github.com> --- scripts/ci/gitlab/pipeline/check.yml | 36 ++++++++++++++-------------- 1 file changed, 18 insertions(+), 18 deletions(-) diff --git a/scripts/ci/gitlab/pipeline/check.yml b/scripts/ci/gitlab/pipeline/check.yml index b6a3ba964028..a9076c9318cd 100644 --- a/scripts/ci/gitlab/pipeline/check.yml +++ b/scripts/ci/gitlab/pipeline/check.yml @@ -52,7 +52,6 @@ check-try-runtime: - cargo check --locked --features try-runtime --all # More info can be found here: https://github.com/paritytech/polkadot/pull/5865 -# Works only in PRs with E1 label .check-runtime-migration: stage: check extends: @@ -61,23 +60,13 @@ check-try-runtime: - .compiler-info script: - | - export has_runtimemigration_label=$(curl -sS -H "Accept: application/vnd.github+json" \ - -H "Authorization: token $GITHUB_PR_TOKEN" \ - https://api.github.com/repos/paritytech/polkadot/issues/$CI_COMMIT_REF_NAME/labels | grep "E0-runtime_migration" | wc -l) - - | - if [[ $has_runtimemigration_label != 0 ]]; then - echo "Found label runtime_migration. Running tests" - export RUST_LOG=remote-ext=debug,runtime=debug - echo "---------- Running try-runtime for ${NETWORK} ----------" - time cargo build --locked --release -p "$NETWORK"-runtime - time cargo run --locked --release --features try-runtime try-runtime \ - --runtime ./target/release/wbuild/"$NETWORK"-runtime/target/wasm32-unknown-unknown/release/"$NETWORK"_runtime.wasm \ - -lruntime=debug \ - --chain=${NETWORK}-dev \ - on-runtime-upgrade live --uri wss://${NETWORK}-try-runtime-node.parity-chains.parity.io:443 - else - echo "runtime_migration label not found. Skipping" - fi + export RUST_LOG=remote-ext=debug,runtime=debug + echo "---------- Running try-runtime for ${NETWORK} ----------" + time cargo build --release --locked -p "$NETWORK"-runtime + time cargo run --locked --release --features try-runtime try-runtime \ + --runtime ./target/release/wbuild/"$NETWORK"-runtime/target/wasm32-unknown-unknown/release/"$NETWORK"_runtime.wasm \ + --chain=${NETWORK}-dev \ + on-runtime-upgrade --checks=pre-and-post live --uri wss://${NETWORK}-try-runtime-node.parity-chains.parity.io:443 check-runtime-migration-polkadot: stage: check @@ -112,6 +101,17 @@ check-runtime-migration-westend: NETWORK: "westend" allow_failure: true # FIXME https://github.com/paritytech/substrate/issues/13107 +check-runtime-migration-rococo: + stage: check + extends: + - .docker-env + - .test-pr-refs + - .compiler-info + - .check-runtime-migration + variables: + NETWORK: "rococo" + allow_failure: true # FIXME https://github.com/paritytech/substrate/issues/13107 + # is broken, need to fix check-no-default-features: stage: check