-
Notifications
You must be signed in to change notification settings - Fork 2
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
Add lustre changelog user stats output #47
Conversation
15474f7
to
44493ef
Compare
This depends on whamcloud/lustre-collector#66 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The patch seems fine. I think the commit message itself should be more along the lines of:
EX-nnnn component: short subject line
Longer description of change, and why
Signed-off-by: you name <[email protected]>
This more closely adheres to Lustre Commit message format.
We decided against this, because this is a public repository. |
53daee7
to
117df22
Compare
A number of CI issues in this one. |
Yes, they will get resolved when whamcloud/lustre-collector#66 is merged |
117df22
to
e3b0ee6
Compare
e3b0ee6
to
6423722
Compare
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #47 +/- ##
==========================================
+ Coverage 90.93% 91.42% +0.48%
==========================================
Files 8 8
Lines 993 1038 +45
Branches 993 1038 +45
==========================================
+ Hits 903 949 +46
+ Misses 82 81 -1
Partials 8 8 ☔ View full report in Codecov by Sentry. |
This PR includes functionality to query and report on the status of changelog users for each MDT.
This will provide a more granular insight into the changelog processing performance.
The output format is as follows:
This depends on whamcloud/lustre-collector#66