-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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 99th percentile to stdout metrics #408
Comments
@aidylewis can you check if the merged PR #462 satisfies this issue? |
Thanks for the enhancement. Is this in master?
|
@aidylewis Yes, and will be in a new release before the end of the week. |
I've done a go get, built the source and moved the k6 binary into $GOPATH/bin. I am not seeing p99. |
p99 is not included by default, but you can now configure the summary output like so: |
Very nice. Speak soon.
|
Hey @robingustafsson, you mention that the 99th percentile is not included by default in the output. However I just wanted to clarify, are the 99th percentile metrics not gather by the core unless the output flag is provided? Or is it collected just not displayed unless the flag is provided? |
@V3ckt0r The core collects raw non-aggregated data points (for "trend" metrics like response times). The percentiles are calculated from the raw data points, so what's printed to stdout is just a calculated summary. |
@V3ckt0r Can we not store the raw metrics in InfluxDB and use the percentile function in InfluxQL? |
On sites that serve traffic at thousands of requests per second, we may need a higher percentile than 95. I think at least 99. Maybe also higher. 99 plus some more 9s?
The text was updated successfully, but these errors were encountered: