Skip to content
This repository has been archived by the owner on Jan 19, 2024. It is now read-only.

Fixed bootstrap sequence #93

Merged
merged 8 commits into from
Jan 21, 2023
Merged

Fixed bootstrap sequence #93

merged 8 commits into from
Jan 21, 2023

Conversation

jolly-fellow
Copy link
Contributor

Fix of the bug #80

Copy link
Contributor

@ScottBailey ScottBailey left a comment

Choose a reason for hiding this comment

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

test/test_bootstrap.py requires update; otherwise this looks good to me.

@jolly-fellow
Copy link
Contributor Author

jolly-fellow commented Dec 30, 2022

test/test_bootstrap.py requires update; otherwise this looks good to me.

Updated

@jolly-fellow jolly-fellow linked an issue Dec 30, 2022 that may be closed by this pull request
Copy link
Contributor

@mikelik mikelik left a comment

Choose a reason for hiding this comment

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

I confirm following issue is fixed with this PR: wasm_serialization_error: Serialization Error Processing WASM env.set_wasm_parameters_packed unresolveable

I still see this issue when running test_boostrap.py:

failed transaction: e7c52027ae987c77daf60092c3b17709b9e862321080ede70266f134dcdd6e28  <unknown> bytes  <unknown> us
error 2022-12-30T11:38:13.301 cleos     main.cpp:700                  print_result         ] soft_except->to_detail_string(): 3100006 subjective_block_production_exception: Subjective exception thrown during block production
protocol feature with digest '825ee6288fb1373eab1b5187ec2f04f6eacb39cb3a97f356a07c91622dd61d16' is unrecognized
    {"digest":"825ee6288fb1373eab1b5187ec2f04f6eacb39cb3a97f356a07c91622dd61d16"}
    nodeos  controller.cpp:2703 preactivate_feature

src/dune/dune.py Show resolved Hide resolved
Copy link
Contributor

@ScottBailey ScottBailey left a comment

Choose a reason for hiding this comment

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

lgtm, though I'd prefer to see tests updated.

Copy link
Contributor

@ScottBailey ScottBailey left a comment

Choose a reason for hiding this comment

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

lgtm

@jolly-fellow jolly-fellow merged commit d437f07 into AntelopeIO:main Jan 21, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

Add a test for DUNE --get-table key to the functional tests --bootstrap-system-full doesn't work correctly.
4 participants