Skip to content

[Security Solution] {{state.signals_count}} Object not working (#156472)#156707

Merged
e40pud merged 1 commit intoelastic:mainfrom
e40pud:security/bugfix/156472-signals-count
May 5, 2023
Merged

[Security Solution] {{state.signals_count}} Object not working (#156472)#156707
e40pud merged 1 commit intoelastic:mainfrom
e40pud:security/bugfix/156472-signals-count

Conversation

@e40pud
Copy link
Contributor

@e40pud e40pud commented May 4, 2023 •
edited by kibanamachine
Loading

Summary

Original ticket: #156472

These changes adds {{state.signals_count}} object to be available in message body for the "For each alert" option.

@e40pud e40pud added Team:ResponseOps Platform ResponseOps team (formerly the Cases and Alerting teams) t// Team:Detection Alerts Security Detection Alerts Area Team backport:prev-minor ci:cloud-deploy Create or update a Cloud deployment labels May 4, 2023
@e40pud e40pud self-assigned this May 4, 2023
@e40pud e40pud requested a review from a team as a code owner May 4, 2023 15:26
@e40pud e40pud requested a review from a team May 4, 2023 15:26
@elasticmachine
Copy link
Contributor

elasticmachine commented May 4, 2023

Pinging @elastic/response-ops (Team:ResponseOps)

@e40pud e40pud added the release_note:skip Skip the PR/issue when compiling release notes label May 4, 2023
ymao1
ymao1 approved these changes May 4, 2023
View reviewed changes
Copy link
Contributor

@ymao1 ymao1 left a comment

Choose a reason for hiding this comment

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

LGTM

@e40pud e40pud mentioned this pull request May 4, 2023
Closed
@kibana-ci
Copy link

kibana-ci commented May 4, 2023

💚 Build Succeeded

Metrics [docs]

Unknown metric groups

ESLint disabled line counts

id before after diff
enterpriseSearch 19 21 +2
securitySolution 398 401 +3
total +5

Total ESLint disabled count

id before after diff
enterpriseSearch 20 22 +2
securitySolution 478 481 +3
total +5

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

cc @e40pud

@e40pud e40pud merged commit 99e5e38 into elastic:main May 5, 2023
@kibanamachine kibanamachine mentioned this pull request May 5, 2023
Merged
kibanamachine pushed a commit to kibanamachine/kibana that referenced this pull request May 5, 2023
@e40pud
[Security Solution] {{state.signals_count}} Object not working (elast…
30432a9 
…ic#156472) (elastic#156707)

## Summary

Original ticket: elastic#156472

These changes adds `{{state.signals_count}}` object to be available in
message body for the `"For each alert"` option.

(cherry picked from commit 99e5e38)
@kibanamachine
Copy link
Contributor

kibanamachine commented May 5, 2023

💚 All backports created successfully

Status Branch Result
✅ 8.8

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

Questions ?

Please refer to the Backport tool documentation

jloleysens added a commit that referenced this pull request May 5, 2023
@jloleysens
Merge branch 'main' into versioned-router-and-oas
630343b 
* main: (153 commits)
  [Security Solution] {{state.signals_count}} Object not working (#156472) (#156707)
  [Synthetics] refresh data on visualization scrubbing (#156777)
  [RAM] Docs for slack improvements (#153885)
  [RAM] Alert search bar only KQL (#155947)
  [ML] Functional tests - stabilize export job tests (#156586)
  [Saved Search] Update saved search schema to allow empty `sort` arrays (#156769)
  [ML] Rename `curated` model type to `elastic` (#156684)
  [Discover] Enable sharing for text based languages (#156652)
  [api-docs] 2023-05-05 Daily api_docs build (#156781)
  Upgrade EUI to v77.2.2 (#155208)
  [RAM][Maintenance Window][8.8]Fix window maintenance workflow (#156427)
  [DOCS] Case file attachments (#156459)
  [D4C] additional error handling for 'block' action added + policy editor UI fixes (#156629)
  [Enterprise Search] refactor(SearchApplications): rename telemetry ids (#156733)
  [Enterprise Search] Add telemetry to ELSER deployment buttons + error (#156545)
  [Security Solution] fixes Data Quality dashboard errors when a `basePath` is configured (#156233)
  [Logs onboarding] StepsFooter outside of main panel (#156686)
  [Security Solution] Add a migration to unmute custom Security Solution rules (#156593)
  [Enterprise Search][Behavioral Analytics] Update formulas (#156704)
  Add API Events to Endpoint Security Advanced Policy (#156718)
  ...
kibanamachine added a commit that referenced this pull request May 5, 2023
@kibanamachine @e40pud
[8.8] [Security Solution] {{state.signals_count}} Object not working (#…
3546b82 
…156472) (#156707) (#156800)

# Backport

This will backport the following commits from `main` to `8.8`:
- [[Security Solution] {{state.signals_count}} Object not working
(#156472) (#156707)](#156707)

<!--- Backport version: 8.9.7 -->

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

<!--BACKPORT [{"author":{"name":"Ievgen
Sorokopud","email":"ievgen.sorokopud@elastic.co"},"sourceCommit":{"committedDate":"2023-05-05T08:23:37Z","message":"[Security
Solution] {{state.signals_count}} Object not working (#156472)
(#156707)\n\n## Summary\r\n\r\nOriginal ticket:
https://github.com/elastic/kibana/issues/156472\r\n\r\nThese changes
adds `{{state.signals_count}}` object to be available in\r\nmessage body
for the `\"For each alert\"`
option.","sha":"99e5e38111b9c82fca16645f939628aa72bfef73","branchLabelMapping":{"^v8.9.0$":"main","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:skip","Team:ResponseOps","Team:Detection
Alerts","backport:prev-minor","ci:cloud-deploy","v8.9.0"],"number":156707,"url":"https://github.com/elastic/kibana/pull/156707","mergeCommit":{"message":"[Security
Solution] {{state.signals_count}} Object not working (#156472)
(#156707)\n\n## Summary\r\n\r\nOriginal ticket:
https://github.com/elastic/kibana/issues/156472\r\n\r\nThese changes
adds `{{state.signals_count}}` object to be available in\r\nmessage body
for the `\"For each alert\"`
option.","sha":"99e5e38111b9c82fca16645f939628aa72bfef73"}},"sourceBranch":"main","suggestedTargetBranches":[],"targetPullRequestStates":[{"branch":"main","label":"v8.9.0","labelRegex":"^v8.9.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/156707","number":156707,"mergeCommit":{"message":"[Security
Solution] {{state.signals_count}} Object not working (#156472)
(#156707)\n\n## Summary\r\n\r\nOriginal ticket:
https://github.com/elastic/kibana/issues/156472\r\n\r\nThese changes
adds `{{state.signals_count}}` object to be available in\r\nmessage body
for the `\"For each alert\"`
option.","sha":"99e5e38111b9c82fca16645f939628aa72bfef73"}}]}]
BACKPORT-->

Co-authored-by: Ievgen Sorokopud <ievgen.sorokopud@elastic.co>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@ymao1 ymao1 ymao1 approved these changes

Assignees

@e40pud e40pud

Labels

ci:cloud-deploy Create or update a Cloud deployment release_note:skip Skip the PR/issue when compiling release notes Team:Detection Alerts Security Detection Alerts Area Team Team:ResponseOps Platform ResponseOps team (formerly the Cases and Alerting teams) t// v8.8.0 v8.9.0

Projects

No open projects
AppEx: ResponseOps - Rules & Alerts Management
Status: Done

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@e40pud @elasticmachine @kibana-ci @kibanamachine @ymao1