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

Relayer logs should be more meaningful #1768

Closed
nambrot opened this issue Feb 6, 2023 · 2 comments · Fixed by #1797
Closed

Relayer logs should be more meaningful #1768

nambrot opened this issue Feb 6, 2023 · 2 comments · Fixed by #1797
Assignees
Labels

Comments

@nambrot
Copy link
Contributor

nambrot commented Feb 6, 2023

The general goal here is to make the relayer logs more useful

That includes, but is not exclusive to:

  • Default verbosity should be INFO (might already be)
  • Should the "indexed block range" log lines to reduced in verbosity?
  • some lower level http logs like incoming body completed� or flushed 806 bytes should be moved to TRACE?
  • serialize some objects more concisely (HyperlaneMessage, MetadataBuilder
  • remove some traces like this one or this one
  • when there is a JsonRPCError, probably don't log it in the inner retries at warn if possible, but when the max number was hit, then do log the errors from at least the last retry
  • have dedicated info or warn messages (or maybe debug) when a message could not be estimated, no metadata could be fetched
  • Log more data about ISM metadata https://discord.com/channels/935678348330434570/1072862711681142874/1072868391045578812
@nambrot nambrot converted this from a draft issue Feb 6, 2023
@nambrot nambrot added the agent label Feb 6, 2023
@nambrot nambrot moved this from Backlog to Sprint in Hyperlane Tasks Feb 6, 2023
@nambrot
Copy link
Contributor Author

nambrot commented Feb 6, 2023

another good source for real world relayer logs that seem very hard to read https://discord.com/channels/935678348330434570/1063471868742467644/1072155638135795772

@distributedstatemachine
Copy link

another good source for real world relayer logs that seem very hard to read https://discord.com/channels/935678348330434570/1063471868742467644/1072155638135795772

I think these logs are harder than usual to read as i had the pretty var enabled. They are easier to parse without it.

@mattiekat mattiekat moved this from Sprint to In Progress in Hyperlane Tasks Feb 7, 2023
@mattiekat mattiekat moved this from In Progress to In Review in Hyperlane Tasks Feb 9, 2023
@github-project-automation github-project-automation bot moved this from In Review to Done in Hyperlane Tasks Feb 12, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

3 participants