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

add two node local net script #1781

Merged
4 commits merged into from
Oct 5, 2020
Merged

add two node local net script #1781

4 commits merged into from
Oct 5, 2020

Conversation

coriolinus
Copy link
Contributor

It's non-trivial to run a multi-node local net. This PR adds a script which makes it a pushbutton operation.

@coriolinus coriolinus self-assigned this Oct 2, 2020
@github-actions github-actions bot added the A3-in_progress Pull request is in progress. No review needed at this stage. label Oct 2, 2020
@coriolinus coriolinus changed the title initial work running a two node local net add two node local net script Oct 5, 2020
@coriolinus coriolinus added A2-insubstantial Pull request requires no code review (e.g., a sub-repository hash update). 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. labels Oct 5, 2020
@coriolinus coriolinus marked this pull request as ready for review October 5, 2020 12:56
Copy link
Member

@ordian ordian left a comment

Choose a reason for hiding this comment

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

There is also docker-compose, but I guess the difference is that this one builds on HEAD instead of the latest released version?

I'm not well versed in bash enough in order tell you if there are better ways of doing this (maybe @s3krit is?), but it seems to work locally, which is what matters the most.

@coriolinus
Copy link
Contributor Author

bot merge

@ghost
Copy link

ghost commented Oct 5, 2020

Waiting for commit status.

@ghost ghost merged commit 935ed8d into master Oct 5, 2020
@ghost ghost deleted the prgn-run-local-polkadot-script branch October 5, 2020 14:51
@bkchr
Copy link
Member

bkchr commented Oct 5, 2020

We already have something more advanced: https://github.com/shawntabrizi/polkadot-launch

@coriolinus
Copy link
Contributor Author

That's interesting, and definitely more advanced, but in this case I think simplicity is a virtue. The point here is to be able to take any given state of the codebase and quickly and accurately answer the question: "Can it still produce blocks?"

ordian pushed a commit that referenced this pull request Oct 6, 2020

Verified

This commit was signed with the committer’s verified signature.
* master:
  Make collation an optional return (#1787)
  XCM: Land xcm-handler and xcm-executor (#1771)
  v0.8.25 (#1785)
  add two node local net script (#1781)
  Adjust max nominators down to 128 (from 256) (#1782)
  Companion for substrate/pull/7215 (#1768)
  Remove Stale Upgrades (#1780)
  Update Polkadot Weights for Substrate 2.0 (#1761)
  Parachains v1 registrar module. (#1559)
  Derive `From` for `AllMessages` and simplify `send_msg` (#1774)
  implement remaining subsystem metrics (#1770)
  Companion for paritytech/substrate#7236 (#1773)
  WIP: remove deprecated only/except clauses, build is now manual on PRs (#1769)
  Increase Westend `spec_version` (#1766)
  move Metrics to utils (#1765)
ordian pushed a commit that referenced this pull request Oct 6, 2020

Verified

This commit was signed with the committer’s verified signature.
* master:
  NetworkBridge: validator (authorities) discovery api (#1699)
  Registrar v1 follow-ups (#1786)
  Make collation an optional return (#1787)
  XCM: Land xcm-handler and xcm-executor (#1771)
  v0.8.25 (#1785)
  add two node local net script (#1781)
  Adjust max nominators down to 128 (from 256) (#1782)
  Companion for substrate/pull/7215 (#1768)
  Remove Stale Upgrades (#1780)
  Update Polkadot Weights for Substrate 2.0 (#1761)
This pull request was closed.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
A2-insubstantial Pull request requires no code review (e.g., a sub-repository hash update). A3-in_progress Pull request is in progress. No review needed at this stage. 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.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants