-
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] cache: Provide the first metric value available alongside the first metric time. #4265
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…ded. This simplifies writing unit tests using the cache.
…nd value. Returning both in one call ensures these values are consistent, i.e. it avoids the case that there might have been an update inbewtween reading time and value.
This also improved the return values to give more information about which error condition occurred.
…d with a gauge metric.
eero-t
requested changes
Jan 31, 2024
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.
Looks OK in general, but there are some potential bug(s) in format_stackdriver.c
changes, and some API inconsistency.
eero-t
requested changes
Jan 31, 2024
eero-t
approved these changes
Jan 31, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
The write_stackdriver plugin uses the first observed time to scale counters so they appear to start from zero. OpenTelemetry calls this inserting true reset points. The new API returns the first metric time and first metric time in one call for consistency and efficiency.
This PR contains a few other, smaller improvements to the cache:
uc_insert()
anduc_get_rate()
has been added.uc_init()
function has been removed in favor of automatic initialization. This simplifies using the cache from unit tests.uc_get_rate()
has been documented.ChangeLog: Daemon: The first observed metric value has been added to the metrics cache.