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(backport): Backport metrics work (KF-5558) to track/1.22 #537

Merged
merged 4 commits into from
Aug 28, 2024

Conversation

rgildein
Copy link
Contributor

Backporting multiple PR at once, but three of them are adding only dashboard and alert rules. Only the #477 is enabling metrics for istio-gateway.

This PR needs to me merged with merge commit instead of squash and merge.

@rgildein rgildein added the enhancement New feature or request label Aug 28, 2024
@rgildein rgildein self-assigned this Aug 28, 2024
@rgildein rgildein requested a review from a team as a code owner August 28, 2024 08:20
rgildein and others added 4 commits August 28, 2024 10:22
Add relations endpoints + update manifest to export metrics via
service port.

There are more rules, but no metrics for them was found. I also
cheded alert rules from [2]. The Grafana dashboards will be added
to istio-pilot in different PR, since most of metrics used in them
are from the istio-pilot.

---
[1]: https://samber.github.io/awesome-prometheus-alerts/rules.html#rule-istio-1-10
[2]: https://github.com/istio/tools/blob/release-1.14/perf/stability/alertmanager/prometheusrule.yaml
* Add istio control plane dashboard

The source of this dashboard is [1].

---
[1]: https://grafana.com/grafana/dashboards/7645-istio-control-plane-dashboard/

* Add aler rule from source [1] based on metrics [2]

Found some intteresting alert rules in [1], however based on istio
doc [2], only the last one will work, so I added it here.

---
[1]: https://samber.github.io/awesome-prometheus-alerts/rules#istio
[2]: https://istio.io/latest/docs/reference/commands/pilot-discovery/#metrics
Copy link
Contributor

@DnPlas DnPlas left a comment

Choose a reason for hiding this comment

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

Thanks @rgildein !

@rgildein rgildein merged commit 3eb9645 into track/1.22 Aug 28, 2024
19 checks passed
@rgildein rgildein deleted the chore/KF-6105/metrics branch August 28, 2024 14:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request Libraries: OK
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants