Skip to content

[8.17] [Synthetics] monitor status rule - remove return to avoid limiting alerts (#219291) #219317

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

Merged
merged 1 commit into from
Apr 26, 2025

Conversation

kibanamachine
Copy link
Contributor

Backport

This will backport the following commits from main to 8.17:

Questions ?

Please refer to the Backport tool documentation

…erts (elastic#219291)

## Summary

Resolves elastic#219292

Ensures that a distinct alert is created for each monitor that matches
the synthetics monitor status rule, by removing the early return.

### Release note
Synthetics monitor status rule - Ensures that an individual alert is
sent per monitor configuration when the "Receive distinct alerts per
location" toggle is unchecked.

### Testing
1. Set your local Kibana to connect to an oblt-cli cluster
2. Create 3 different HTTP monitors. Ensure they are always failing by
including a bad url like `https://doesnotexist.coooommmm.
3. Create a new "Synthetics monitor status rule" and disable the option
"Receive distinct alerts for each location"
4. Ensure you receive a distinct alert per monitor

(cherry picked from commit f6dbd0d)
@kibanamachine kibanamachine enabled auto-merge (squash) April 25, 2025 21:05
@botelastic botelastic bot added ci:project-deploy-observability Create an Observability project Team:obs-ux-management Observability Management User Experience Team labels Apr 25, 2025
@elasticmachine
Copy link
Contributor

Pinging @elastic/obs-ux-management-team (Team:obs-ux-management)

Copy link
Contributor

🤖 GitHub comments

Expand to view the GitHub comments

Just comment with:

  • /oblt-deploy : Deploy a Kibana instance using the Observability test environments.
  • run docs-build : Re-trigger the docs validation. (use unformatted text in the comment!)

@dominiqueclarke
Copy link
Contributor

/ci

@elasticmachine
Copy link
Contributor

elasticmachine commented Apr 26, 2025

💚 Build Succeeded

  • Buildkite Build
  • Commit: c633824
  • Kibana Serverless Image: docker.elastic.co/kibana-ci/kibana-serverless:pr-219317-c6338244a1c6

The CI Stats report is too large to be displayed here, check out the CI build annotation for this information.

History

cc @dominiqueclarke

@kibanamachine kibanamachine merged commit cb7940c into elastic:8.17 Apr 26, 2025
15 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport ci:project-deploy-observability Create an Observability project Team:obs-ux-management Observability Management User Experience Team
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants