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
Robuster waiting for validators' readiness #36
Conversation
3dd7cc5
to
ff9ac51
Compare
ff9ac51
to
5694246
Compare
a4c0e12
to
55227b6
Compare
return 2 | ||
fi | ||
local container_name_substring=$1; shift | ||
local log_message=$1; shift |
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.
What's the benefit of shift
over $2
?
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.
Adding and removing parameters generates fewer mistakes
wait-for-bootstrap: | ||
image: docker | ||
commands: | ||
- until ci/is-container-log-line-present.sh _step_4 'Making a transition to ApprovedBlockRecievedHandler state.'; do echo $?; sleep 5; done |
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.
Does drone yaml version support constants? Making a transition to ApprovedBlockRecievedHandler state.
is a good candidate
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.
I don't believe it does
https://rchain.atlassian.net/browse/RCHAIN-1459