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

staking-miner: docker images #3682

Merged
13 commits merged into from
Sep 7, 2021

Conversation

chevdor
Copy link
Contributor

@chevdor chevdor commented Aug 20, 2021

Docker image + doc for the staking-miner. This PR adds 2 ways to create a docker image for the staking-miner:

  • injected: we first build a binary on a linux host and inject it into a base image
  • multi-stage: the staking-miner is fully built within the multi-stage image

@chevdor chevdor 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 Aug 25, 2021
@chevdor chevdor force-pushed the wk-staking-miner-docker-image branch from d1dc02a to 09dfda7 Compare August 25, 2021 10:41
@chevdor chevdor force-pushed the wk-staking-miner-docker-image branch from c8e91ed to af2e3f8 Compare August 27, 2021 12:43
@chevdor chevdor changed the title staking-miner: docker image staking-miner: docker images Aug 31, 2021
@chevdor chevdor added the A0-please_review Pull request needs code review. label Sep 3, 2021
Substrate chains validators compute a basic solution for the NPoS election. The optimization of the solution is computing-intensive and can be delegated to the `staking-miner`. The `staking-miner` does not act as validator and focuses solely on the optimization of the solution.

The staking miner connects to a specified chain and keeps listening to new Signed phase of the [pallet-election-provider-multi-phase](https://crates.parity.io/pallet_election_provider_multi_phase/index.html) in order to submit solutions to the NPoS election. When the correct time comes, it computes its solution and submit it to the chain.
The default miner algorithm is [sequential-phragmen](https://crates.parity.io/sp_npos_elections/phragmen/fn.seq_phragmen_core.html)] with a configurable number of balancing iterations that improve the score.
Copy link
Contributor

Choose a reason for hiding this comment

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

Soon to change in #3752. I think this will be merged sooner, so the other PR should update this.

Copy link
Contributor

@kianenigma kianenigma left a comment

Choose a reason for hiding this comment

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

Thanks!

@kianenigma kianenigma requested a review from emostov September 7, 2021 08:50
@chevdor
Copy link
Contributor Author

chevdor commented Sep 7, 2021

bot merge

@ghost
Copy link

ghost commented Sep 7, 2021

Trying merge.

@ghost ghost merged commit f9f4754 into paritytech:master Sep 7, 2021
@chevdor chevdor deleted the wk-staking-miner-docker-image branch September 7, 2021 08:53
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. 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.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants