Skip to content

sync-dir-stat misses space/inodes in trash #6035

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.

Sign up for GitHub

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

Jump to bottom
Closed
polyrabbit opened this issue Apr 22, 2025 · 2 comments · Fixed by #6105
Closed

sync-dir-stat misses space/inodes in trash #6035

polyrabbit opened this issue Apr 22, 2025 · 2 comments · Fixed by #6105
Assignees
@zhijian-pro
Labels
kind/bug Something isn't working
Milestone
Release 1.3

Comments

@polyrabbit
Copy link
Contributor

What happened:

The sync-dir-stat in cmd/fsck tool missed contents of .trash, causing the repaired capacity to be smaller than expected.

What you expected to happen:

How to reproduce it (as minimally and precisely as possible):

Anything else we need to know?

Environment:

  • JuiceFS version (use juicefs --version) or Hadoop Java SDK version:
    main branch
@polyrabbit polyrabbit added the kind/bug Something isn't working label Apr 22, 2025
@polyrabbit polyrabbit mentioned this issue Apr 22, 2025
Merged
@davies
Copy link
Contributor

davies commented Apr 24, 2025

fixed

@davies davies closed this as completed Apr 24, 2025
@jiefenghuang
Copy link
Contributor

The .trash part still not been included yet?

@davies davies reopened this Apr 24, 2025
@davies davies added this to the Release 1.3 milestone May 6, 2025
@zhijian-pro zhijian-pro self-assigned this May 8, 2025
@zhijian-pro zhijian-pro mentioned this issue May 13, 2025
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@zhijian-pro zhijian-pro

Labels
kind/bug Something isn't working
Projects
None yet
Milestone
Release 1.3
Development

Successfully merging a pull request may close this issue.

fix: calculate trash files when recalculating volume stats juicedata/juicefs
4 participants
@davies @polyrabbit @zhijian-pro @jiefenghuang