Skip to content

Commit

Permalink
leave comment + rename job
Browse files Browse the repository at this point in the history
  • Loading branch information
@svyatonik
svyatonik committed Jan 12, 2024
1 parent c4ba86f commit 9b36e5c
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion .gitlab/pipeline/zombienet/bridges.yml
View file
Original file line number Diff line number Diff line change
Expand Up @@ -45,11 +45,12 @@
- cp /tmp/polkadot.* ./zombienet-logs/
# copy logs of our polkadot-parachain binary wrapper (polkadot-parachain-x.sh)
- cp /tmp/polkadot-parachain.* ./zombienet-logs/
# following lines are causing spurious test failures ("At least one of the nodes fails to start")
# retry: 2
# tags:
# - zombienet-polkadot-integration-test

run-zombienet-bridges-tests:
zombienet-bridges-0001-asset-transfer-works:

This comment has been minimized.

Sign in to view

Copy link
@bkontur

bkontur Jan 12, 2024

Contributor

@svyatonik what about adding discriminator "rococo-westend" to job name or run_tests.sh file name? maybe we will add something similar for rococo-eth?

This comment has been minimized.

Sign in to view

Copy link
@svyatonik

svyatonik Jan 12, 2024

Author Contributor

Yes - the idea is to add test-name argument. By default run-tests.sh will be running all tests, but when argument is passed, it'll start tests, starting with this prefix (0001). I have some more test cases - especially interested in ensuring that no extra headers are synced (i.e. only mandatory and those required for messages sync). Let's add this argument later too, if you don't mind

This comment has been minimized.

Sign in to view

Copy link
@bkontur

bkontur Jan 12, 2024

Contributor

aha, ok, cool, got it
extends:
- .zombienet-bridges-common
script:
Expand Down

0 comments on commit 9b36e5c

Please sign in to comment.