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

bios boot tutorial sleep was too short on linux #210

Closed
wants to merge 1 commit into from
Closed

bios boot tutorial sleep was too short on linux #210

wants to merge 1 commit into from

Conversation

ClaytonCalabrese
Copy link
Contributor

port of EOSIO/eos#9738

The bios-boot-tutorial.py needs to give nodeos additional time starting up before it continues on with execution. If the script moves too fast, nodeos may not be at a point it is accepting p2p transactions and the follow on steps in the script will fail. 10 seconds seems long enough to allow the script to consistently succeed.

issue #207

@ClaytonCalabrese ClaytonCalabrese self-assigned this May 10, 2022
@oschwaldp-oci oschwaldp-oci self-requested a review May 10, 2022 18:06
oschwaldp-oci
oschwaldp-oci reviewed May 10, 2022
View reviewed changes
Copy link
Contributor

@oschwaldp-oci oschwaldp-oci left a comment

Choose a reason for hiding this comment

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

Port of change looks good. Holding off on your analysis of the tutorial itself.

@ClaytonCalabrese
Copy link
Contributor Author

There are other problems in the tutorial. Futher investigation is needed.

@heifner heifner added the OCI OCI working this issue... label May 12, 2022
@ClaytonCalabrese ClaytonCalabrese deleted the cc_enfs119_bios_boot_tutorial_sleep branch May 20, 2022 04:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@oschwaldp-oci oschwaldp-oci oschwaldp-oci approved these changes

Assignees

@ClaytonCalabrese ClaytonCalabrese

Labels
OCI OCI working this issue...
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@ClaytonCalabrese @oschwaldp-oci @heifner