-
Notifications
You must be signed in to change notification settings - Fork 3.6k
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
Randomized Fuzzer: tried to delete a non-existent validator #2326
Comments
oh that one might have been introduced by staking transient store… pretty confused as to why though - I can’t reproduce locally with |
Its failing on most PR's: https://circleci.com/gh/cosmos/cosmos-sdk/30294?utm_campaign=vcs-integration-link&utm_medium=referral&utm_source=github-build-link, I guess we should make a command so its easy to reproduce the seed for these failing simulations. |
@ValarDragon do most of those PRs have |
I think so. It seems likely to me that transient store break this, as the failures began after that was merged, and are happening really early on. (On the weekend I was running a couple simulations that got to 1500 blocks, and many that failed at like blocks 100-200, never sub 10 blocks.) |
I'll think we should hold off on heavily investigating this error specifically until we have done the refactors as the update store will be removed at that point (however the transient store should still be getting used). Unless of course this is due to some new form of non-determinism IN the transient store (which is seems as though it may be? - so maybe actually still valuable to still investigate |
There isn't non-determinism. The reason for it only sometimes failing is because gaia_test_sim_modules is set to use a different seed each run. Maybe transient store isn't getting set again properly? |
So how should we proceed with this issue? |
still think this is low priority short term, as the solution may be thrown away - But if you're looking for something to do then it may reveal some valuable deeper level bug to do with the transient store. However - there is also a store refactor that joon is working on (which may affect this?) - so yeah probs not the best way to spend time |
I expect whatever this was has been fixed by #2394 - at minimum, the old code path is no longer relevant. Reopen if it can be replicated. |
Steps to Reproduce
I got the following from test_sim_modules on develop, in the staking simulation. Re-running a few times will probably recreate another failing seed.
For Admin Use
The text was updated successfully, but these errors were encountered: