Skip to content

improve error log when attempting to relay redundant packets #1135

@jtieri

Description

@jtieri

The error logged when the relayer attempts to relay packets already handled by another relayer is very verbose and for an inexperienced operator this can look like something that requires intervention when in reality it is a normal situation to observe.
We should clean this up and make it clear that there is nothing to be done.

ex:

2023-03-22T16:43:02.481620Z     error   Failed sending cosmos transaction       {"provider_type": "cosmos", "chain_id": "osmosis", "msg_types": ["/ibc.core.client.v1.MsgUpdateClient", "/ibc.core.channel.v1.MsgAcknowledgement"], "error": "packet messages are redundant", "errorVerbose": "packet messages are redundant\ncosmossdk.io/errors.Wrap\n\t/home/laurens/go/pkg/mod/cosmossdk.io/errors@v1.0.0-beta.7/errors.go:187\ncosmossdk.io/errors.ABCIError\n\t/home/laurens/go/pkg/mod/cosmossdk.io/errors@v1.0.0-beta.7/errors.go:75\ngithub.com/cosmos/relayer/v2/relayer/chains/cosmos.(*CosmosProvider).sdkError\n\t/home/laurens/quasar-finance/relayer/relayer/chains/cosmos/tx.go:170\ngithub.com/cosmos/relayer/v2/relayer/chains/cosmos.(*CosmosProvider).broadcastTx\n\t/home/laurens/quasar-finance/relayer/relayer/chains/cosmos/tx.go:206\ngithub.com/cosmos/relayer/v2/relayer/chains/cosmos.(*CosmosProvider).SendMessagesToMempool\n\t/home/laurens/quasar-finance/relayer/relayer/chains/cosmos/tx.go:152\ngithub.com/cosmos/relayer/v2/relayer/processor.(*messageProcessor).sendSingleMessage\n\t/home/laurens/quasar-finance/relayer/relayer/processor/message_processor.go:439\nruntime.goexit\n\t/home/laurens/.go/src/runtime/asm_amd64.s:1594"}

Metadata

Metadata

Assignees

Labels

Type

No type

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions