-
Notifications
You must be signed in to change notification settings - Fork 8.5k
chore(streams): Significant events advanced settings #221316
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
chore(streams): Significant events advanced settings #221316
Conversation
| // 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( |
There was a problem hiding this comment.
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?
csr
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
miltonhultgren
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Streams changes LGTM
afharo
left a comment
There was a problem hiding this 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
|
Starting backport for target branches: 8.19 |
💚 Build Succeeded
Metrics [docs]Public APIs missing comments
History
cc @kdelemme |
💔 All backports failed
Manual backportTo create the backport manually run: Questions ?Please refer to the Backport tool documentation |
## 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
💚 All backports created successfully
Note: Successful backport PRs will be merged automatically after passing CI. Questions ?Please refer to the Backport tool documentation |
## 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>
…#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>
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.