Fix CI (#1316)

* Fix deterministic WASM check Signed-off-by: Oliver Tale-Yazdi <[email protected]> * Fix build-staking-miner Signed-off-by: Oliver Tale-Yazdi <[email protected]> * Remove Kusama and Polkadot runtime-migration checks Will be removed in #1304 anyway. Signed-off-by: Oliver Tale-Yazdi <[email protected]> --------- Signed-off-by: Oliver Tale-Yazdi <[email protected]>
paritytech · Aug 30, 2023 · 36b4f25 · 36b4f25
1 parent 5462409
commit 36b4f25
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 31 deletions.
diff --git a/.gitlab/pipeline/build.yml b/.gitlab/pipeline/build.yml
@@ -88,7 +88,7 @@ build-staking-miner:
     - job: build-malus
       artifacts: false
   script:
-    - time cargo build --locked --release --package staking-miner
+    - time cargo build -q --locked --release --package staging-staking-miner
     # # pack artifacts
     # - mkdir -p ./artifacts
     # - mv ./target/release/staking-miner ./artifacts/.
@@ -353,4 +353,3 @@ build-subkey-linux:
 #   after_script: [""]
 #   tags:
 #     - osx
-
diff --git a/.gitlab/pipeline/check.yml b/.gitlab/pipeline/check.yml
@@ -121,30 +121,6 @@ test-rust-feature-propagation:
           --runtime ./target/release/wbuild/"$NETWORK"-runtime/target/wasm32-unknown-unknown/release/"$NETWORK"_runtime.wasm \
           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
-  extends:
-    - .docker-env
-    - .test-pr-refs
-    - .check-runtime-migration
-  variables:
-    NETWORK: "polkadot"
-  allow_failure: true # FIXME https://github.com/paritytech/substrate/issues/13107
-
-check-runtime-migration-kusama:
-  stage: check
-  # DAG
-  needs:
-    - job: check-runtime-migration-polkadot
-      artifacts: false
-  extends:
-    - .docker-env
-    - .test-pr-refs
-    - .check-runtime-migration
-  variables:
-    NETWORK: "kusama"
-  allow_failure: true # FIXME https://github.com/paritytech/substrate/issues/13107
-
 check-runtime-migration-westend:
   stage: check
   extends:
@@ -153,7 +129,6 @@ check-runtime-migration-westend:
     - .check-runtime-migration
   variables:
     NETWORK: "westend"
-  allow_failure: true # FIXME https://github.com/paritytech/substrate/issues/13107
 
 check-runtime-migration-rococo:
   stage: check
@@ -167,7 +142,6 @@ check-runtime-migration-rococo:
     - .check-runtime-migration
   variables:
     NETWORK: "rococo"
-  allow_failure: true # FIXME https://github.com/paritytech/substrate/issues/13107
 
 find-fail-ci-phrase:
   stage: check

diff --git a/.gitlab/test_deterministic_wasm.sh b/.gitlab/test_deterministic_wasm.sh
@@ -1,15 +1,17 @@
 #!/usr/bin/env bash
+set -e
 
 #shellcheck source=../common/lib.sh
 source "$( cd "$( dirname "${BASH_SOURCE[0]}" )" >/dev/null 2>&1 && pwd )/common/lib.sh"
 
 # build runtime
-WASM_BUILD_NO_COLOR=1 cargo build --verbose --release -p kusama-runtime -p polkadot-runtime -p westend-runtime
+WASM_BUILD_NO_COLOR=1 cargo build -q --locked --release -p staging-kusama-runtime -p polkadot-runtime -p westend-runtime
 # make checksum
 sha256sum target/release/wbuild/*-runtime/target/wasm32-unknown-unknown/release/*.wasm > checksum.sha256
-# clean up - FIXME: can we reuse some of the artifacts?
+
 cargo clean
+
 # build again
-WASM_BUILD_NO_COLOR=1 cargo build --verbose --release -p kusama-runtime -p polkadot-runtime -p westend-runtime
+WASM_BUILD_NO_COLOR=1 cargo build -q --locked --release -p staging-kusama-runtime -p polkadot-runtime -p westend-runtime
 # confirm checksum
 sha256sum -c checksum.sha256