-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
[collectd 6] df plugin: Align metrics with OpenTelemetry recommendations. #4218
Conversation
The metrics were attached to the wrong metric family previously.
This also changes the report of usage to a ratio, as recommended by OpenTelemetry. Previously we reported a percentage.
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.
Approved.
I've suggested option names that are IMHO more readable/understandable, but such change can be done in separate PR (updating also options for already merged other PRs).
# ReportUsage true | ||
# ReportUtilization false |
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.
While I haven't complained about these in other PRs, I think these names are too similar i.e. easily confused. Adding unit to their names would make them more understandable without user needing to resort to documentation:
# ReportUsage true | |
# ReportUtilization false | |
# UsageBytes true | |
# UtilizationRatio false |
What do you think?
static bool report_usage = true; | ||
static bool report_utilization; |
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.
If config option names are changed, maybe these too?
static bool report_usage = true; | |
static bool report_utilization; | |
static bool report_bytes = true; | |
static bool report_ratio; |
Absolute/relative is also much better than usage/utilization. I don't really care whether it's that, or adding units, as long as those IMHO usability-wise awful usage/utilization names are avoided.
Yes, those names should be consistent across plugins. I.e. such changes are better in separate PR that changes all relevant names. |
system.filesystem.state
instead ofstate
.ChangeLog: DF plugin: The metric schema has been aligned with OpenTelemetry semantic conventions.