-
Notifications
You must be signed in to change notification settings - Fork 836
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.
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
imp(tests): improvements to local node script #1805
Conversation
ENG-2034 Add flags to local node script
ContextThe ExpectedFor copy-pasteable instructions it would be great to be able to pass flags like |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice work @MalteHerrmann this would prove very useful!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
tACK! Great work @MalteHerrmann!!
Left one small fix to address
@Mergifyio backport release/v14.2.x |
✅ Backports have been created
|
* use preconfigured keys * add flags to local node script * address linter * Update local_node.sh --------- Co-authored-by: Vladislav Varadinov <vladislav.varadinov@gmail.com> Co-authored-by: Tom <54514587+GAtom22@users.noreply.github.com> (cherry picked from commit cffda8f)
imp(tests): improvements to local node script (#1805) * use preconfigured keys * add flags to local node script * address linter * Update local_node.sh --------- Co-authored-by: Vladislav Varadinov <vladislav.varadinov@gmail.com> Co-authored-by: Tom <54514587+GAtom22@users.noreply.github.com> (cherry picked from commit cffda8f) Co-authored-by: MalteHerrmann <42640438+MalteHerrmann@users.noreply.github.com>
Description
This PR adds the following improvements to the
local_node.sh
script:-y
: overwrite existing chain data and reset everything-n
: do not overwrite existing chain data, continue from last block--no-install
: skip installation of the binary (useful if correct version is already installed)Closes ENG-2034