Skip to content

Added Argo Workflow Alerts - #1700

Merged
hazdl merged 1 commit into
mainfrom
hz/add-argo-workflow-alerts
Oct 6, 2026
Merged

hazdl merged 1 commit into
mainfrom
hz/add-argo-workflow-alerts

Conversation

@hazdl

@hazdl hazdl commented Oct 5, 2026

Copy link
Copy Markdown
Contributor

Review Alerts and Implement High Priority / High Level Alerts AP-1339

Summary
Add initial Prometheus alerting for Argo Workflows and update the monitoring Helm chart version.

Changes
Added ArgoWorkflowControllerFailing alert for the Argo Workflow Controller entering CrashLoopBackOff.
Added ArgoWorkflowsUnhealthy alert to detect newly failed or errored workflows using argo_workflows_count_total.
Added ArgoWorkflowsPending alert when more than 5 workflows remain pending for 15 minutes.
Added ArgoWorkflowSuccessRateLow alert when workflow success rate falls below 80%, with a minimum completed-workflow threshold.

Investigated Argo workflow metrics and changed the unhealthy alert from the historical argo_workflows_gauge metric to the workflow completion counter to avoid duplicate/historical failure counts.
Removed dynamic Prometheus $value templating from the alert description because the rules are rendered through Helm.
Bumped the monitoring Helm chart version from 0.1.34 to 0.1.37 to satisfy the chart version validation.

Validation
helm lint passes locally.
Helm template rendering successfully produces all four alert rules.
Verified the rendered Prometheus alerting configuration.

Follow-up
The dashboard contains additional signals that could be converted into alerts once production baselines and thresholds are agreed:

  • Workflow queue depth
  • Workflow queue latency
  • Workflow operation duration
  • Workflow pod pending
  • Missing workflow pods
  • WorkflowTemplate failure/runtime

These should be tracked as follow-up tickets rather than adding arbitrary thresholds to this change.

Comment thread release-please-config.json Outdated
Comment thread charts/monitoring/values.yaml Outdated
Comment thread charts/monitoring/values.yaml
Comment thread charts/monitoring/values.yaml Outdated
@hazdl
hazdl force-pushed the hz/add-argo-workflow-alerts branch from d6d2680 to 0d1ac04 Compare October 6, 2026 10:12
fix(workflows observability): apdate argo workflow success rate alert thresholds

fix: remove unintended file from pr
@hazdl
hazdl force-pushed the hz/add-argo-workflow-alerts branch from 0d1ac04 to 52a8aef Compare October 6, 2026 10:16
@hazdl
hazdl merged commit 961c0a3 into main Oct 6, 2026
71 checks passed
@hazdl
hazdl deleted the hz/add-argo-workflow-alerts branch October 6, 2026 12:25
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.

3 participants