Skip to content

[8.14] [Telemetry][Security Solution] Use the proper index to query builtin alerts (#187859)#188235

Merged
szaffarano merged 2 commits into8.14from
szaffarano/backport/8.14/pr_187859
Jul 15, 2024
Merged

[8.14] [Telemetry][Security Solution] Use the proper index to query builtin alerts (#187859)#188235
szaffarano merged 2 commits into8.14from
szaffarano/backport/8.14/pr_187859

Conversation

@szaffarano
Copy link
Contributor

@szaffarano szaffarano commented Jul 12, 2024

Backport

This will backport the following commits from main to 8.14:

Questions ?

Please refer to the Backport tool documentation

@szaffarano
[Telemetry][Security Solution] Use the proper index to query builtin …
cb3118c 
…alerts (#187859)

#177263 changed the way
`telemetry-prebuilt-rule-alerts` get data from elastic, but it changed
the index used to run the queries. This PR fixes it using the proper
index.

(cherry picked from commit a120c51)
@szaffarano szaffarano added backport This PR is a backport of another PR release_note:skip Skip the PR/issue when compiling release notes Team: SecuritySolution Security Solutions Team working on SIEM, Endpoint, Timeline, Resolver, etc. labels Jul 12, 2024
@elasticmachine
Copy link
Contributor

elasticmachine commented Jul 12, 2024

Pinging @elastic/security-solution (Team: SecuritySolution)

@szaffarano szaffarano requested review from a team and JDKurma July 12, 2024 15:10
dejadavi-el
dejadavi-el approved these changes Jul 12, 2024
View reviewed changes
Copy link
Contributor

@dejadavi-el dejadavi-el left a comment

Choose a reason for hiding this comment

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

✅ LGTM

@szaffarano
Copy link
Contributor Author

szaffarano commented Jul 15, 2024

@elasticmachine merge upstream

@szaffarano szaffarano enabled auto-merge (squash) July 15, 2024 07:53
@elasticmachine
Copy link
Contributor

elasticmachine commented Jul 15, 2024 •
edited
Loading

💔 Build Failed

Failed CI Steps

Metrics [docs]

Unknown metric groups

ESLint disabled line counts

id before after diff
securitySolution 515 516 +1

Total ESLint disabled count

id before after diff
securitySolution 592 593 +1

History

@szaffarano szaffarano merged commit 655f25e into 8.14 Jul 15, 2024
@szaffarano szaffarano deleted the szaffarano/backport/8.14/pr_187859 branch July 15, 2024 09:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@JDKurma JDKurma JDKurma approved these changes

@dejadavi-el dejadavi-el dejadavi-el approved these changes

Assignees

No one assigned

Labels

backport This PR is a backport of another PR release_note:skip Skip the PR/issue when compiling release notes Team: SecuritySolution Security Solutions Team working on SIEM, Endpoint, Timeline, Resolver, etc.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@szaffarano @elasticmachine @JDKurma @dejadavi-el