-
Notifications
You must be signed in to change notification settings - Fork 65
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
Experimental Native Histogram #57
Conversation
cce38fa
to
a70d836
Compare
0c0f15c
to
10e57f2
Compare
@olegbespalov I have rebased and addressed the request changes |
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.
Generally LGTM 💪
I left a few minor suggestions about the panics. The idea is to make them more transparent for the facing user.
Last but not least. Do you think we should update the README with the info about K6_PROMETHEUS_TREND_AS_NATIVE_HISTOGRAM
🤔
664496a
to
85a2b50
Compare
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.
👍
a0a8edb
to
b8b5d68
Compare
It requires the experimental Native Histogram feature enabled.
goos: linux goarch: amd64 pkg: github.com/grafana/xk6-output-prometheus-remote/pkg/remotewrite cpu: AMD Ryzen 7 4800H with Radeon Graphics BenchmarkK6TrendSinkAdd-16 92493090 13.25 ns/op BenchmarkHistogramSinkAdd-16 22255527 57.16 ns/op PASS ok github.com/grafana/xk6-output-prometheus-remote/pkg/remotewrite 4.167s
b8b5d68
to
7f7eee1
Compare
It adds an experimental mapping for Trend. It uses the new Prometheus' experimental feature Native Histogram.
A new
TrendAsNativeHistogram
option has been added for enabling it. If it is not explicitly enabled the Output continues to use the current AsGauges mapping.It also contains other changes not dedicated to this feature so they will be separated in the next few days with specific PRs:
Manual testing
The Docker compose has been updated so it can be used for testing. A new xk6 build based on this branch is required.
An example PromQL query for checking the generated data: