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

fix: Update DLQNotEmpty evaluationPeriods #1401

Closed
wants to merge 1 commit into from
Closed

Conversation

adaviso
Copy link

@adaviso adaviso commented Apr 26, 2024

Problem


We are observing that the DLQNotEmpty alarms for both staging and ingestion are being regularly triggered. They auto-resolve after a single data point and do not leave meaningful info.

Solution


Update the evaluation period from 1 (5 minutes) to 2 (10 minutes) to allow the issue to auto-resolve without alarming.


By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license

@adaviso adaviso marked this pull request as draft May 2, 2024 23:41
auto-merge was automatically disabled May 2, 2024 23:41

Pull request was converted to draft

@adaviso adaviso closed this May 2, 2024
github-merge-queue bot pushed a commit that referenced this pull request May 2, 2024
## Problem

We are observing that the `DLQNotEmpty` alarms for both staging and
ingestion are being regularly triggered. They auto-resolve after a
single data point and do not leave meaningful info.

 ## Solution

Update the evaluation period from 1 (5 minutes) to 2 (10 minutes) to
allow the issue to auto-resolve without alarming.
 
_By submitting this pull request, I confirm that my contribution is made
under the terms of the Apache-2.0 license_

This was original authored as #1401, but needed the snapshots updated.
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.

2 participants