Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Genesis Presets' para-id and the chain-extenstion para-id might diverge #7618

Closed
2 tasks done
clangenb opened this issue Feb 19, 2025 · 1 comment
Closed
2 tasks done
Labels
I2-bug The node fails to follow expected behavior. I10-unconfirmed Issue might be valid, but it's not yet known.

Comments

@clangenb
Copy link
Contributor

clangenb commented Feb 19, 2025

Is there an existing issue?

  • I have searched the existing issues

Experiencing problems? Have you tried our Stack Exchange first?

  • This is not a support question.

Description of bug

In the genesis config presets we assume the established system chain para-id, as can be seen here. Yet, in the polkadot-parachain chain-spec, we also have to pass the para-id to the chain extensions, as can be seen here.

If one is not careful (as it happened to me 🙈 in https://github.com/paritytech/polkadot-sdk/pull/7476/files?notification_referrer_id=NT_kwDOAkHJB7QxNDYzNzE3OTM0MjozNzg2NTczNQ#discussion_r1959229620) it can be that a discrepancy is introduced.

It would be better if we allow passing the para-id into the genesis preset builder, so that we can cleanly propagate the para-id from the top to the very button.

Steps to reproduce

No response

@clangenb clangenb added I10-unconfirmed Issue might be valid, but it's not yet known. I2-bug The node fails to follow expected behavior. labels Feb 19, 2025
@bkchr
Copy link
Member

bkchr commented Feb 19, 2025

I see you already found: #7384

So yeah, this issue is actually a duplicate.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
I2-bug The node fails to follow expected behavior. I10-unconfirmed Issue might be valid, but it's not yet known.
Projects
None yet
Development

No branches or pull requests

2 participants