Skip to content

Commit

Permalink
test(IDX): skip some tests using the HEAD NNS canisters to save resou…
Browse files Browse the repository at this point in the history
…rces (dfinity#2078)

The release-testing workflow is very expensive in terms of vCPUs needed
for testnet VMs:
![Screenshot 2024-10-16 at 14 15
32](https://github.com/user-attachments/assets/9a9e7b31-3193-4fa7-80b7-bc75295518e8)

This actually prevents us from running concurrent release-testing
workflows. To help reduce resources a bit we disable a few test variants
that run the test against the HEAD NNS canisters, namely:

*
`//rs/tests/consensus/subnet_recovery:sr_app_large_with_tecdsa_test_head_nns`
*
`//rs/tests/message_routing/xnet:xnet_slo_120_subnets_staging_test_head_nns`

They will now only be tested using the mainnet NNS canisters.
  • Loading branch information
basvandijk authored Oct 16, 2024
1 parent a3bd65a commit bd862af
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 0 deletions.
1 change: 1 addition & 0 deletions rs/tests/consensus/subnet_recovery/BUILD.bazel
Original file line number Diff line number Diff line change
Expand Up @@ -228,6 +228,7 @@ system_test_nns(

system_test_nns(
name = "sr_app_large_with_tecdsa_test",
extra_head_nns_tags = ["manual"], # Let's not run this expensive test against the HEAD NNS canisters to save resources.
flaky = True,
proc_macro_deps = MACRO_DEPENDENCIES,
tags = [
Expand Down
1 change: 1 addition & 0 deletions rs/tests/message_routing/xnet/BUILD.bazel
Original file line number Diff line number Diff line change
Expand Up @@ -8,6 +8,7 @@ system_test_nns(
env = {
"XNET_TEST_CANISTER_WASM_PATH": "$(rootpath //rs/rust_canisters/xnet_test:xnet-test-canister)",
},
extra_head_nns_tags = ["manual"], # Let's not run this expensive test against the HEAD NNS canisters to save resources.
flaky = False,
proc_macro_deps = MACRO_DEPENDENCIES,
tags = [
Expand Down

0 comments on commit bd862af

Please sign in to comment.