Skip to content

Conversation

@Davidson-Souza
Copy link
Member

@Davidson-Souza Davidson-Souza commented Jun 23, 2025

What is the purpose of this pull request?

  • Bug fix
  • Documentation update
  • New feature
  • Test
  • Other:

Which crates are being modified?

  • floresta-chain
  • floresta-cli
  • floresta-common
  • floresta-compact-filters
  • floresta-electrum
  • floresta-watch-only
  • floresta-wire
  • floresta
  • florestad
  • Other:

Description

With this commit, you can now use tokio-console with florestad.

Edit: Some screenshots of tokio-console running on my machine:

The main screen shows all the tasks that were spawned, how many time it woken up, how long did it run, wait or blocked.
Screenshot_2025-06-23_20-35-03

This is the resource tab, it shows all resources that tokio is menaging (locks, files, sockets, channels...). You can toggle this view by pressing r, and go back to tasks by pressing t.
Screenshot_2025-06-23_20-35-25

This shows some info about one specific task, like a histogram of CPU usage (you can toggle this by pressing ENTER while on the main screen)
Screenshot_2025-06-23_20-35-44

Notes to the reviewers

Install tokio-console with cargo and run tokio-console with floresta running to use it. Then start floresta with RUSTFLAGS=--cfg tokio_unstable cargo run --release --bin florestad --features tokio-console

With this commit, you can now use [tokio-console](https://github.com/tokio-rs/console)
with florestad. Just need to build with thte `tokio-console` feature and
pass `RUSTFLAGS="--cfg tokio_unstable"` to cargo
@Davidson-Souza Davidson-Souza added enhancement New feature or request dependencies Pull requests that update a dependency file labels Jun 23, 2025
@JoseSK999
Copy link
Contributor

Install tokio-console with cargo and run tokio-console with floresta running to use it. Then start floresta with RUSTFLAGS=--cfg tokio_unstable cargo run --release --bin florestad --features tokio-console

A doc explaining this step-by-step (even showing the screenshots) would be needed IMO

@jaoleal
Copy link
Collaborator

jaoleal commented Jun 24, 2025

Nice!!

Copy link
Collaborator

@jaoleal jaoleal left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ACK aa78d48

Thats actually a strong feature, i might package this on nix later...

@Davidson-Souza Davidson-Souza merged commit 1f12f3f into vinteumorg:master Jun 26, 2025
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies Pull requests that update a dependency file enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants