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

vfs: merge fuse ops metrics in .stats #4560

Merged
merged 1 commit into from
Mar 22, 2024
Merged

vfs: merge fuse ops metrics in .stats #4560

merged 1 commit into from
Mar 22, 2024

Conversation

SandyXSD
Copy link
Contributor

A patch for #4554 .
Without it there will be multiple lines for fuse ops metrics:

juicefs_fuse_ops_durations_histogram_seconds_total 101
juicefs_fuse_ops_durations_histogram_seconds_sum 1.305467913
juicefs_fuse_ops_durations_histogram_seconds_total 205
juicefs_fuse_ops_durations_histogram_seconds_sum 1.6931756689999993
juicefs_fuse_ops_durations_histogram_seconds_total 211
juicefs_fuse_ops_durations_histogram_seconds_sum 0.05859979800000001
juicefs_fuse_ops_durations_histogram_seconds_total 315
juicefs_fuse_ops_durations_histogram_seconds_sum 0.09348225000000002

Copy link

codecov bot commented Mar 22, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 55.72%. Comparing base (7de5d50) to head (8b922b7).
Report is 2 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #4560      +/-   ##
==========================================
- Coverage   55.72%   55.72%   -0.01%     
==========================================
  Files         157      157              
  Lines       42968    42969       +1     
==========================================
- Hits        23946    23944       -2     
- Misses      16364    16366       +2     
- Partials     2658     2659       +1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@SandyXSD SandyXSD requested a review from davies March 22, 2024 03:55
@davies davies merged commit a0d24c8 into main Mar 22, 2024
32 checks passed
@davies davies deleted the fix-fuse-metric branch March 22, 2024 06:58
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