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

Conversation

@mxinden
Copy link
Contributor

@mxinden mxinden commented Jan 20, 2021

Sentry nodes are deprecated. Thus there is no need for
.maintain/sentry-node to spin up a sentry node test environment.
Instead this commit rewrites the setup with an additional light client.

With the steps below one can now spin up a local test network with
two validators, one light-client, Prometheus and Grafana.

  • cargo build --release
  • sudo docker-compose -f .maintain/local-docker-test-network/docker-compose.yml up

I did initially do this for #7895 to test light client requests. I hope this can be of some help for other use-cases as well, thus I am opening this as a separate pull request.

Sentry nodes are deprecated. Thus there is no need for
`.maintain/sentry-node` to spin up a sentry node test environment.
Instead this commit rewrites the setup to contain two full-connected
validators and one light client.

With the steps below one can now spin up a local test network with
two validators, one light-client, Prometheus and Grafana.

- cargo build --release
- sudo docker-compose -f .maintain/local-docker-test-network/docker-compose.yml up
@mxinden mxinden added A0-please_review Pull request needs code review. 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 Jan 20, 2021
@mxinden mxinden requested a review from tomaka January 20, 2021 19:58
Copy link
Contributor

@tomaka tomaka left a comment

Choose a reason for hiding this comment

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

I haven't tested if it works, but looks generally good to me.

@mxinden
Copy link
Contributor Author

mxinden commented Jan 21, 2021

bot merge

@ghost
Copy link

ghost commented Jan 21, 2021

Trying merge.

@ghost ghost merged commit 1baeefa into paritytech:master Jan 21, 2021
This pull request was closed.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

A0-please_review Pull request needs code review. 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.

3 participants