-
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] Swap plugin: Migrate to collectd 6 data structures. #4190
Conversation
@mrunge could you check this one? |
5baa427
to
305b92b
Compare
I think it would be better to merge #4198 first. |
8f6ee5c
to
e02789e
Compare
That PR has been merged. PTAL. |
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.
Code looks ok, but swap_submit_*
functions do not submit (or dispatch) metrics, they only append them => maybe the function names should change also in addition to their signature?
Thanks for the review! I have renamed |
This should align this plugin with the OpenTelemetry semantic conventions: https://opentelemetry.io/docs/specs/semconv/system/system-metrics/#pagingswap-metrics
Depending on the "ReportIO" setting, the plugin may emit paging operations or bytes. Use a different metric family for these two metrics. This also moves the decision whether or not to report I/O to the `swap_submit_io`. This fixes a bug on NetBSD where I/O was reported even if "ReportIO" was set to false.
This is a replacement for #3815.
ChangeLog: swap plugin: Migration to v6.0.