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.

Sign up for GitHub

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

Jump to bottom

Update HW requirements, node start commands, and node shell scripts #3351

Open
wants to merge 20 commits into
base: mainnet-staging
Choose a base branch
from
Open

Update HW requirements, node start commands, and node shell scripts #3351

wants to merge 20 commits into from

Conversation

kpandl
Copy link
Contributor

@kpandl kpandl commented Jul 5, 2024 •
edited
Loading

This PR changes the README.md file and the validator/client/prover shell scripts.

Rendered Version

For the README.md, the PR specifically:

  • provides updated HW requirements for validators/clients
  • removes official HW recommendations for provers
  • introduces the concept of core and outer clients
  • updates the recommended start commands of validators/clients

For the corresponding shell scripts, it:

  • makes use of the updated start commands
  • provides a shell script for core/outer clients instead of general clients
  • removes the automatic GitHub update loop (which could trigger a large number of nodes to go offline around the same time)

kpandl and others added 19 commits May 25, 2024 00:37
@kpandl
hw specs and script start command
7192fef
@kpandl @iamalwaysuncomfortable
Update README.md
80f722b 
Co-authored-by: Mike Turner <[email protected]>
Signed-off-by: Konstantin Pandl <[email protected]>
@kpandl @iamalwaysuncomfortable
Update README.md
2b0a2fb 
Co-authored-by: Mike Turner <[email protected]>
Signed-off-by: Konstantin Pandl <[email protected]>
@kpandl @iamalwaysuncomfortable
Update README.md
a225af1 
Co-authored-by: Mike Turner <[email protected]>
Signed-off-by: Konstantin Pandl <[email protected]>
@kpandl @iamalwaysuncomfortable
Update README.md
e0ec38a 
Co-authored-by: Mike Turner <[email protected]>
Signed-off-by: Konstantin Pandl <[email protected]>
@kpandl @iamalwaysuncomfortable
Update README.md
4397f14 
Co-authored-by: Mike Turner <[email protected]>
Signed-off-by: Konstantin Pandl <[email protected]>
@kpandl @iamalwaysuncomfortable
Update README.md
a615c7c 
Co-authored-by: Mike Turner <[email protected]>
Signed-off-by: Konstantin Pandl <[email protected]>
@kpandl @iamalwaysuncomfortable
Update README.md
49c4d74 
Co-authored-by: Mike Turner <[email protected]>
Signed-off-by: Konstantin Pandl <[email protected]>
@kpandl @iamalwaysuncomfortable
Apply suggestions from code review
e22a75d 
Co-authored-by: Mike Turner <[email protected]>
Signed-off-by: Konstantin Pandl <[email protected]>
@kpandl
fix typo and core client script
723b541
@kpandl
update validator machine checks
1053924
@kpandl
GiB instead of GB for RAM
acc9405
@kpandl
update min RAM in GiB
a563f1e
@kpandl
update README formatting
7ea2222
@kpandl
fix typo
a72232b
@kpandl
update README - add or larger preferred
197cfb1
@kpandl
validators as peers
988f8ec
@kpandl
double quotes
4a1b4e6
@kpandl
Merge branch 'mainnet-staging' into mainnet-staging
b183b26
@kpandl kpandl marked this pull request as ready for review July 5, 2024 05:28
@kpandl kpandl mentioned this pull request Jul 5, 2024
Closed
@vicsn vicsn mentioned this pull request Jul 9, 2024
Open
raychu86
raychu86 reviewed Jul 15, 2024
View reviewed changes
run-outer-client.sh Outdated Show resolved Hide resolved
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@raychu86 raychu86 raychu86 approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@kpandl @raychu86