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

chore: make SLI graphs more readable #177

Merged
merged 1 commit into from
Dec 12, 2023
Merged

Conversation

stehessel
Copy link
Contributor

@stehessel stehessel commented Dec 12, 2023

Make the SLI graphs more readable by showing a proxy that renders better in Grafana. The problem right now is that Grafana swallows up spikes and instead renders a flat line. The proxy metric works by extending the "downtime" interval via changes(...[1h]), such that Grafana has an easier way rendering it. The downside is that downtimes may appear longer than they were - however, the overall percentages (9' notation) are still exact.

Before:
Screenshot 2023-12-12 at 21 07 02

After:
Screenshot 2023-12-12 at 21 06 55

The dashboard can be viewed temporarily under https://grafana-route-rhacs-observability.apps.acs-prod-dp-01.pnz3.p1.openshiftapps.com/d/a3d89363-95c2-470f-82d8-cd9c62fe3840/rhacs-dataplane-central-slos-copy?orgId=1.

@stehessel stehessel requested a review from a team as a code owner December 12, 2023 20:16
Copy link
Member

@rukletsov rukletsov left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm impressed every time by the size of the diff a simple change like this generates. This time it is apparently different indentation, are you sure you haven't accidentally messed it up somehow?

Approving the declared change, I resign verifying the diff.

Copy link
Member

@rukletsov rukletsov left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ah, it's spaces vs. tabs. Maybe you can fix this one, Stephan.

@stehessel
Copy link
Contributor Author

Ah, it's spaces vs. tabs. Maybe you can fix this one, Stephan.
done

@stehessel stehessel merged commit 4e0bbbd into master Dec 12, 2023
1 check passed
@stehessel stehessel deleted the chore/sli-graphs-readable branch December 12, 2023 21:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants