Skip to content
This repository has been archived by the owner on Nov 15, 2023. It is now read-only.

Polkadot companion for Substrate#10463 #4519

Merged
merged 12 commits into from
Jan 5, 2022
Merged

Polkadot companion for Substrate#10463 #4519

merged 12 commits into from
Jan 5, 2022

Conversation

acatangiu
Copy link
Contributor

@acatangiu acatangiu commented Dec 13, 2021

Companion for paritytech/substrate#10463

Include the chain uid to the Grandpa protocol name.
Include the chain uid to the Beefy protocol name.
Related to paritytech/substrate#7252 and paritytech/substrate#7746 and paritytech/grandpa-bridge-gadget#240.

@acatangiu acatangiu added B0-silent Changes should not be mentioned in any release notes A4-companion A PR that needs a companion PR to merge in parallel for one of its downstream dependencies. C1-low PR touches the given topic and has a low impact on builders. D3-trivial 🧸 PR contains trivial changes in a runtime directory that do not require an audit. labels Dec 13, 2021
@acatangiu acatangiu self-assigned this Dec 13, 2021
node/service/src/lib.rs Outdated Show resolved Hide resolved
node/service/src/lib.rs Outdated Show resolved Hide resolved
@acatangiu
Copy link
Contributor Author

@bkchr addressed comments, ptal

@acatangiu acatangiu changed the title Companion for Substrate#10463 Polkadot companion for Substrate#10463 Jan 5, 2022
@paritytech-processbot
Copy link

Waiting for commit status.

@pepyakin
Copy link
Contributor

pepyakin commented Jan 6, 2022

The latest master have continuous-integration/gitlab-zombienet-tests-parachains-smoke-test failing. It apparently started with this PR. Can you, @acatangiu, look into that?

@acatangiu
Copy link
Contributor Author

The latest master have continuous-integration/gitlab-zombienet-tests-parachains-smoke-test failing. It apparently started with this PR. Can you, @acatangiu, look into that?

Sure, I'll look into it. How do I run that test? 😄

@pepyakin
Copy link
Contributor

pepyakin commented Jan 6, 2022

TBH, I have no idea 🤷 Maybe @pepoviola could help with that?

@ordian
Copy link
Member

ordian commented Jan 6, 2022

I suspect #4668 fixes the issue (to be confirmed).

@pepoviola
Copy link
Contributor

@ordian, yes looks like an issue with the image version. Also notice that the malus test are no using a fixed image of the collator. I don't know if you also want to update that test.
Thanks!

@ordian
Copy link
Member

ordian commented Jan 6, 2022

It does, but it's not clear why.
According to paritytech/substrate#10463 (comment), the change is backwards compatible for collators as well. Were there any breaking changes other than this PR in substrate maybe?

sandreim added a commit that referenced this pull request Jan 7, 2022
* Grandpa and Beefy protocol names include chain id

Signed-off-by: acatangiu <adrian@parity.io>

* chain_spec: include fork id

* use simplified protocol name

* fix after merge

* avoid using hash default, even for protocol names

* update lockfile for substrate

Co-authored-by: parity-processbot <>
paritytech-processbot bot pushed a commit that referenced this pull request Jan 10, 2022
* Remove sleep and use polkadot test service

Signed-off-by: Andrei Sandu <andrei-mihail@parity.io>

* updates

Signed-off-by: Andrei Sandu <andrei-mihail@parity.io>

* Fix other tests

Signed-off-by: Andrei Sandu <andrei-mihail@parity.io>

* Run metrics tests separately

Signed-off-by: Andrei Sandu <andrei-mihail@parity.io>

* copy some substrate utilities

Signed-off-by: Andrei Sandu <andrei-mihail@parity.io>

* update runtime metric test

Signed-off-by: Andrei Sandu <andrei-mihail@parity.io>

* Remove sleep from cli tests

Signed-off-by: Andrei Sandu <andrei-mihail@parity.io>

* cargo

Signed-off-by: Andrei Sandu <andrei-mihail@parity.io>

* Polkadot companion for Substrate#10463 (#4519)

* Grandpa and Beefy protocol names include chain id

Signed-off-by: acatangiu <adrian@parity.io>

* chain_spec: include fork id

* use simplified protocol name

* fix after merge

* avoid using hash default, even for protocol names

* update lockfile for substrate

Co-authored-by: parity-processbot <>

* configuration: Update upgrade validation delay doc (#4662)

* typo

Signed-off-by: Andrei Sandu <andrei-mihail@parity.io>

* review feedback

Signed-off-by: Andrei Sandu <andrei-mihail@parity.io>

* cargo lock

Signed-off-by: Andrei Sandu <andrei-mihail@parity.io>

* use testnet profile

Signed-off-by: Andrei Sandu <andrei-mihail@parity.io>

* Don't run with runtime-benchmark feature

Signed-off-by: Andrei Sandu <andrei-mihail@parity.io>

* conditional compile up one level

Signed-off-by: Andrei Sandu <andrei-mihail@parity.io>

Co-authored-by: Sergei Shulepov <sergei@parity.io>
Wizdave97 pushed a commit to ComposableFi/polkadot that referenced this pull request Feb 3, 2022
* Grandpa and Beefy protocol names include chain id

Signed-off-by: acatangiu <adrian@parity.io>

* chain_spec: include fork id

* use simplified protocol name

* fix after merge

* avoid using hash default, even for protocol names

* update lockfile for substrate

Co-authored-by: parity-processbot <>
Wizdave97 pushed a commit to ComposableFi/polkadot that referenced this pull request Feb 3, 2022
* Remove sleep and use polkadot test service

Signed-off-by: Andrei Sandu <andrei-mihail@parity.io>

* updates

Signed-off-by: Andrei Sandu <andrei-mihail@parity.io>

* Fix other tests

Signed-off-by: Andrei Sandu <andrei-mihail@parity.io>

* Run metrics tests separately

Signed-off-by: Andrei Sandu <andrei-mihail@parity.io>

* copy some substrate utilities

Signed-off-by: Andrei Sandu <andrei-mihail@parity.io>

* update runtime metric test

Signed-off-by: Andrei Sandu <andrei-mihail@parity.io>

* Remove sleep from cli tests

Signed-off-by: Andrei Sandu <andrei-mihail@parity.io>

* cargo

Signed-off-by: Andrei Sandu <andrei-mihail@parity.io>

* Polkadot companion for Substrate#10463 (paritytech#4519)

* Grandpa and Beefy protocol names include chain id

Signed-off-by: acatangiu <adrian@parity.io>

* chain_spec: include fork id

* use simplified protocol name

* fix after merge

* avoid using hash default, even for protocol names

* update lockfile for substrate

Co-authored-by: parity-processbot <>

* configuration: Update upgrade validation delay doc (paritytech#4662)

* typo

Signed-off-by: Andrei Sandu <andrei-mihail@parity.io>

* review feedback

Signed-off-by: Andrei Sandu <andrei-mihail@parity.io>

* cargo lock

Signed-off-by: Andrei Sandu <andrei-mihail@parity.io>

* use testnet profile

Signed-off-by: Andrei Sandu <andrei-mihail@parity.io>

* Don't run with runtime-benchmark feature

Signed-off-by: Andrei Sandu <andrei-mihail@parity.io>

* conditional compile up one level

Signed-off-by: Andrei Sandu <andrei-mihail@parity.io>

Co-authored-by: Sergei Shulepov <sergei@parity.io>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
A4-companion A PR that needs a companion PR to merge in parallel for one of its downstream dependencies. B0-silent Changes should not be mentioned in any release notes C1-low PR touches the given topic and has a low impact on builders. D3-trivial 🧸 PR contains trivial changes in a runtime directory that do not require an audit.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants