Skip to content
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

[Security Solution] Remove rule update/install callouts #160269

Merged
merged 1 commit into from
Jun 23, 2023

Conversation

xcrzx
Copy link
Contributor

@xcrzx xcrzx commented Jun 22, 2023

Summary

This PR removes the callouts displaying new or updated rules. Given that the callouts cannot be dismissed, we've decided to remove them temporarily to prevent UI clutter, as there may be other callouts on the page. We're going to reintroduce these callouts in 8.10 with the ability to dismiss them till the next detection rules package release.

Ticket to re-add the callouts in 8.10: #160270

@xcrzx xcrzx added release_note:skip Skip the PR/issue when compiling release notes Feature:Detection Rules Security Solution rules and Detection Engine Team:Detections and Resp Security Detection Response Team Team: SecuritySolution Security Solutions Team working on SIEM, Endpoint, Timeline, Resolver, etc. Team:Detection Rule Management Security Detection Rule Management Team v8.9.0 v8.10.0 labels Jun 22, 2023
@xcrzx xcrzx self-assigned this Jun 22, 2023
@xcrzx xcrzx requested a review from a team as a code owner June 22, 2023 13:14
@xcrzx xcrzx requested a review from maximpn June 22, 2023 13:14
@elasticmachine
Copy link
Contributor

Pinging @elastic/security-detections-response (Team:Detections and Resp)

@elasticmachine
Copy link
Contributor

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

@xcrzx xcrzx removed the v8.10.0 label Jun 22, 2023
@xcrzx xcrzx changed the title Remove rule update/install callouts [Security Solution] Remove rule update/install callouts Jun 22, 2023
@kibana-ci
Copy link
Collaborator

💚 Build Succeeded

Metrics [docs]

Module Count

Fewer modules leads to a faster build time

id before after diff
securitySolution 4213 4211 -2

Async chunks

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

id before after diff
securitySolution 11.0MB 11.0MB -2.6KB
Unknown metric groups

ESLint disabled line counts

id before after diff
enterpriseSearch 13 15 +2
securitySolution 416 420 +4
total +6

Total ESLint disabled count

id before after diff
enterpriseSearch 14 16 +2
securitySolution 497 501 +4
total +6

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

cc @xcrzx

@xcrzx xcrzx requested review from jpdjere and removed request for maximpn June 22, 2023 15:58
@kibanamachine
Copy link
Contributor

💚 All backports created successfully

Status Branch Result
8.9

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

Questions ?

Please refer to the Backport tool documentation

kibanamachine pushed a commit to kibanamachine/kibana that referenced this pull request Jun 23, 2023
## Summary

This PR removes the callouts displaying new or updated rules. Given that
the callouts cannot be dismissed, we've decided to remove them
temporarily to prevent UI clutter, as there may be other callouts on the
page. We're going to reintroduce these callouts in 8.10 with the ability
to dismiss them till the next detection rules package release.

Ticket to re-add the callouts in 8.10:
elastic#160270

(cherry picked from commit 4dfb7bc)
kibanamachine added a commit that referenced this pull request Jun 30, 2023
… (#160359)

# Backport

This will backport the following commits from `main` to `8.9`:
- [[Security Solution] Remove rule update/install callouts
(#160269)](#160269)

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

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

<!--BACKPORT [{"author":{"name":"Dmitrii
Shevchenko","email":"[email protected]"},"sourceCommit":{"committedDate":"2023-06-23T08:08:24Z","message":"[Security
Solution] Remove rule update/install callouts (#160269)\n\n##
Summary\r\n\r\nThis PR removes the callouts displaying new or updated
rules. Given that\r\nthe callouts cannot be dismissed, we've decided to
remove them\r\ntemporarily to prevent UI clutter, as there may be other
callouts on the\r\npage. We're going to reintroduce these callouts in
8.10 with the ability\r\nto dismiss them till the next detection rules
package release.\r\n\r\nTicket to re-add the callouts in
8.10:\r\nhttps://github.com//issues/160270","sha":"4dfb7bcdb9e021360d56364bc1eef142e6682c68","branchLabelMapping":{"^v8.10.0$":"main","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:skip","Feature:Detection
Rules","Team:Detections and Resp","Team:
SecuritySolution","Team:Detection Rule
Management","v8.9.0","v8.10.0"],"number":160269,"url":"https://github.com/elastic/kibana/pull/160269","mergeCommit":{"message":"[Security
Solution] Remove rule update/install callouts (#160269)\n\n##
Summary\r\n\r\nThis PR removes the callouts displaying new or updated
rules. Given that\r\nthe callouts cannot be dismissed, we've decided to
remove them\r\ntemporarily to prevent UI clutter, as there may be other
callouts on the\r\npage. We're going to reintroduce these callouts in
8.10 with the ability\r\nto dismiss them till the next detection rules
package release.\r\n\r\nTicket to re-add the callouts in
8.10:\r\nhttps://github.com//issues/160270","sha":"4dfb7bcdb9e021360d56364bc1eef142e6682c68"}},"sourceBranch":"main","suggestedTargetBranches":["8.9"],"targetPullRequestStates":[{"branch":"8.9","label":"v8.9.0","labelRegex":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"},{"branch":"main","label":"v8.10.0","labelRegex":"^v8.10.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/160269","number":160269,"mergeCommit":{"message":"[Security
Solution] Remove rule update/install callouts (#160269)\n\n##
Summary\r\n\r\nThis PR removes the callouts displaying new or updated
rules. Given that\r\nthe callouts cannot be dismissed, we've decided to
remove them\r\ntemporarily to prevent UI clutter, as there may be other
callouts on the\r\npage. We're going to reintroduce these callouts in
8.10 with the ability\r\nto dismiss them till the next detection rules
package release.\r\n\r\nTicket to re-add the callouts in
8.10:\r\nhttps://github.com//issues/160270","sha":"4dfb7bcdb9e021360d56364bc1eef142e6682c68"}}]}]
BACKPORT-->

Co-authored-by: Dmitrii Shevchenko <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature:Detection Rules Security Solution rules and Detection Engine release_note:skip Skip the PR/issue when compiling release notes Team:Detection Rule Management Security Detection Rule Management Team Team:Detections and Resp Security Detection Response Team Team: SecuritySolution Security Solutions Team working on SIEM, Endpoint, Timeline, Resolver, etc. v8.9.0 v8.10.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants