Skip to content

fix(observe): render alert cadence in minutes - #184

Merged
rldyourmnd merged 1 commit into
mainfrom
fix/openobserve-alert-cadence
Aug 25, 2026
Merged

fix(observe): render alert cadence in minutes#184
rldyourmnd merged 1 commit into
mainfrom
fix/openobserve-alert-cadence

Conversation

@rldyourmnd

Copy link
Copy Markdown
Contributor

Summary

  • convert second-based rule cadence to OpenObserve's supported minute frequency
  • recompute consecutive-evaluation thresholds using the rendered cadence
  • keep the alert lookback period at least as wide as its evaluation interval
  • cover 30-second, 60-second, and 300-second contracts

Root cause

The renderer emitted evaluation_seconds: 30 as frequency: 30, frequency_type: minutes. The live OOM alert therefore evaluated every 30 minutes while querying only a five-minute delta and missed natural OOM incidents.

Validation

  • red test observed against the old renderer
  • make verify

Closes #182.

@rldyourmnd
rldyourmnd merged commit c0c952d into main Aug 25, 2026
10 checks passed
@rldyourmnd
rldyourmnd deleted the fix/openobserve-alert-cadence branch August 25, 2026 20:41
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.

fix(observe): scheduled alert seconds are rendered as minutes and miss short incidents

1 participant