-
Notifications
You must be signed in to change notification settings - Fork 10
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
assurances/3_007 panics on Fib(9) without checkpoint but the new service and bless changes are in the post-state #125
Comments
Addresses several issues reported by @jaymansfield: * #125 Incorporating new services upon panic - Assurances/3_006 * #123 Add D to chain specs - assurances/3_002 * #121 Fix state_snapshots lookup_meta value - assurances/2_007 Also: * #112 Adjust Chainspecs to be EC friendly as per advice from Dave Emmett * update README
Addresses several issues reported by @jaymansfield: * #125 Incorporating new services upon panic - Assurances/3_006 * #123 Add D to chain specs - assurances/3_002 * #121 Fix state_snapshots lookup_meta value - assurances/2_007 Also: * #112 Adjust Chainspecs to be EC friendly as per advice from Dave Emmett * update README
fixed -- no new services are created in assurances/3_006 (Fib(9)) |
@sourabhniyogi assurances/2_011 has a similar issue now with the collapse function. Service 2953942612 ends in panic but includes an assign host call and then a checkpoint host call after. The post-state does not have any record of the assign host call but the accumulate state should have been persisted to Yu at the checkpoint. Only changes after the checkpoint should be discarded. My phi value: All other assurances vectors pass for me with the latest release. |
Addresses: * #125 (part 2) - assurances/2_011 panics then checkpoints but should bring in assign result Added tiny readme explanation
Addresses: * #125 (part 2) - assurances/2_011 panics then checkpoints but should bring in assign result Added tiny readme explanation
solved here -- Thank you for a thorough check! |
@sourabhniyogi Thought we had it this time, but in the final vector (3_006) we have the issue again. Service 2953942612 ends in panic without a checkpoint, but the new service and bless changes were not reverted and are included in post state. UPDATE: After the vectors were re-generated the issue is now in 3_007 (and not 3_006). |
Ok, unless we have another issue from 0.6.2.9 (from another team), we will get this perfected in 0.6.3.0, probably need 3-4 days to get there. |
I can confirm we (boka) also encountered this issue in state transition 2011 and 3007 |
Addresses: * #125 - panics without checkpoint should not include new services and bless
Addresses: * #125 - panics without checkpoint should not include new services and bless
Addresses: * #125 - panics without checkpoint should not include new services and bless
Updated dataset and tagged 0.6.2.10. Notes:
|
Looks good! All data sets pass for JavaJAM now. |
So in Assurances/3_006 there are two services that are accumulated. Both end up panic according to assurances.txt.
When a panic occurs, the collapse function discards changes by reverting to Yu so I don't think the new service 2227924372 should be in post state.
The text was updated successfully, but these errors were encountered: