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

feat: add demo spec #1593

Merged
merged 1 commit into from
Oct 17, 2023
Merged

feat: add demo spec #1593

merged 1 commit into from
Oct 17, 2023

Conversation

wischli
Copy link
Contributor

@wischli wischli commented Oct 16, 2023

Description

  • Adds demo spec which enables syncing with Demo chain from scratch with latest docker image
  • Without this PR, demo can only be synched via centrifugeio/centrifuge-chain:demo-genesis-20220801173354-feffc7c

Generated via

  1. Build spec from docker image which is able to create genesis state
docker run --platform=linux/amd64 centrifugeio/centrifuge-chain:parachain-20220801173354-feffc7c build-spec --chain=development --disable-default-bootnode > demo-spec.json 
  1. Change hardcoded parachain id
      "parachainInfo": {
-        "parachainId": 10001
+        "parachainId": 2031
      },
  1. Copy spec into docker image
# Dockerfile
FROM centrifugeio/centrifuge-chain:parachain-20220801173354-feffc7c
COPY ./demo-spec.json /resources/demo-spec.json
  1. Generate raw spec
docker run --platform=linux/amd64 tmp-demo-genesis build-spec --chain=/resources/demo-spec.json --disable-default-bootnode --raw > demo-spec-raw.json 

Checklist:

  • I have added Rust doc comments to structs, enums, traits and functions
  • I have made corresponding changes to the documentation
  • I have performed a self-review of my code
  • I have added tests that prove my fix is effective or that my feature works

@wischli wischli added I3-annoyance The code behaves as expected, but "expected" is an issue. D0-ready Pull request can be merged without special precaution and notification. labels Oct 16, 2023
@wischli wischli self-assigned this Oct 16, 2023
Copy link
Contributor

@NunoAlexandre NunoAlexandre left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for all the context in the description, this is good built-in documentation 👌

@wischli wischli merged commit 71760fa into main Oct 17, 2023
10 checks passed
@NunoAlexandre NunoAlexandre deleted the feat/demo-spec branch October 17, 2023 12:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
D0-ready Pull request can be merged without special precaution and notification. I3-annoyance The code behaves as expected, but "expected" is an issue.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants