Skip to content

Commit

Permalink
fix chunk fetching network compatibility zombienet test (#6988)
Browse files Browse the repository at this point in the history
Fix this zombienet test

It was failing because in
#6452 I enabled the v2
receipts for testnet genesis,
so the collators started sending v2 receipts with zeroed collator
signatures to old validators that were still checking those signatures
(which lead to disputes, since new validators considered the candidates
valid).

The fix is to also use an old image for collators, so that we don't
create v2 receipts.

We cannot remove this test yet because collators also perform chunk
recovery, so until all collators are upgraded, we need to maintain this
compatibility with the old protocol version (which is also why
systematic recovery was not yet enabled)
  • Loading branch information
alindima authored Jan 6, 2025
1 parent 6eca764 commit ffa90d0
Show file tree
Hide file tree
Showing 2 changed files with 7 additions and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -42,7 +42,8 @@ chain = "glutton-westend-local-{{id}}"

[parachains.collator]
name = "collator"
image = "{{CUMULUS_IMAGE}}"
# Use an old image that does not send out v2 receipts, as the old validators will still check the collator signatures.
image = "docker.io/paritypr/polkadot-parachain-debug:master-bde0bbe5"
args = ["-lparachain=debug"]

{% endfor %}
5 changes: 5 additions & 0 deletions prdoc/pr_6988.prdoc
Original file line number Diff line number Diff line change
@@ -0,0 +1,5 @@
doc: []

crates:
- name: polkadot
bump: none

0 comments on commit ffa90d0

Please sign in to comment.