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

New ABCI method to surface to the application that there was a consensus error #2051

Open
Tracked by #9
adizere opened this issue Jan 16, 2024 · 1 comment
Open
Tracked by #9

Comments

@adizere
Copy link
Member

adizere commented Jan 16, 2024

The context for why we forked to include this right now, was because there have been node stability problems for months with folks getting random app hashes due to some issue in a query somewhere, and we've been massively struggling to get proper info to debug them. (It affects several cosmos chains, not just osmosis)

So we wanted to add some basic logging app side (block results, module app hashes, we'd eventually do logging the gas steps in a tx, etc.) so we can start finding the source of this issue.

Definitely don't see us being on a long term fork for this, but we did need some more debug info on the default binary we give nodes for now! We'll likely just keep rebasing onto the latest v0.37.x release line until we can get the app hash situation sorted at least

Originally posted by @ValarDragon in #1978 (comment)

@adizere
Copy link
Member Author

adizere commented Jan 16, 2024

See #1978

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

1 participant