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

Cleanup output of relayer commands #536

Closed
5 tasks
ancazamfir opened this issue Jan 19, 2021 · 2 comments · Fixed by #541
Closed
5 tasks

Cleanup output of relayer commands #536

ancazamfir opened this issue Jan 19, 2021 · 2 comments · Fixed by #541
Assignees
Labels
I: CLI Internal: related to the relayer's CLI O: usability Objective: cause to improve the user experience (UX) and ease using the product
Milestone

Comments

@ancazamfir
Copy link
Collaborator

Crate

relayer
relayer-cli

Summary of Bug

All debugs are currently info! or same level, making the output very chatty.

Version

master

Steps to Reproduce

Run the CLIs


For Admin Use

  • Not duplicate issue
  • Appropriate labels applied
  • Appropriate milestone (priority) applied
  • Appropriate contributors tagged
  • Contributor assigned/self-assigned
@ancazamfir ancazamfir added this to the v0.1.0 milestone Jan 19, 2021
@ancazamfir
Copy link
Collaborator Author

@vitorenesduarte mentioned:

It’s possible to filter logs by application. For example, RUST_LOG=hyper=info would only should logs for hyper.
(Nevertheless, the first line “Message CreateClient for…” in the log above doesn’t seem to come from tracing, so this probably won’t help)
...
it’s also possible to say something like RUST_LOG=app1=info,app2=debug

@andynog
Copy link
Contributor

andynog commented Jan 19, 2021

I thought that the RUST_LOG would work but apparently there are some messages (see below) that use status_info! macro from abscissa and that seems that it doesn't use log or tracing. So not sure there's a way to control, seems a custom macro.

https://github.com/informalsystems/ibc-rs/blob/1a77c0af20dee97d45096c4bcafe189c6a4f7689/relayer-cli/src/commands/tx/client.rs#L83-L89

@adizere adizere mentioned this issue Jan 20, 2021
5 tasks
@adizere adizere self-assigned this Jan 20, 2021
@adizere adizere added I: CLI Internal: related to the relayer's CLI O: usability Objective: cause to improve the user experience (UX) and ease using the product labels Jan 20, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
I: CLI Internal: related to the relayer's CLI O: usability Objective: cause to improve the user experience (UX) and ease using the product
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants