[Zombienet] add dummy upgrade test

[pepoviola]

• Add upgrade test, close zombienet: test parachain upgrades #4599 .

UPDATED cc @pepyakin / @ordian

Zombienet new available commands

• <node name>: parachain perform dummy upgrade within x seconds

Zombienet will connect to the collator and get the :code, after decompress a custom section is appended to the wasm and re-compressed. The new compressed code is used to perform the upgrade and then zombienet will try to match the new hash in the <node name> defined for x amount of seconds.

• <node name>: parachain perform upgrade with within x seconds

Zombienetwill download the wasm from the provided url and use it for perform the upgrade. Then will try to match the new hash in the <node name> defined for x amount of seconds.

NOTE: The images for the relay chain nodes and collator are fixed to polkadot:latest and polkadot-collator:latest. I try using the paritypr for the relay chain nodes but the collator produce this error:

2022-01-13 10:45:04 | Error: Input("Relay chain argument error: Invalid input: Error parsing spec file: unknown field `forkId` at line 157 column 1")

I'm not sure if we want to set this test to run with those fixed version in the CI or we want to make it works with the image produced by the pr (docker.io/paritypr/synth-wave).

Thanks!!

[ordian]

Thanks!

A couple of things:

1. The naming generate dummy upgrade is a bit incomplete, because IIUC you also apply the upgrade, not only generate it.
   Is it possible to rename to chain upgrade dummy?
2. Is it possible to support urls instead of path for runtime upgrade? I assume we don't want to store a few megabytes in the git history.
3. Maybe replace chain with parachain 100 for clarity?

[pepoviola]

Hi @ordian, thanks for the feedback! Related to your points:

• The naming generate dummy upgrade is a bit incomplete, because IIUC you also apply the upgrade, not only generate it.

Yes, generate and perform the upgrade. I think we can use perform dummy upgrade since the generation is implied.

Is it possible to rename to chain upgrade dummy?

Yes, we can agree to use a better expression.

• Is it possible to support urls instead of path for runtime upgrade? I assume we don't want to store a few megabytes in the git history.

Yes, support url is a better option here.

• Maybe replace chain with parachain 100 for clarity?

Something like:

alice: parachain 100 perform dummy upgrade

or

alice: parachain 100 perform upgrade with <wasm url>

quick question: Do we need to be explicit about parachain Id?

Thanks!

[ordian]

quick question: Do we need to be explicit about parachain Id?

For tests where we have more than one parachain. But we currently don't have such tests.

[drahnr]

What's the wasm blob that is used for the upgrade?

[pepoviola]

Hi @drahnr, for this use case we generate a new wasm from the previous one by appending a custom section.

A high level process definition:

Zombienet will connect to the collator and get the :code, after decompress a custom section is appended to the wasm and re-compressed. The new compressed code is used to perform the upgrade and then zombienet will try to match the new hash in the <node name> defined for x amount of seconds.

Also, as mention in the pr this test is using polkadot-collator and polkadot latest image versions because I try with the image build within the pr but doesn't work with the collator version.

Thx!

[ordian]

Nice!