Skip to content
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

Use rate instead of irate in Hubble L7 workloads dashboard #21791

Merged
merged 1 commit into from
Oct 21, 2022

Conversation

chancez
Copy link
Contributor

@chancez chancez commented Oct 18, 2022

Initially I used irate because I was thinking these requests could be considered "fast moving", but irate only uses the most recent 2 datapoints for calculating rate, which causes graphs to fluctuate between the extremes a lot more than rate does. Rate will use all datapoints in the range vector, providing smoother graphs that actually better represent the data.

In particular, the error rate graph previously jumped from 100% to 0% on basically any changes to the value. See the screenshots for a before/after comparison.

Before:
Screen Shot 2022-10-18 at 9 51 39 AM

After:
Screen Shot 2022-10-18 at 9 51 49 AM

Use rate instead of irate in Hubble L7 workloads dashboard

Initially I used irate because I was thinking these requests could be
considered "fast moving", but irate only uses the most recent 2
datapoints for calculating rate, which causes graphs to fluctuate
between the extremes a lot more than rate does. Rate will use all
datapoints in the range vector, providing smoother graphs that actually
better represent the data.

Signed-off-by: Chance Zibolski <chance.zibolski@gmail.com>
@maintainer-s-little-helper maintainer-s-little-helper bot added the dont-merge/needs-release-note-label The author needs to describe the release impact of these changes. label Oct 18, 2022
@chancez chancez marked this pull request as ready for review October 18, 2022 16:52
@chancez chancez requested review from a team as code owners October 18, 2022 16:52
@rolinh rolinh added sig/hubble Impacts hubble server or relay area/metrics Impacts statistics / metrics gathering, eg via Prometheus. release-note/misc This PR makes changes that have no direct user impact. labels Oct 18, 2022
@maintainer-s-little-helper maintainer-s-little-helper bot removed the dont-merge/needs-release-note-label The author needs to describe the release impact of these changes. label Oct 18, 2022
@aanm aanm merged commit 713d3ea into master Oct 21, 2022
@aanm aanm deleted the pr/chancez/hubble_dashboard_rate_func branch October 21, 2022 22:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/metrics Impacts statistics / metrics gathering, eg via Prometheus. release-note/misc This PR makes changes that have no direct user impact. sig/hubble Impacts hubble server or relay
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants