-
Notifications
You must be signed in to change notification settings - Fork 264
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
Consider increasing TimeoutBroadcastTxCommit
#1273
Comments
The 10 second timeout looks already overwritten in celestia-app here. Related: celestiaorg/celestia-app#1034 In a node I have locally, the # How long to wait for a tx to be committed during /broadcast_tx_commit.
# WARNING: Using a value larger than 10s will result in increasing the
# global HTTP write timeout, which applies to all connections and endpoints.
# See https://github.com/tendermint/tendermint/issues/3435
timeout_broadcast_tx_commit = "50s" which makes me think the 10 second timeout alluded to in the OP is not actually the timeout getting hit. There are other timeouts in the config files. For example # RPCReadTimeout defines the Tendermint RPC read timeout (in seconds).
rpc-read-timeout = 10 |
@joroshiba do you know which consensus full node you're connected to? It would be nice if we can verify what value they use for |
…lestiaorg#1273) Bumps [bufbuild/buf-setup-action](https://github.com/bufbuild/buf-setup-action) from 1.26.0 to 1.26.1. - [Release notes](https://github.com/bufbuild/buf-setup-action/releases) - [Commits](bufbuild/buf-setup-action@v1.26.0...v1.26.1) --- updated-dependencies: - dependency-name: bufbuild/buf-setup-action dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Context
celestia-core/config/config.go
Line 439 in 5cfce0b
Problem
Feedback from @joroshiba
Proposal
The text was updated successfully, but these errors were encountered: