Skip to content

[SLO] Allow users to easily view good/bad events in Discover for event panel - #178008

Merged
simianhacker merged 4 commits into
elastic:mainfrom
simianhacker:slo-view-events-kql
Mar 7, 2024
Merged

simianhacker merged 4 commits into
elastic:mainfrom
simianhacker:slo-view-events-kql

Conversation

@simianhacker

@simianhacker simianhacker commented Mar 5, 2024 •

Copy link
Copy Markdown
Member

Summary

This PR adds a link to Discover for the "Good vs Bad" event chart for non-APM indicators for the "Last 24 hours" with all the appropriate filters applied. If the indicator is a "Custom KQL", the link will include disabled filters for "Good events" and "Bad events". If the user clicks on a "Good" or "Bad" bar on the chart for the "Custom KQL" indicator, this will open Discover with the appropriate filter activated along with the appropriate time range for the bar.

image image

@simianhacker simianhacker changed the title [SLO] Allow users to easily view good/bad events in Discover for even… [SLO] Allow users to easily view good/bad events in Discover for event panel Mar 5, 2024
@ghost

ghost commented Mar 5, 2024

Copy link
Copy Markdown

🤖 GitHub comments

Expand to view the GitHub comments

Just comment with:

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

@simianhacker
simianhacker force-pushed the slo-view-events-kql branch from 3f951eb to e56fec4 Compare March 5, 2024 16:22
@simianhacker
simianhacker marked this pull request as ready for review March 5, 2024 18:25
@simianhacker
simianhacker requested a review from a team as a code owner March 5, 2024 18:25
@simianhacker
simianhacker requested a review from kdelemme March 5, 2024 18:30
@simianhacker simianhacker added release_note:feature Makes this part of the condensed release notes Team:actionable-obs - DEPRECATED DEPRECATED v8.14.0 labels Mar 5, 2024
@elasticmachine

Copy link
Copy Markdown
Contributor

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

@kdelemme kdelemme left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code LGTM, let me test locally

@kdelemme kdelemme left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@simianhacker

Copy link
Copy Markdown
Member Author

@elasticmachine merge upstream

@simianhacker
simianhacker enabled auto-merge (squash) March 6, 2024 23:00
@kibana-ci

Copy link
Copy Markdown

💚 Build Succeeded

Metrics [docs]

Module Count

Fewer modules leads to a faster build time

id before after diff
observability 615 616 +1

Async chunks

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

id before after diff
observability 661.0KB 663.5KB +2.4KB

To update your PR or re-run it, just comment with:
@elasticmachine merge upstream

@simianhacker
simianhacker merged commit 77141f7 into elastic:main Mar 7, 2024
@kibanamachine kibanamachine added the backport:skip This PR does not require backporting label Mar 7, 2024
@simianhacker
simianhacker deleted the slo-view-events-kql branch April 17, 2024 15:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport:skip This PR does not require backporting Feature:SLO release_note:feature Makes this part of the condensed release notes Team:actionable-obs - DEPRECATED DEPRECATED v8.14.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants