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

fix proofHash related sporadic concurrency bug #2109

Merged
merged 1 commit into from
Feb 5, 2024

Conversation

stenolog
Copy link
Contributor

@stenolog stenolog commented Feb 3, 2024

Testing

This needs a long-running stress-test outside of the CI.

See branch:

https://github.com/stenolog/ergo/tree/i1387-i1023-repro-and-fix

for the fix incl. the reproducer and documentation.

Quick Examples

# with fix
cd ~
mkdir proofhash
cd proofhash
git clone -b i1387-i1023-repro-and-fix https://github.com/stenolog/ergo.git
cd ergo
sbt compile && time while [ $? -eq 0 ]; do sbt "testOnly *UtxoStateSpecification -- -z looper"; done

in another window

# without fix, should fail in around 10 min.
cd ~/proofhash
git clone -b i1387-i1023-repro-and-fix https://github.com/stenolog/ergo.git ergofail
cd ergofail
git checkout HEAD~1 # disable fix (last commit)
sbt compile && time while [ $? -eq 0 ]; do sbt "testOnly *UtxoStateSpecification -- -z looper"; done

@kushti kushti changed the base branch from master to v5.0.20 February 4, 2024 21:24
@kushti
Copy link
Member

kushti commented Feb 5, 2024

awesome work!

Please send me your reward address to [email protected] or PM over Discord to get attached bounty paid!

@kushti kushti merged commit 7b37d84 into ergoplatform:v5.0.20 Feb 5, 2024
4 checks passed
@stenolog
Copy link
Contributor Author

stenolog commented Feb 5, 2024

awesome work!

Thank you. Email send!

@stenolog stenolog mentioned this pull request Feb 15, 2024
3 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants