grafana / alloy

OpenTelemetry Collector distribution with programmable pipelines
https://grafana.com/oss/alloy
Apache License 2.0
1.31k stars 180 forks source link

Alloy-Mixin: add severity and description to alerts, plus other improvements #1081

Closed gaantunes closed 3 months ago

gaantunes commented 3 months ago

PR Description

This is adapting Alloy Mixin alerts to comply with https://monitoring.mixins.dev/ best practices.

Additionally:

Which issue(s) this PR fixes

Notes to the Reviewer

During integration testing, we noticed alloy_component_controller_evaluating emits 2 active series for each instance, thus duplicating the running instances panel value.

image image

PR Checklist

gaantunes commented 3 months ago

@mattdurham have updated it to exclude the instance label, if guess we can merge now?