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

Reduce FUSE transport log level. #200

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

nrath-js
Copy link

@nrath-js nrath-js commented Dec 4, 2024

The FUSE init flags are unlikely to be of use unless one is debugging a problem with the filesystem.

The messages about channel status get printed for every thread, and it's rather confusing for end-users to have a FUSE filesystem terminate with:

fuse filesystem umounted
FUSE channel already closed!
FUSE channel already closed!
FUSE channel already closed!
FUSE channel already closed!
FUSE channel already closed!
FUSE channel already closed!

...when everything went well.

The FUSE init flags are unlikely to be of use unless one is debugging a problem
with the filesystem.

The messages about channel status get printed for every thread, and it's rather
confusing for end-users to have a FUSE filesystem terminate with:

fuse filesystem umounted
FUSE channel already closed!
FUSE channel already closed!
FUSE channel already closed!
FUSE channel already closed!
FUSE channel already closed!
FUSE channel already closed!

...when everything went well.
@flokli
Copy link

flokli commented Mar 20, 2025

This is the same as #195.

I'm fine with merging this or the other, but it'd be nice to make a decision on this.

cc @imeoer @jiangliu @bergwolf

@flokli flokli mentioned this pull request Mar 20, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants