Skip to content

Start a client#3148

Merged
nkuba merged 12 commits intomainfrom
start-client
Aug 19, 2022
Merged

Start a client#3148
nkuba merged 12 commits intomainfrom
start-client

Conversation

@dimpar
Copy link
Copy Markdown
Contributor

@dimpar dimpar commented Aug 9, 2022

This PR uses changes made in #3129 to use the locally deployed artifacts. Now install.sh will build the client with the local artifacts. Also, start.sh can be used now to start a client.

To start a client we need to:

Manually configure config.toml in the /configs dir. Providing contract addresses is no longer required for "development" env. Next:

  • run scripts/install.sh
  • run scripts/initialize.sh --stake-owner <address>
  • run scripts/start.sh

Available params for start.sh

  • KEEP_ETHEREUM_PASSWORD env var. If not provided, "password" is used
  • flag: --config-dir path to the config file. /configs is default

Refs #3135

dimpar added 3 commits August 9, 2022 11:22
It is possible now to build a client with locally avaialble artifacts
that were deployed on Geth. No need to provide contracts addresses that
were required by the client for the local start.
@dimpar dimpar closed this Aug 9, 2022
@dimpar dimpar reopened this Aug 9, 2022
- Setting KEEP_ETHEREUM_PASSWORD as env. var. Default password is
  password
- Changing ./keep-core -> ./keep-client
@dimpar dimpar requested review from nkuba and pdyraga August 11, 2022 13:32
@dimpar dimpar self-assigned this Aug 11, 2022
@dimpar dimpar mentioned this pull request Aug 11, 2022
1 task
@dimpar dimpar changed the title Start client Start a client Aug 13, 2022
Comment thread scripts/install.sh Outdated
Comment thread scripts/install.sh Outdated
Comment thread scripts/install.sh Outdated
Comment thread scripts/install.sh Outdated
Comment thread scripts/install.sh Outdated
Comment thread scripts/install.sh Outdated
Comment thread scripts/install.sh Outdated
Comment thread scripts/install.sh Outdated
Comment thread scripts/install.sh Outdated
Comment thread scripts/start.sh Outdated
dimpar added 2 commits August 18, 2022 13:41
It is not necessary adding a LOCAL-* prefix, we can simplify namings.
@dimpar
Copy link
Copy Markdown
Contributor Author

dimpar commented Aug 18, 2022

@nkuba A lot of the above comments were related to namings. I've changed everything in one commit 65dc748 so it was easier to test everything during the run.

Comment thread scripts/install.sh Outdated
Copy link
Copy Markdown
Member

@nkuba nkuba left a comment

Choose a reason for hiding this comment

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

Non-blocking comment, it's just my feeling.

Comment thread scripts/start.sh Outdated
@nkuba nkuba merged commit a7520cf into main Aug 19, 2022
@nkuba nkuba deleted the start-client branch August 19, 2022 05:41
@pdyraga pdyraga added this to the v2.0.0-m1 milestone Sep 30, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants