Skip to content

Conversation

@kdelemme
Copy link
Contributor

@kdelemme kdelemme commented May 22, 2025

Summary

Resolves https://github.com/elastic/streams-program/issues/206?issue=elastic%7Cstreams-program%7C317

This PR introduces a new Advanced Settings for enabling streams significant events. It is disabled by default.

This PR also registers the rule regardless of the settings.

Using an Advanced Settings makes it easier for internal customers to toggle the feature on serverless.

Screenshot 2025-06-17 at 11 15 11 AM

@kdelemme kdelemme requested a review from a team as a code owner June 18, 2025 12:52
// expect(rules.body.data[0].params.query).to.eql(
// `FROM ${STREAM_NAME},${STREAM_NAME}.* METADATA _id, _source | WHERE KQL("""message:'OutOfMemoryError'""")`
// );
const rules = await alertingApi.searchRules(
Copy link
Contributor

Choose a reason for hiding this comment

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

Now that the feature flag is removed, should we remove this line too from feature_flags.ts?

Copy link
Contributor

@csr csr left a comment

Choose a reason for hiding this comment

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

LGTM!

Copy link
Contributor

@miltonhultgren miltonhultgren left a comment

Choose a reason for hiding this comment

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

Streams changes LGTM

Copy link
Member

@afharo afharo left a comment

Choose a reason for hiding this comment

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

UI Settings telemetry LGTM

@dgieselaar dgieselaar requested a review from a team as a code owner June 24, 2025 09:01
@dgieselaar dgieselaar enabled auto-merge (squash) June 24, 2025 09:02
@dgieselaar dgieselaar merged commit 8b65a14 into elastic:main Jun 24, 2025
10 checks passed
@kibanamachine
Copy link
Contributor

Starting backport for target branches: 8.19

https://github.com/elastic/kibana/actions/runs/15852765818

@elasticmachine
Copy link
Contributor

💚 Build Succeeded

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/management-settings-ids 126 127 +1
Unknown metric groups

API count

id before after diff
@kbn/management-settings-ids 127 128 +1

History

cc @kdelemme

@kibanamachine
Copy link
Contributor

💔 All backports failed

Status Branch Result
8.19 Backport failed because of merge conflicts

Manual backport

To create the backport manually run:

node scripts/backport --pr 221316

Questions ?

Please refer to the Backport tool documentation

dgieselaar pushed a commit to dgieselaar/kibana that referenced this pull request Jun 25, 2025
## Summary

Resolves
elastic/streams-program#317

This PR introduces a new Advanced Settings for enabling streams
significant events. It is disabled by default.

This PR also registers the rule regardless of the settings.

Using an Advanced Settings makes it easier for internal customers to
toggle the feature on serverless.

<img width="1495" alt="Screenshot 2025-06-17 at 11 15 11 AM"
src="https://github.com/user-attachments/assets/27023c52-20a9-476f-9dfd-d3b8b3f03e94"
/>

---------

Co-authored-by: kibanamachine <42973632+kibanamachine@users.noreply.github.com>
Co-authored-by: Dario Gieselaar <dario.gieselaar@elastic.co>
(cherry picked from commit 8b65a14)

# Conflicts:
#	x-pack/platform/plugins/private/discover_enhanced/ui_tests/tests/discover_cdp_perf.spec.ts
#	x-pack/test/tsconfig.json
@dgieselaar
Copy link
Contributor

💚 All backports created successfully

Status Branch Result
8.19

Note: Successful backport PRs will be merged automatically after passing CI.

Questions ?

Please refer to the Backport tool documentation

akowalska622 pushed a commit to akowalska622/kibana that referenced this pull request Jun 25, 2025
## Summary

Resolves
elastic/streams-program#317

This PR introduces a new Advanced Settings for enabling streams
significant events. It is disabled by default.

This PR also registers the rule regardless of the settings. 

Using an Advanced Settings makes it easier for internal customers to
toggle the feature on serverless.

<img width="1495" alt="Screenshot 2025-06-17 at 11 15 11 AM"
src="https://github.com/user-attachments/assets/27023c52-20a9-476f-9dfd-d3b8b3f03e94"
/>

---------

Co-authored-by: kibanamachine <42973632+kibanamachine@users.noreply.github.com>
Co-authored-by: Dario Gieselaar <dario.gieselaar@elastic.co>
dgieselaar added a commit that referenced this pull request Jun 25, 2025
…#225207)

# Backport

This will backport the following commits from `main` to `8.19`:
- [chore(streams): Significant events advanced settings
(#221316)](#221316)

<!--- Backport version: 10.0.1 -->

### Questions ?
Please refer to the [Backport tool
documentation](https://github.com/sorenlouv/backport)

<!--BACKPORT [{"author":{"name":"Kevin
Delemme","email":"kevin.delemme@elastic.co"},"sourceCommit":{"committedDate":"2025-06-24T14:07:48Z","message":"chore(streams):
Significant events advanced settings (#221316)\n\n##
Summary\n\nResolves\nhttps://github.com/elastic/streams-program/issues/206?issue=elastic%7Cstreams-program%7C317\n\nThis
PR introduces a new Advanced Settings for enabling streams\nsignificant
events. It is disabled by default.\n\nThis PR also registers the rule
regardless of the settings. \n\nUsing an Advanced Settings makes it
easier for internal customers to\ntoggle the feature on
serverless.\n\n<img width=\"1495\" alt=\"Screenshot 2025-06-17 at 11 15
11 AM\"\nsrc=\"https://github.com/user-attachments/assets/27023c52-20a9-476f-9dfd-d3b8b3f03e94\"\n/>\n\n---------\n\nCo-authored-by:
kibanamachine
<42973632+kibanamachine@users.noreply.github.com>\nCo-authored-by: Dario
Gieselaar
<dario.gieselaar@elastic.co>","sha":"8b65a14fcc566ee92876374910cb3b691f01cdee","branchLabelMapping":{"^v9.1.0$":"main","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:skip","backport:version","v9.1.0","v8.19.0","author:obs-ux-management"],"title":"chore(streams):
Significant events advanced
settings","number":221316,"url":"https://github.com/elastic/kibana/pull/221316","mergeCommit":{"message":"chore(streams):
Significant events advanced settings (#221316)\n\n##
Summary\n\nResolves\nhttps://github.com/elastic/streams-program/issues/206?issue=elastic%7Cstreams-program%7C317\n\nThis
PR introduces a new Advanced Settings for enabling streams\nsignificant
events. It is disabled by default.\n\nThis PR also registers the rule
regardless of the settings. \n\nUsing an Advanced Settings makes it
easier for internal customers to\ntoggle the feature on
serverless.\n\n<img width=\"1495\" alt=\"Screenshot 2025-06-17 at 11 15
11 AM\"\nsrc=\"https://github.com/user-attachments/assets/27023c52-20a9-476f-9dfd-d3b8b3f03e94\"\n/>\n\n---------\n\nCo-authored-by:
kibanamachine
<42973632+kibanamachine@users.noreply.github.com>\nCo-authored-by: Dario
Gieselaar
<dario.gieselaar@elastic.co>","sha":"8b65a14fcc566ee92876374910cb3b691f01cdee"}},"sourceBranch":"main","suggestedTargetBranches":["8.19"],"targetPullRequestStates":[{"branch":"main","label":"v9.1.0","branchLabelMappingKey":"^v9.1.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/221316","number":221316,"mergeCommit":{"message":"chore(streams):
Significant events advanced settings (#221316)\n\n##
Summary\n\nResolves\nhttps://github.com/elastic/streams-program/issues/206?issue=elastic%7Cstreams-program%7C317\n\nThis
PR introduces a new Advanced Settings for enabling streams\nsignificant
events. It is disabled by default.\n\nThis PR also registers the rule
regardless of the settings. \n\nUsing an Advanced Settings makes it
easier for internal customers to\ntoggle the feature on
serverless.\n\n<img width=\"1495\" alt=\"Screenshot 2025-06-17 at 11 15
11 AM\"\nsrc=\"https://github.com/user-attachments/assets/27023c52-20a9-476f-9dfd-d3b8b3f03e94\"\n/>\n\n---------\n\nCo-authored-by:
kibanamachine
<42973632+kibanamachine@users.noreply.github.com>\nCo-authored-by: Dario
Gieselaar
<dario.gieselaar@elastic.co>","sha":"8b65a14fcc566ee92876374910cb3b691f01cdee"}},{"branch":"8.19","label":"v8.19.0","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"}]}]
BACKPORT-->

---------

Co-authored-by: Kevin Delemme <kevin.delemme@elastic.co>
Co-authored-by: kibanamachine <42973632+kibanamachine@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

author:obs-ux-management PRs authored by the obs ux management team backport:version Backport to applied version labels release_note:skip Skip the PR/issue when compiling release notes v8.19.0 v9.1.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.