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.

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

Closed
jaymansfield opened this issue Mar 4, 2025 · 8 comments
Assignees

Comments

@jaymansfield
Copy link

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.

Image

@sourabhniyogi sourabhniyogi self-assigned this Mar 4, 2025
@sourabhniyogi sourabhniyogi changed the title Assurances/3_006 - Post state Incorporating new services upon panic - Assurances/3_006 Mar 4, 2025
sourabhniyogi added a commit that referenced this issue Mar 4, 2025
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
sourabhniyogi added a commit that referenced this issue Mar 5, 2025
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
sourabhniyogi added a commit that referenced this issue Mar 5, 2025
Major:
* #125 Y/Checkpoint treatment (panic of Assurances/3_006) -- see new fib2

Minor:
* #123 forget case - Add D to chain specs - assurances/3_002
* #121 minor fix state_snapshots lookup_meta value - assurances/2_007
sourabhniyogi added a commit that referenced this issue Mar 5, 2025
Major:
* #125 Y/Checkpoint treatment (panic of Assurances/3_006) -- see new fib2

Minor:
* #123 forget case - Add D to chain specs - assurances/3_002
* #121 minor fix state_snapshots lookup_meta value - assurances/2_007
@sourabhniyogi
Copy link
Contributor

fixed -- no new services are created in assurances/3_006 (Fib(9))

@jaymansfield
Copy link
Author

jaymansfield commented Mar 5, 2025

@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:
phi=[AuthQueue{authQueue=[0x6a0d4a19d199505713fc65f531038e73f1d885645632c8ae503c4f0c4d5e19a7, 0x6a0d4a19d199505713fc65f531038e73f1d885645632c8ae503c4f0c4d5e19a7, 0x6a0d4a19d199505713fc65f531038e73f1d885645632c8ae503c4f0c4d5e19a7, 0x6a0d4a19d199505713fc65f531038e73f1d885645632c8ae503c4f0c4d5e19a7, .......

All other assurances vectors pass for me with the latest release.

@sourabhniyogi sourabhniyogi reopened this Mar 5, 2025
@sourabhniyogi sourabhniyogi changed the title Incorporating new services upon panic - Assurances/3_006 assurances/2_011 panics then checkpoints but should bring in assign result Mar 5, 2025
sourabhniyogi added a commit that referenced this issue Mar 5, 2025
Addresses:
* #125 (part 2) - assurances/2_011 panics then checkpoints but should bring in assign result

Added tiny readme explanation
sourabhniyogi added a commit that referenced this issue Mar 5, 2025
Addresses:
* #125 (part 2) - assurances/2_011 panics then checkpoints but should bring in assign result

Added tiny readme explanation
@sourabhniyogi
Copy link
Contributor

solved here -- Thank you for a thorough check!

@jaymansfield
Copy link
Author

jaymansfield commented Mar 6, 2025

@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).

@sourabhniyogi sourabhniyogi reopened this Mar 6, 2025
@sourabhniyogi sourabhniyogi changed the title assurances/2_011 panics then checkpoints but should bring in assign result assurances/3_006 panics then checkpoints but should NOT bring in assign result Mar 6, 2025
@sourabhniyogi
Copy link
Contributor

sourabhniyogi commented Mar 6, 2025

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.

@sourabhniyogi sourabhniyogi changed the title assurances/3_006 panics then checkpoints but should NOT bring in assign result assurances/3_006 panics on Fib(9) without checkpoint but the new service and bless changes are in the post-state Mar 6, 2025
@qiweiii
Copy link

qiweiii commented Mar 7, 2025

I can confirm we (boka) also encountered this issue in state transition 2011 and 3007

@jaymansfield jaymansfield changed the title assurances/3_006 panics on Fib(9) without checkpoint but the new service and bless changes are in the post-state assurances/3_007 panics on Fib(9) without checkpoint but the new service and bless changes are in the post-state Mar 7, 2025
sourabhniyogi added a commit that referenced this issue Mar 7, 2025
Addresses:
* #125 - panics without checkpoint should not include new services and bless
sourabhniyogi added a commit that referenced this issue Mar 7, 2025
Addresses:
* #125 - panics without checkpoint should not include new services and bless
sourabhniyogi added a commit that referenced this issue Mar 7, 2025
Addresses:
* #125 - panics without checkpoint should not include new services and bless
@sourabhniyogi
Copy link
Contributor

Updated dataset and tagged 0.6.2.10.

Notes:

  1. @jaymansfield @qiweiii This fixes assurances/3_007 in that
    (a) the NEW service (2227924372) is NOT in the post-state and
    (b) the BLESS host result (m=0 a=1 v=1 g=map[0:911911]) is NOT in the post-state

  2. @qiweiii Even though both assurances/2_011 (Fib(6)) and assurances/3_007 (Fib(9)) have a panic, in assurances/2_011 there is a CHECKPOINT, so almost all the state (everything except this) makes it through. In contrast, 3_007 does not have a checkpoint, so nothing makes it through.

@jaymansfield
Copy link
Author

jaymansfield commented Mar 8, 2025

Looks good! All data sets pass for JavaJAM now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants