UNCLASSIFIED - NO CUI

Skip to content
Snippets Groups Projects

Istio sidecar mem PrometheusRule Alert Template Addition

Merged Ryan Garcia requested to merge istio-sidecar-mem-alerts into main
1 unresolved thread

Adding PrometheusRule template to chart/templates/bigbang which conditionally create an monitoring-istio-sidecar-heap-memory alert group with two alerts:

  • Warning at 50MB heap memory usage for sidecar pod
  • Critical alert at 100MB of memory usage for sidecar pod

went with heap portion since that is what is actively requested on the system and pod rather than what is in use, and was generally a few MBs lower.

Relates https://repo1.dso.mil/platform-one/big-bang/apps/core/istio-controlplane/-/issues/6

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
1 {{- if and .Values.istio.enabled (.Capabilities.APIVersions.Has "monitoring.coreos.com/v1") }}
2 apiVersion: monitoring.coreos.com/v1
3 kind: PrometheusRule
4 metadata:
5 name: istio-sidecar-heap-memory
6 namespace: {{ .Release.Namespace }}
7 spec:
8 groups:
9 - name: istio.rules
10 rules:
11 - alert: IstioSidecarMemModerate
12 annotations:
13 description: Istio Sidecar Memory is above 50MB for {{`{{`}} $labels.pod_name {{`}}`}}
14 summary: Istio Sidecar Memory usage is above average
15 expr: istio_agent_go_memstats_heap_sys_bytes{} * 0.00000095357432 > 50
  • Ryan Garcia added 1 commit

    added 1 commit

    • 6570fb0b - Update istio-proxy-mem-prometheusrule.yaml

    Compare with previous version

  • Lynn Still approved this merge request

    approved this merge request

  • Ryan Garcia changed milestone to %1.35.0

    changed milestone to %1.35.0

  • Ryan Garcia enabled an automatic merge when the pipeline for 6570fb0b succeeds

    enabled an automatic merge when the pipeline for 6570fb0b succeeds

  • merged

  • Ryan Garcia mentioned in commit 318070d1

    mentioned in commit 318070d1

  • Please register or sign in to reply
    Loading