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

iroh-blobs logs are being swalloed #2577

Closed
dignifiedquire opened this issue Aug 2, 2024 · 1 comment · Fixed by #2735
Closed

iroh-blobs logs are being swalloed #2577

dignifiedquire opened this issue Aug 2, 2024 · 1 comment · Fixed by #2735
Assignees
Labels
bug Something isn't working c-iroh-blobs
Milestone

Comments

@dignifiedquire
Copy link
Contributor

there is no logging output anymore even on TRACE level from iroh-blobs

@dignifiedquire dignifiedquire added bug Something isn't working c-iroh-blobs labels Aug 2, 2024
@dignifiedquire dignifiedquire added this to the v0.22.0 milestone Aug 2, 2024
@flub
Copy link
Contributor

flub commented Aug 2, 2024

RUST_LOG=iroh_blobs=trace cargo test -p iroh --lib -- test_blob_get_self --show-output produces no logs on fix-download-self branch

@dignifiedquire dignifiedquire modified the milestones: v0.22.0, v0.23.0 Aug 5, 2024
@dignifiedquire dignifiedquire removed this from the v0.23.0 milestone Aug 14, 2024
@flub flub added this to the v0.24.0 milestone Aug 26, 2024
@ramfox ramfox modified the milestones: v0.24.0, v0.25.0 Aug 30, 2024
@dignifiedquire dignifiedquire modified the milestones: v0.25.0, v0.26.0 Sep 16, 2024
github-merge-queue bot pushed a commit that referenced this issue Sep 17, 2024
## Description

This preserves the tracing subscriber that is installed in the current
thread when the LocalPool is created.  It then installs it into every
thread managed by the pool, ensuring that tracing output from the pool
is preserved.

## Breaking Changes

Why is this even pub?  But fine:

`iroh_blobs::util::local_pool::LocalPool` will now install the tracing
subscriber of the thread creating the pool, into each thread managed
by the pool.  Practically this should not break any code already
managing their tracing subscribers either manually inside tasks or by
setting a global subscriber before creating the pool.  But if you
really liked the behaviour of swallowing the logs on doing this it's a
breaking change.

## Notes & open questions

Maybe this should be configurable, though since this is just an
internal tool for us I think we're fine just always having this
behaviour.

Fixes #2577 
Replaces #2589

## Change checklist

- [x] Self-review.
- [x] Documentation updates following the [style
guide](https://rust-lang.github.io/rfcs/1574-more-api-documentation-conventions.html#appendix-a-full-conventions-text),
if relevant.
- [x] Tests if relevant.
- [x] All breaking changes documented.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working c-iroh-blobs
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

3 participants