Tutorials: Update parachain tutorial #234

CrackTheCode016 · 2024-12-10T02:01:12Z

Content Changes

This PR aims to better illustrate the process of deploying on Paseo and to use some of the 'cutting' edge tools which should be available soon (tm), such as:

polkadot-omni-node
chain-spec-builder

This PR also fills in some missing blanks throughout the Paseo deployment process.

IA Changes

This PR adds a new section to the IA called "Deploy on Relay Chain", wherein should contain tutorials relating to deploying on various networks.

Closes #176

CrackTheCode016 · 2024-12-10T21:54:51Z

Note that the IA may change in the future, but this should be fine for now.

tutorials/polkadot-sdk/parachains/deploy-to-relay-chain/deploy-on-paseo.md

tutorials/polkadot-sdk/parachains/deploy-to-relay-chain/index.md

kianenigma · 2024-12-12T09:32:42Z

requested review from some omni-node devs within Parity.

CrackTheCode016 · 2024-12-12T12:36:45Z

@kianenigma FYI - The versions of polkadot-omni-node and chain-spec-builder used are off of the master branch (installed from git via cargo install). I assume these will be deployed with the next major release?

iulianbarbu · 2024-12-12T13:00:00Z

@kianenigma FYI - The versions of polkadot-omni-node and chain-spec-builder used are off of the master branch (installed from git via cargo install). I assume these will be deployed with the next major release?

The binaries should be downloadable from the release candidate artifacts here: https://github.com/paritytech/polkadot-sdk/releases/tag/polkadot-stable2412-rc2, but they will also be part of the next major polkadot-sdk release page, on github.

However the latest crates versions can not be found anywhere yet, even with the release candidate. They should be published with the next major release.

Co-authored-by: Nicolás Hussein <[email protected]>

...t-sdk/parachains/connect-to-relay-chain/acquire-a-testnet-slot/acquire-a-testnet-slot-2.html

tutorials/polkadot-sdk/parachains/deploy-to-relay-chain/deploy-on-paseo.md

tutorials/polkadot-sdk/parachains/deploy-to-relay-chain/index.md

tutorials/polkadot-sdk/parachains/deploy-to-relay-chain/deploy-on-paseo.md

skunert

In general this looks good to me 👍

...t-sdk/parachains/connect-to-relay-chain/acquire-a-testnet-slot/acquire-a-testnet-slot-2.html

tutorials/polkadot-sdk/parachains/deploy-to-relay-chain/deploy-on-paseo.md

tutorials/polkadot-sdk/parachains/deploy-to-relay-chain/index.md

…evelopers/polkadot-docs into bd-parachain-tutorial

…-on-paseo.md Co-authored-by: Iulian Barbu <[email protected]>

iulianbarbu · 2024-12-12T16:16:43Z

Also, I think this PR closes: #176. Maybe worth adding it in the PR description.

CrackTheCode016 · 2024-12-12T19:45:28Z

@0xLucca @nhussein11 This PR can probably be merged now, but we should test some of the local tutorials to make sure they are in working order with these new binaries before they are merged. I can address this tomorrow and make a new PR to ensure they are working if needed. WDYT?

CrackTheCode016 · 2024-12-12T19:47:32Z

Also, it might be nice to have a page somewhere to explain the role of these different binaries, and links to where you can install them. Maybe even at the beginning of tutorials, would love to hear feedback on this

iulianbarbu

Overall looks good!👍 I haven't rendered the docs to check how they look like, and follow precisely along the tutorial, but I can take a look tomorrow more closely. Thanks!

iulianbarbu · 2024-12-12T20:07:09Z

tutorials/polkadot-sdk/parachains/connect-to-relay-chain/prepare-parachain.md

-    ./target/release/parachain-template-node build-spec \
-      --disable-default-bootnode > plain-parachain-chainspec.json
+    chain-spec-builder create --relay-chain paseo \
+        --para-id 4508 \


Should we have a placeholder instead of 4508 here?

I think the only reason it's that specific ID is because earlier the para id reserved is 4508

iulianbarbu · 2024-12-12T20:10:19Z

tutorials/polkadot-sdk/parachains/deploy-to-relay-chain/deploy-on-paseo.md

-    ./target/release/parachain-template-node build-spec \
-      --disable-default-bootnode > plain-parachain-chainspec.json
+    chain-spec-builder create --relay-chain paseo \
+    --para-id 4508 \


tutorials/polkadot-sdk/parachains/connect-to-relay-chain/prepare-parachain.md

kianenigma · 2024-12-13T11:25:30Z

Note that the matter of specifying the parachain ID in the template is a bit shitty now, but Shawn improved it lately here:

https://github.com/paritytech/polkadot-sdk/pull/6744/files

One of the areas is a hardcoded para id in the node.

one is the genesis config

one in the runtime

kianenigma · 2024-12-13T11:27:59Z

This PR adds a new section to the IA called "Deploy on Relay Chain", wherein should contain tutorials relating to deploying on various networks.

Can you please make an issue to address later that you can basically skip setting up a local relay by using --experimental-rpc-endpoint?

0xLucca

LGTM!

nhussein11

LGTM!

* updates so far * Add finishing touches * comments * Apply suggestions from code review Co-authored-by: Nicolás Hussein <[email protected]> * change to chain-spec-builder * Update tutorials/polkadot-sdk/parachains/deploy-to-relay-chain/deploy-on-paseo.md Co-authored-by: Iulian Barbu <[email protected]> * feedback * replace parachain-template-node * fix most instructions * few things --------- Co-authored-by: Nicolás Hussein <[email protected]> Co-authored-by: Iulian Barbu <[email protected]>

updates so far

2524668

CrackTheCode016 added the In Progress label Dec 10, 2024

CrackTheCode016 requested a review from a team as a code owner December 10, 2024 02:01

CrackTheCode016 marked this pull request as draft December 10, 2024 02:01

Add finishing touches

56e3773

CrackTheCode016 marked this pull request as ready for review December 10, 2024 21:54

CrackTheCode016 requested review from 0xLucca and nhussein11 December 10, 2024 21:54

CrackTheCode016 added Ready for Review and removed In Progress labels Dec 10, 2024

nhussein11 reviewed Dec 11, 2024

View reviewed changes

tutorials/polkadot-sdk/parachains/deploy-to-relay-chain/deploy-on-paseo.md Outdated Show resolved Hide resolved

nhussein11 reviewed Dec 11, 2024

View reviewed changes

tutorials/polkadot-sdk/parachains/deploy-to-relay-chain/index.md Outdated Show resolved Hide resolved

tutorials/polkadot-sdk/parachains/deploy-to-relay-chain/index.md Outdated Show resolved Hide resolved

tutorials/polkadot-sdk/parachains/deploy-to-relay-chain/index.md Show resolved Hide resolved

CrackTheCode016 and others added 2 commits December 12, 2024 08:05

comments

efa9306

Apply suggestions from code review

9a9943e

Co-authored-by: Nicolás Hussein <[email protected]>

iulianbarbu reviewed Dec 12, 2024

View reviewed changes

skunert approved these changes Dec 12, 2024

View reviewed changes

CrackTheCode016 and others added 3 commits December 12, 2024 10:55

change to chain-spec-builder

0182e2b

Merge branch 'bd-parachain-tutorial' of https://github.com/polkadot-d…

b148b1a

…evelopers/polkadot-docs into bd-parachain-tutorial

Update tutorials/polkadot-sdk/parachains/deploy-to-relay-chain/deploy…

f61b0c6

…-on-paseo.md Co-authored-by: Iulian Barbu <[email protected]>

CrackTheCode016 added 3 commits December 12, 2024 14:25

feedback

660e117

replace parachain-template-node

1e27654

fix most instructions

ab256f1

CrackTheCode016 requested review from iulianbarbu and nhussein11 December 12, 2024 19:45

iulianbarbu approved these changes Dec 12, 2024

View reviewed changes

kianenigma reviewed Dec 13, 2024

View reviewed changes

tutorials/polkadot-sdk/parachains/connect-to-relay-chain/prepare-parachain.md Show resolved Hide resolved

kianenigma approved these changes Dec 13, 2024

View reviewed changes

few things

fabd1fb

CrackTheCode016 mentioned this pull request Dec 13, 2024

Skipping "Local Relay Chain" Tutorial with --experimental-rpc-endpoint #253

Open

0xLucca approved these changes Dec 13, 2024

View reviewed changes

Merge branch 'master' into bd-parachain-tutorial

13cbb2b

CrackTheCode016 requested a review from 0xLucca December 13, 2024 13:51

nhussein11 approved these changes Dec 13, 2024

View reviewed changes

0xLucca merged commit 37ee5fe into master Dec 13, 2024
1 check passed

0xLucca deleted the bd-parachain-tutorial branch December 13, 2024 14:26

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Tutorials: Update parachain tutorial #234

Tutorials: Update parachain tutorial #234

CrackTheCode016 commented Dec 10, 2024 •

edited

Loading

CrackTheCode016 commented Dec 10, 2024

kianenigma commented Dec 12, 2024

CrackTheCode016 commented Dec 12, 2024

iulianbarbu commented Dec 12, 2024

skunert left a comment

iulianbarbu commented Dec 12, 2024

CrackTheCode016 commented Dec 12, 2024

CrackTheCode016 commented Dec 12, 2024

iulianbarbu left a comment

iulianbarbu Dec 12, 2024

CrackTheCode016 Dec 13, 2024

iulianbarbu Dec 12, 2024

kianenigma commented Dec 13, 2024

kianenigma commented Dec 13, 2024

0xLucca left a comment

nhussein11 left a comment

Tutorials: Update parachain tutorial #234

Tutorials: Update parachain tutorial #234

Conversation

CrackTheCode016 commented Dec 10, 2024 • edited Loading

Content Changes

IA Changes

CrackTheCode016 commented Dec 10, 2024

kianenigma commented Dec 12, 2024

CrackTheCode016 commented Dec 12, 2024

iulianbarbu commented Dec 12, 2024

skunert left a comment

Choose a reason for hiding this comment

iulianbarbu commented Dec 12, 2024

CrackTheCode016 commented Dec 12, 2024

CrackTheCode016 commented Dec 12, 2024

iulianbarbu left a comment

Choose a reason for hiding this comment

iulianbarbu Dec 12, 2024

Choose a reason for hiding this comment

CrackTheCode016 Dec 13, 2024

Choose a reason for hiding this comment

iulianbarbu Dec 12, 2024

Choose a reason for hiding this comment

kianenigma commented Dec 13, 2024

kianenigma commented Dec 13, 2024

0xLucca left a comment

Choose a reason for hiding this comment

nhussein11 left a comment

Choose a reason for hiding this comment

CrackTheCode016 commented Dec 10, 2024 •

edited

Loading