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

Support custom timeout_commit in testing mode #4080

Open
cmwaters opened this issue Dec 4, 2024 · 3 comments · May be fixed by #4103
Open

Support custom timeout_commit in testing mode #4080

cmwaters opened this issue Dec 4, 2024 · 3 comments · May be fixed by #4103
Assignees
Labels
WS: Maintenance 🔧 includes bugs, refactors, flakes, and tech debt etc

Comments

@cmwaters
Copy link
Contributor

cmwaters commented Dec 4, 2024

Summary

Since v3, timeout commits are now dictated by the application. This has made it difficult in the case of testing, where we and others want to be able to have short timeouts. Both sovereign and eiger have reported this being problematic for them.

Proposal

Introduce either an environment variable or a flag to force set the timeout commit. We may want to consider this to also set the square size as well. If we're concerned about misuse on a live network we can add chainID checks.

@cmwaters
Copy link
Contributor Author

cmwaters commented Dec 4, 2024

Furthermore, we might want to, if it's not too breaking, to rename the current timeout_commit and timeout_propose in the config.toml (or at least the go struct) to something else so as to avoid confusion. Most teams may still be setting these values and think it will change the times

@rootulp
Copy link
Collaborator

rootulp commented Dec 4, 2024

Introduce either an environment variable or a flag to force set the timeout commit. We may want to consider this to also set the square size as well.

We have the OVERRIDE_MAX_SQUARE_SIZE env variable which can override the max square size at binary build time so we could add another for overriding timeout_commit.

Alternatively, we could override the timeout_commit when the chain_id is test similar to how we override the upgrade height delay to 3 here.

Furthermore, we might want to, if it's not too breaking, to rename the current timeout_commit and timeout_propose in the config.toml (or at least the go struct) to something else so as to avoid confusion. Most teams may still be setting these values and think it will change the times

IMO we should deprecate the config timeout_commit and timeout_propose in v3.x.x and remove them in v4.x.x.

@evan-forbes evan-forbes added WS: Maintenance 🔧 includes bugs, refactors, flakes, and tech debt etc and removed needs:triage labels Dec 5, 2024
@rootulp
Copy link
Collaborator

rootulp commented Dec 6, 2024

Related: #4023

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
WS: Maintenance 🔧 includes bugs, refactors, flakes, and tech debt etc
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants