polkadot-fellows / runtimes

The various runtimes which make up the core subsystems of networks for which the Fellowship is represented.
GNU General Public License v3.0
135 stars 85 forks source link

Update bridges integration-tests instructions #366

Closed bkontur closed 2 months ago

bkontur commented 3 months ago

This PR adjusts the instructions on how to run bridges integration tests.

Future improvements: https://github.com/paritytech/parity-bridges-common/issues/3016

bkontur commented 3 months ago

/merge

fellowship-merge-bot[bot] commented 3 months ago

Enabled auto-merge in Pull Request

Available commands - `/merge`: Enables auto-merge for Pull Request - `/merge cancel`: Cancels auto-merge for Pull Request - `/merge help`: Shows this menu For more information see the [documentation](https://github.com/paritytech/auto-merge-bot)
bkontur commented 3 months ago

Looks quite complicated and also raises the question why you need the sudo pallet with fast runtime? Let's merge this, but would be nice to improve this.

Yes, you're right. That's why I created issue #3016. When we started with the zombienet bridges tests, preopen_hrmp stopped working at some point (more info). As a workaround, I used force_open_hrmp_channel because establish_system_channel didn't exist at that time. Sudo is needed for the init-bridge governance call from the relay, but that could probably be replaced with a genesis setup or with Chopsticks and governance call.

fellowship-merge-bot[bot] commented 3 months ago

Failed to update PR ❌

There was an error while trying to keep this PR up-to-date

You may have conflicts ‼️ or may have to manually sync it with the target branch 👉❇️

More info in the logs 📋

fellowship-merge-bot[bot] commented 2 months ago

Failed to update PR ❌

There was an error while trying to keep this PR up-to-date

You may have conflicts ‼️ or may have to manually sync it with the target branch 👉❇️

More info in the logs 📋

fellowship-merge-bot[bot] commented 2 months ago

Failed to update PR ❌

There was an error while trying to keep this PR up-to-date

You may have conflicts ‼️ or may have to manually sync it with the target branch 👉❇️

More info in the logs 📋