chore(streams): disable sig events on logs essentials#226853

Merged

kdelemme merged 9 commits intoelastic:mainfrom

kdelemme:disable-sig-events-log-essentials

Jul 8, 2025

Contributor

kdelemme commented Jul 7, 2025 •

edited

Loading

Summary

Resolves #225632

This PR conditionally registers the streams significant events UI settings based on the current tier (disabled for log essentials)
It also extracts both streams UI settings registration into a common function, and conditionally registers the settings in the allow list based on the tier.

Testing

Run Kibana/ES with serverless logs essentials

yarn es serverless --projectType oblt --serverless.observability.tier=logs_essentials -E xpack.ml.enabled=false
yarn serverless-oblt --pricing.tiers.products='[{"name":"observability","tier":"logs_essentials"}]'\

Assert advanced setting for Streams Significant Events is not displayed
Enable streams: POST kbn:/api/streams/_enable
Add a doc:

POST logs/_doc
{
    "@timestamp": "2025-07-07T10:00:10.000Z",
    "message": "test 2",
}

Go to Streams UI, Assert Significant Events tab is not displayed
Assert going to the hidden tab URL redirects to another tab: http://localhost:5601/app/streams/logs/management/significantEvents

kdelemme added 5 commits

July 7, 2025 09:13


          Check pricing tier for sig events preview API

0b1db8b


          Move both feature flag into same register function

c250d89


          Register sig events UI settings only when feature available for tier

b954dd9


          Assert sig events feature available for tier in UI

951e037

Fix

130ce7f

github-actions bot added the author:obs-ux-management label

kdelemme marked this pull request as ready for review

July 7, 2025 16:03

kdelemme requested review from a team as code owners

July 7, 2025 16:03


          [CI] Auto-commit changed files from 'node scripts/styled_components_m…

fb7dd8a

…apping'

botelastic bot added the ci:project-deploy-observability label

Contributor

github-actions bot commented Jul 7, 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!)

kdelemme added 2 commits

July 7, 2025 13:56


          Handle rejection

18f8a38


          Redirect when attempting to access the tab by url

68d9949

kdelemme added release_note:skip Team:actionable-obs backport:version v9.2.0 labels

kdelemme self-assigned this

Contributor

elasticmachine commented Jul 7, 2025

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


          handle error

ddd3529

Contributor

elasticmachine commented Jul 7, 2025 •

edited

Loading

💚 Build Succeeded

Buildkite Build
Commit: ddd3529
Kibana Serverless Image: docker.elastic.co/kibana-ci/kibana-serverless:pr-226853-ddd3529a1415

Metrics [docs]

Public APIs missing comments

Total count of every public API that lacks a comment. Target amount is 0. Run node scripts/build_api_docs --plugin [yourplugin] --stats comments for more detailed information.

id	before	after	diff
`@kbn/serverless-observability-settings`	2	3	+1

Async chunks

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

id	before	after	diff
`streamsApp`	578.2KB	578.4KB	+130.0B

Unknown metric groups

API count

id	before	after	diff
`@kbn/serverless-observability-settings`	2	3	+1

History

💔 Build #316481 failed 68d9949
💔 Build #316444 failed fb7dd8a

tonyghiani approved these changes

View reviewed changes

Contributor

tonyghiani left a comment

Changes LGTM, thanks for this quick update!

tonyghiani changed the title ~~chore(streams): diasble sig events on logs essentials~~ chore(streams): disable sig events on logs essentials

shahzad31 approved these changes

View reviewed changes

kdelemme merged commit 18352a4 into elastic:main

12 checks passed

kibanamachine added the backport missing label

Contributor

kibanamachine commented Jul 10, 2025

Friendly reminder: Looks like this PR hasn’t been backported yet.
To create automatically backports add a backport:* label or prevent reminders by adding the backport:skip label.
You can also create backports manually by running node scripts/backport --pr 226853 locally
cc: @kdelemme

6 similar comments

Contributor

kibanamachine commented Jul 11, 2025

Friendly reminder: Looks like this PR hasn’t been backported yet.
To create automatically backports add a backport:* label or prevent reminders by adding the backport:skip label.
You can also create backports manually by running node scripts/backport --pr 226853 locally
cc: @kdelemme

Contributor

kibanamachine commented Jul 14, 2025

Friendly reminder: Looks like this PR hasn’t been backported yet.
To create automatically backports add a backport:* label or prevent reminders by adding the backport:skip label.
You can also create backports manually by running node scripts/backport --pr 226853 locally
cc: @kdelemme

Contributor

kibanamachine commented Jul 15, 2025

Friendly reminder: Looks like this PR hasn’t been backported yet.
To create automatically backports add a backport:* label or prevent reminders by adding the backport:skip label.
You can also create backports manually by running node scripts/backport --pr 226853 locally
cc: @kdelemme

Contributor

kibanamachine commented Jul 16, 2025

Friendly reminder: Looks like this PR hasn’t been backported yet.
To create automatically backports add a backport:* label or prevent reminders by adding the backport:skip label.
You can also create backports manually by running node scripts/backport --pr 226853 locally
cc: @kdelemme

Contributor

kibanamachine commented Jul 17, 2025

Friendly reminder: Looks like this PR hasn’t been backported yet.
To create automatically backports add a backport:* label or prevent reminders by adding the backport:skip label.
You can also create backports manually by running node scripts/backport --pr 226853 locally
cc: @kdelemme

Contributor

kibanamachine commented Jul 18, 2025

Friendly reminder: Looks like this PR hasn’t been backported yet.
To create automatically backports add a backport:* label or prevent reminders by adding the backport:skip label.
You can also create backports manually by running node scripts/backport --pr 226853 locally
cc: @kdelemme

kdelemme added backport:skip and removed backport missing backport:version labels

kertal pushed a commit to kertal/kibana that referenced this pull request


          chore(streams): disable sig events on logs essentials (elastic#226853)

7ac9500

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

Labels

author:obs-ux-management backport:skip ci:project-deploy-observability release_note:skip Team:actionable-obs v9.2.0