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] Removes outdated rule tour for rule management workflow #196731

Merged
merged 3 commits into from
Oct 21, 2024

Conversation

dplumlee
Copy link
Contributor

@dplumlee dplumlee commented Oct 17, 2024

Summary

Removes the tour for rule features implemented in #176767 as they are no longer new. Keeps the component and logic to use for future tours, just removes reference to the specific rules management tour.

Steps to test

  1. Make sure the securitySolution.rulesManagementPage.newFeaturesTour.v8.13 local storage key is cleared from your kibana page (this will make the tour show up normally)
  2. Notice that navigating to the rule management page does not display the rule tour

@dplumlee dplumlee added release_note:skip Skip the PR/issue when compiling release notes v9.0.0 Team:Detections and Resp Security Detection Response Team Team: SecuritySolution Security Solutions Team working on SIEM, Endpoint, Timeline, Resolver, etc. Feature:Rule Management Security Solution Detection Rule Management area Team:Detection Rule Management Security Detection Rule Management Team v8.16.0 backport:version Backport to applied version labels labels Oct 17, 2024
@dplumlee dplumlee self-assigned this Oct 17, 2024
@dplumlee dplumlee requested a review from a team as a code owner October 17, 2024 15:34
@dplumlee dplumlee requested a review from maximpn October 17, 2024 15:34
@elasticmachine
Copy link
Contributor

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

@elasticmachine
Copy link
Contributor

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

@elasticmachine
Copy link
Contributor

Pinging @elastic/security-detection-rule-management (Team:Detection Rule Management)

@dplumlee
Copy link
Contributor Author

@elasticmachine merge upstream

@banderror
Copy link
Contributor

@elasticmachine merge upstream

Copy link
Contributor

@banderror banderror left a comment

Choose a reason for hiding this comment

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

Thanks @dplumlee

@elasticmachine
Copy link
Contributor

💚 Build Succeeded

Metrics [docs]

Module Count

Fewer modules leads to a faster build time

id before after diff
securitySolution 6037 6035 -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 20.7MB 20.7MB -2.1KB

History

cc @dplumlee

@dplumlee dplumlee merged commit 3547e15 into elastic:main Oct 21, 2024
47 checks passed
@kibanamachine
Copy link
Contributor

Starting backport for target branches: 8.16

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

@kibanamachine
Copy link
Contributor

💔 All backports failed

Status Branch Result
8.16 Backport failed because of merge conflicts

Manual backport

To create the backport manually run:

node scripts/backport --pr 196731

Questions ?

Please refer to the Backport tool documentation

@dplumlee
Copy link
Contributor Author

💚 All backports created successfully

Status Branch Result
8.16

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

Questions ?

Please refer to the Backport tool documentation

@dplumlee dplumlee deleted the remove-rule-tour branch October 21, 2024 14:50
dplumlee added a commit to dplumlee/kibana that referenced this pull request Oct 21, 2024
…rkflow (elastic#196731)

## Summary

Removes the tour for rule features implemented in
elastic#176767 as they are no longer new.
Keeps the component and logic to use for future tours, just removes
reference to the specific rules management tour.

### Steps to test

1. Make sure the
`securitySolution.rulesManagementPage.newFeaturesTour.v8.13` local
storage key is cleared from your kibana page (this will make the tour
show up normally)
2. Notice that navigating to the rule management page does not display
the rule tour

Co-authored-by: Elastic Machine <[email protected]>
(cherry picked from commit 3547e15)

# Conflicts:
#	x-pack/plugins/security_solution/public/detection_engine/rule_management_ui/pages/rule_management/index.tsx
@dplumlee
Copy link
Contributor Author

💚 All backports created successfully

Status Branch Result
8.x

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

Questions ?

Please refer to the Backport tool documentation

dplumlee added a commit to dplumlee/kibana that referenced this pull request Oct 21, 2024
…rkflow (elastic#196731)

## Summary

Removes the tour for rule features implemented in
elastic#176767 as they are no longer new.
Keeps the component and logic to use for future tours, just removes
reference to the specific rules management tour.

### Steps to test

1. Make sure the
`securitySolution.rulesManagementPage.newFeaturesTour.v8.13` local
storage key is cleared from your kibana page (this will make the tour
show up normally)
2. Notice that navigating to the rule management page does not display
the rule tour

Co-authored-by: Elastic Machine <[email protected]>
(cherry picked from commit 3547e15)

# Conflicts:
#	x-pack/plugins/security_solution/public/detection_engine/rule_management_ui/pages/rule_management/index.tsx
dplumlee added a commit that referenced this pull request Oct 21, 2024
…ment workflow (#196731) (#197080)

# Backport

This will backport the following commits from `main` to `8.16`:
- [[Security Solution] Removes outdated rule tour for rule management
workflow (#196731)](#196731)

<!--- Backport version: 8.9.8 -->

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

<!--BACKPORT [{"author":{"name":"Davis
Plumlee","email":"[email protected]"},"sourceCommit":{"committedDate":"2024-10-21T14:33:26Z","message":"[Security
Solution] Removes outdated rule tour for rule management workflow
(#196731)\n\n## Summary\r\n\r\nRemoves the tour for rule features
implemented in\r\nhttps://github.com//pull/176767 as they
are no longer new.\r\nKeeps the component and logic to use for future
tours, just removes\r\nreference to the specific rules management
tour.\r\n\r\n### Steps to test\r\n\r\n1. Make sure
the\r\n`securitySolution.rulesManagementPage.newFeaturesTour.v8.13`
local\r\nstorage key is cleared from your kibana page (this will make
the tour\r\nshow up normally)\r\n2. Notice that navigating to the rule
management page does not display\r\nthe rule tour\r\n\r\nCo-authored-by:
Elastic Machine
<[email protected]>","sha":"3547e154b57adbdbf36abda5c4571452b4ee0b26","branchLabelMapping":{"^v9.0.0$":"main","^v8.17.0$":"8.x","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:skip","v9.0.0","Team:Detections
and Resp","Team: SecuritySolution","Feature:Rule
Management","Team:Detection Rule
Management","v8.16.0","backport:version"],"number":196731,"url":"https://github.com/elastic/kibana/pull/196731","mergeCommit":{"message":"[Security
Solution] Removes outdated rule tour for rule management workflow
(#196731)\n\n## Summary\r\n\r\nRemoves the tour for rule features
implemented in\r\nhttps://github.com//pull/176767 as they
are no longer new.\r\nKeeps the component and logic to use for future
tours, just removes\r\nreference to the specific rules management
tour.\r\n\r\n### Steps to test\r\n\r\n1. Make sure
the\r\n`securitySolution.rulesManagementPage.newFeaturesTour.v8.13`
local\r\nstorage key is cleared from your kibana page (this will make
the tour\r\nshow up normally)\r\n2. Notice that navigating to the rule
management page does not display\r\nthe rule tour\r\n\r\nCo-authored-by:
Elastic Machine
<[email protected]>","sha":"3547e154b57adbdbf36abda5c4571452b4ee0b26"}},"sourceBranch":"main","suggestedTargetBranches":["8.16"],"targetPullRequestStates":[{"branch":"main","label":"v9.0.0","labelRegex":"^v9.0.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/196731","number":196731,"mergeCommit":{"message":"[Security
Solution] Removes outdated rule tour for rule management workflow
(#196731)\n\n## Summary\r\n\r\nRemoves the tour for rule features
implemented in\r\nhttps://github.com//pull/176767 as they
are no longer new.\r\nKeeps the component and logic to use for future
tours, just removes\r\nreference to the specific rules management
tour.\r\n\r\n### Steps to test\r\n\r\n1. Make sure
the\r\n`securitySolution.rulesManagementPage.newFeaturesTour.v8.13`
local\r\nstorage key is cleared from your kibana page (this will make
the tour\r\nshow up normally)\r\n2. Notice that navigating to the rule
management page does not display\r\nthe rule tour\r\n\r\nCo-authored-by:
Elastic Machine
<[email protected]>","sha":"3547e154b57adbdbf36abda5c4571452b4ee0b26"}},{"branch":"8.16","label":"v8.16.0","labelRegex":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"}]}]
BACKPORT-->
dplumlee added a commit that referenced this pull request Oct 21, 2024
…ent workflow (#196731) (#197086)

# Backport

This will backport the following commits from `main` to `8.x`:
- [[Security Solution] Removes outdated rule tour for rule management
workflow (#196731)](#196731)

<!--- Backport version: 8.9.8 -->

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

<!--BACKPORT [{"author":{"name":"Davis
Plumlee","email":"[email protected]"},"sourceCommit":{"committedDate":"2024-10-21T14:33:26Z","message":"[Security
Solution] Removes outdated rule tour for rule management workflow
(#196731)\n\n## Summary\r\n\r\nRemoves the tour for rule features
implemented in\r\nhttps://github.com//pull/176767 as they
are no longer new.\r\nKeeps the component and logic to use for future
tours, just removes\r\nreference to the specific rules management
tour.\r\n\r\n### Steps to test\r\n\r\n1. Make sure
the\r\n`securitySolution.rulesManagementPage.newFeaturesTour.v8.13`
local\r\nstorage key is cleared from your kibana page (this will make
the tour\r\nshow up normally)\r\n2. Notice that navigating to the rule
management page does not display\r\nthe rule tour\r\n\r\nCo-authored-by:
Elastic Machine
<[email protected]>","sha":"3547e154b57adbdbf36abda5c4571452b4ee0b26","branchLabelMapping":{"^v9.0.0$":"main","^v8.17.0$":"8.x","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:skip","v9.0.0","Team:Detections
and Resp","Team: SecuritySolution","Feature:Rule
Management","Team:Detection Rule
Management","v8.16.0","backport:version"],"number":196731,"url":"https://github.com/elastic/kibana/pull/196731","mergeCommit":{"message":"[Security
Solution] Removes outdated rule tour for rule management workflow
(#196731)\n\n## Summary\r\n\r\nRemoves the tour for rule features
implemented in\r\nhttps://github.com//pull/176767 as they
are no longer new.\r\nKeeps the component and logic to use for future
tours, just removes\r\nreference to the specific rules management
tour.\r\n\r\n### Steps to test\r\n\r\n1. Make sure
the\r\n`securitySolution.rulesManagementPage.newFeaturesTour.v8.13`
local\r\nstorage key is cleared from your kibana page (this will make
the tour\r\nshow up normally)\r\n2. Notice that navigating to the rule
management page does not display\r\nthe rule tour\r\n\r\nCo-authored-by:
Elastic Machine
<[email protected]>","sha":"3547e154b57adbdbf36abda5c4571452b4ee0b26"}},"sourceBranch":"main","suggestedTargetBranches":[],"targetPullRequestStates":[{"branch":"main","label":"v9.0.0","labelRegex":"^v9.0.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/196731","number":196731,"mergeCommit":{"message":"[Security
Solution] Removes outdated rule tour for rule management workflow
(#196731)\n\n## Summary\r\n\r\nRemoves the tour for rule features
implemented in\r\nhttps://github.com//pull/176767 as they
are no longer new.\r\nKeeps the component and logic to use for future
tours, just removes\r\nreference to the specific rules management
tour.\r\n\r\n### Steps to test\r\n\r\n1. Make sure
the\r\n`securitySolution.rulesManagementPage.newFeaturesTour.v8.13`
local\r\nstorage key is cleared from your kibana page (this will make
the tour\r\nshow up normally)\r\n2. Notice that navigating to the rule
management page does not display\r\nthe rule tour\r\n\r\nCo-authored-by:
Elastic Machine
<[email protected]>","sha":"3547e154b57adbdbf36abda5c4571452b4ee0b26"}},{"branch":"8.16","label":"v8.16.0","labelRegex":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"url":"https://github.com/elastic/kibana/pull/197080","number":197080,"state":"OPEN"}]}]
BACKPORT-->
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport:version Backport to applied version labels Feature:Rule Management Security Solution Detection Rule Management area 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.16.0 v8.17.0 v9.0.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants