Skip to content
This repository has been archived by the owner on May 21, 2024. It is now read-only.

Implement quota stats #58

Merged
merged 2 commits into from
May 8, 2024
Merged

Implement quota stats #58

merged 2 commits into from
May 8, 2024

Conversation

RDruon
Copy link
Contributor

@RDruon RDruon commented Jul 10, 2023

Will fix #51

@RDruon RDruon self-assigned this Jul 10, 2023
src/fixtures/valid/valid.txt Outdated Show resolved Hide resolved
@RDruon RDruon changed the title Partially implement qmt quota stats Implement quota stats Jul 10, 2023
@RDruon RDruon requested a review from jgrund July 11, 2023 07:22
@RDruon RDruon force-pushed the rdruon/51 branch 2 times, most recently from 9e583dc to c7ff72f Compare July 18, 2023 12:55
@RDruon RDruon marked this pull request as ready for review July 18, 2023 12:55
@breuhan breuhan added the enhancement New feature or request label Apr 18, 2024
@breuhan breuhan self-assigned this Apr 18, 2024
@breuhan
Copy link
Contributor

breuhan commented Apr 22, 2024

@jgrund Can we have this reviewed? This would allow us to merge whamcloud/lustrefs-exporter#46

src/types.rs Outdated Show resolved Hide resolved
Copy link

codecov bot commented May 8, 2024

Codecov Report

Attention: Patch coverage is 98.19005% with 4 lines in your changes are missing coverage. Please review.

Project coverage is 93.38%. Comparing base (76653d7) to head (821a79c).

Files Patch % Lines
src/types.rs 50.00% 4 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master      #58      +/-   ##
==========================================
+ Coverage   93.13%   93.38%   +0.25%     
==========================================
  Files          31       33       +2     
  Lines        4077     4294     +217     
  Branches     4077     4294     +217     
==========================================
+ Hits         3797     4010     +213     
- Misses        259      263       +4     
  Partials       21       21              

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

src/quota/quota_parser.rs Outdated Show resolved Hide resolved
src/quota/quota_parser.rs Outdated Show resolved Hide resolved
src/quota/quota_parser.rs Outdated Show resolved Hide resolved
src/quota/quota_parser.rs Outdated Show resolved Hide resolved
src/quota/quota_parser.rs Outdated Show resolved Hide resolved
src/quota/quota_parser.rs Outdated Show resolved Hide resolved
jgrund added 2 commits May 8, 2024 16:19
Implement osd quota stats

Fix test

Add fixtures

Add QuotaManager enum and removed "glb-" from qmt_stat

Fix typo in QMTStat::Prj assignement

Fix wording

Signed-off-by: Joe Grund <jgrund@whamcloud.io>
Signed-off-by: Joe Grund <jgrund@whamcloud.io>
@jgrund jgrund merged commit 3f166c3 into master May 8, 2024
8 checks passed
@jgrund jgrund deleted the rdruon/51 branch May 8, 2024 20:25
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Support for quotas stats
3 participants