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

Conway ledger state deserialize failure #3740

Closed
dnadales opened this issue Sep 19, 2023 · 0 comments · Fixed by #4072
Closed

Conway ledger state deserialize failure #3740

dnadales opened this issue Sep 19, 2023 · 0 comments · Fixed by #4072
Assignees
@aniketd
Labels
bug Something isn't working conway

Comments

@dnadales
Copy link
Member

When running the consensus CBOR roundtrip tests we get the following error. We're using the Cardano examples.

        Conway:        FAIL
          *** Failed! Falsified (after 1 test):
          DeserialiseFailure 839 "expected word"
          When deserialising 878283000000831b0018de76816d80001903e80a82831b0018de76816d80001903e80a831b001ff973cafa80001907d01482831b001ff973cafa80001907d014831b0027147114878000190bb8181e82831b0027147114878000190bb8181e831b002e2f6e5e148000190fa0182882831b002e2f6e5e148000190fa01828831b00354a6ba7a18000191388183282831b00354a6ba7a180001913881832831b003c6568f12e8000191770183c82831b003c6568f12e8000191770183c82028381830903582003170a2e7597b7b7e3d84c05391d139a62b157e78786d8c082f29dcf4c1113148700a1581ce0a714319812c3f773ba04ec5d6b3ffcd5aad85006805b047b0825410aa1581ca646474b8f5431261506b6c273d307c7569a4eb6c96b42dd4a29520a0384821927101903e8828383a081a0a084a0a0a0a08482a0a0a0a084a0a0000086a1825820ee155ace9c40292074cb6aff8c9ccdd273c81648ff1149ef36bcea6ebb8a3e2500a200583900cb9358529df4729c3246a2a033cb9821abbfd16de4888005904abc410d6a577e9441ad8ed9663931906e4d43ece8f82c712b1d0235affb0601821864a1581ce0a714319812c3f773ba04ec5d6b3ffcd5aad85006805b047b082541a14a636f75747473436f696e1903e81903e80182a083878082826058200000000000000000000000000000000000000000000000000000000000000000f6820900981e000000190800000000001864d81e820001d81e820001d81e8200018209000001a082d81e820001d81e8200018200008200000018960584d81e820001d81e820001d81e820001d81e8200018ad81e820001d81e820001d81e820001d81e820001d81e820001d81e820001d81e820001d81e820001d81e820001d81e820001000000000000981e000000190800000000001864d81e820001d81e820001d81e8200018209000000a082d81e820001d81e8200018200008200000018960584d81e820001d81e820001d81e820001d81e8200018ad81e820001d81e820001d81e820001d81e820001d81e820001d81e820001d81e820001d81e820001d81e820001d81e82000100000000000000a0808082a0a0008483a0a0a083a0a0a083a0a0a00082a000818300880082090082a000000000a0a0840185a0803903ba820900a0a082a0a0a1581ce0a714319812c3f773ba04ec5d6b3ffcd5aad85006805b047b08254182d81e8201015820c5e21ab1c9f6022d81c3b25e3436cb7f1df77f9652ae3e1310c28e621dd87b4cf600
          Use --quickcheck-replay=445680 to reproduce.
          Use -p '/Example/&&/Ledger state.Conway/' to rerun this test only.
@dnadales dnadales added the bug Something isn't working label Sep 19, 2023
@lehins lehins added the conway label Sep 19, 2023
@dnadales dnadales mentioned this issue Oct 31, 2023
Merged
github-merge-queue bot pushed a commit to IntersectMBO/ouroboros-consensus that referenced this issue Nov 13, 2023
Test that consensus emits valid CBOR (#479)
d490501 
Fixes IntersectMBO/ouroboros-network#3099

Sadly some CBOR validity tests need to be skipped due to:

- IntersectMBO/cardano-ledger#3800
- IntersectMBO/cardano-ledger#3741
- IntersectMBO/cardano-ledger#3740

This patch introduces an ad-hoc filtering mechanism to fix problems with
legacy encoders.
@aniketd aniketd mentioned this issue Feb 12, 2024
Merged
9 tasks
@aniketd aniketd mentioned this issue Feb 12, 2024
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@aniketd aniketd

Labels
bug Something isn't working conway
Projects
Conway
Status: Done
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fix ouroborus-consensus serialization roundtrip test IntersectMBO/cardano-ledger
3 participants
@dnadales @aniketd @lehins