Skip to content

[Actionable Obs] - Update alert table muting icon to rely on the alert doc#245514

Open
fkanout wants to merge 1 commit intoelastic:mainfrom
fkanout:alert-muting-follow-up-fix-alert-table-ui
Open

[Actionable Obs] - Update alert table muting icon to rely on the alert doc#245514
fkanout wants to merge 1 commit intoelastic:mainfrom
fkanout:alert-muting-follow-up-fix-alert-table-ui

Conversation

@fkanout
Copy link
Copy Markdown
Contributor

@fkanout fkanout commented Dec 8, 2025

Summary

After merging Store Alert Muted Status Directly in Alert Documents
We need the alert table to use the new field instead of the rule SO, especially when the KQL filter is used to have a consistent user experience
Also, this PR adds mute and unmute functional tests.

How to test 🧪

  • Create a rule that fires alerts
  • Mute alert(s)
  • Apply kibana.alert.muted: true in the KQL filter
  • You should ONLY see the muted alerts

(redo the same thing with unmute)

@fkanout fkanout requested review from a team as code owners December 8, 2025 11:52
@botelastic botelastic bot added the ci:project-deploy-observability Create an Observability project label Dec 8, 2025
@github-actions github-actions bot added the author:actionable-obs PRs authored by the actionable obs team label Dec 8, 2025
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Dec 8, 2025

🤖 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!)

@fkanout fkanout self-assigned this Dec 8, 2025
@fkanout fkanout added release_note:skip Skip the PR/issue when compiling release notes backport:skip This PR does not require backporting v9.3.0 labels Dec 8, 2025
@kibanamachine
Copy link
Copy Markdown
Contributor

Flaky Test Runner Stats

🎉 All tests passed! - kibana-flaky-test-suite-runner#10007

[✅] x-pack/solutions/observability/test/observability_functional/with_rac_write.config.ts: 50/50 tests passed.

see run history

@momovdg momovdg requested a review from a team as a code owner March 11, 2026 16:33
Read muted state directly from the kibana.alert.muted field on the alert
document instead of fetching it from the rule saved object via a separate
API call. This ensures the mute/unmute toggle in the alerts table is
immediately consistent with KQL filters like kibana.alert.muted: true,
eliminating a brief incorrect "Mute" label while the rule SO loads.

Made-with: Cursor
@fkanout fkanout force-pushed the alert-muting-follow-up-fix-alert-table-ui branch from 76ebe25 to 18fc7ac Compare March 16, 2026 11:31
@elasticmachine
Copy link
Copy Markdown
Contributor

elasticmachine commented Mar 16, 2026

💚 Build Succeeded

  • Buildkite Build
  • Commit: 18fc7ac
  • Kibana Serverless Image: docker.elastic.co/kibana-ci/kibana-serverless:pr-245514-18fc7acd94d7

Metrics [docs]

Async chunks

Total size of all lazy-loaded chunks that will be downloaded as the user navigates the app

id before after diff
cases 2.0MB 2.0MB -82.0B
embeddableAlertsTable 1.2MB 1.2MB -193.0B
ml 5.8MB 5.8MB -217.0B
observability 2.0MB 2.0MB -226.0B
securitySolution 11.4MB 11.4MB -59.0B
triggersActionsUi 1.9MB 1.9MB -202.0B
total -979.0B

Page load bundle

Size of the bundles that are downloaded on every page load. Target size is below 100kb

id before after diff
cases 202.4KB 202.4KB +24.0B
ml 86.5KB 86.5KB +24.0B
total +48.0B

History

cc @fkanout

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

author:actionable-obs PRs authored by the actionable obs team backport:skip This PR does not require backporting ci:project-deploy-observability Create an Observability project release_note:skip Skip the PR/issue when compiling release notes v9.3.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants