-
Notifications
You must be signed in to change notification settings - Fork 8.3k
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] Rename siem plugin to security_solution #67902
Conversation
Pinging @elastic/endpoint-response (Team:Endpoint Response) |
Pinging @elastic/endpoint-management (Team:Endpoint Management) |
Pinging @elastic/endpoint-data-visibility-team (Team:Endpoint Data Visibility) |
Pinging @elastic/siem (Team:SIEM) |
ffe38be
to
68e4f1d
Compare
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.
Operations: Jenkins, eslint, codeowners, storybook, and ts project configs all LGTM
@@ -46,8 +46,8 @@ const names: Record<string, string> = { | |||
search: i18n.translate('advancedSettings.categoryNames.searchLabel', { | |||
defaultMessage: 'Search', | |||
}), | |||
siem: i18n.translate('advancedSettings.categoryNames.siemLabel', { | |||
defaultMessage: 'SIEM', | |||
securitySolution: i18n.translate('advancedSettings.categoryNames.securitySolutionLabel', { |
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.
I'm not certain you will want to change the securitySolution
category key here, since x-pack/plugins/security_solution/server/ui_settings.ts
is still using siem
as the category name when registering settings, and I believe that's how we find the right category here.
It might be safer to leave the key as siem
since it isn't surfaced in the UI, and just keep the new defaultMessage
. (Or change all of the uiSettings you register to use the new category)
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.
I think I will prefer to change it everywhere and avoid a mismatch of stuff
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.
@lukeelmers Thank you so much for finding it!!!
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.
After talking with some good folks, We will need to create a follow up to migrate our siem settings to securitySolution
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.
app arch changes lgtm
It is already broken in master let's create an issue to fix it. |
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 on Cases!
Checked out locally and manually tested the following:
- all cases table and filters
- create case
- create case, no default connector
- create new connector
- create case, set default connector
- case comment
- delete case
- close case
- case updates
- push case to 3rd party
Thanks for the biggest SIEM security_solution PR ever @XavierM
💚 Build SucceededHistory
To update your PR or re-run it, just comment with: |
* master: (26 commits) [Console]remove completion for type for filter queries and aggs (elastic#68103) [ML] Transforms: Filter aggregation support (elastic#67591) [ES UI Shared] Monaco XJSON (elastic#67485) [Index Management] Add data streams functionality to indices tab (elastic#67940) [Discover] Fix renaming of saved search not displayed in breadcrumb (elastic#67577) [SECURITY] Rename siem plugin to security_solution (elastic#67902) [Uptime] Fix Telemetry Api flaky test (elastic#67358) [Data plugin] Add configuration property to enable / disable autocomplete (elastic#67847) remove scripts. prettire update has been done (elastic#68130) Closes elastic#68055 by detecting the local Kibana version and using that as (elastic#68198) [apm] docs: add deployment annotation example (elastic#67408) [ML] Extend population preview chart to show actual and typical value (elastic#67569) Refactor index management client integration tests for scalability (elastic#67917) Add generator function that creates multiple alerts (elastic#67713) chore(NA): remove config arg from os packages (elastic#67871) [Reporting] Move code out of Legacy (elastic#67904) [Metrics UI] Add overrides to Snapshot API to support alert previews (elastic#68125) [Security] [Cases] Manage timeline UI API (elastic#67719) [ENDPOINT][INGEST]Task/endpoint ingest update (elastic#67234) Fix code coverage for jest, upload merged reports (elastic#68149) ...
* rename siem to security_solution * rename siem to security solution inside of code * rename translation keys * fix snapshot * replace siem for security solution in tutorial * missing translation to be renamed * fix types for api test integration * updates runner file to match the new path * change category for kibana settings * miss renaming in advance settings * fixes cypress tests * fix api integration test * fix new translation * fix unit test * update translation i18n * update translation i18n II Co-authored-by: Gloria Hornero <[email protected]>
* rename siem to security_solution * rename siem to security solution inside of code * rename translation keys * fix snapshot * replace siem for security solution in tutorial * missing translation to be renamed * fix types for api test integration * updates runner file to match the new path * change category for kibana settings * miss renaming in advance settings * fixes cypress tests * fix api integration test * fix new translation * fix unit test * update translation i18n * update translation i18n II Co-authored-by: Gloria Hornero <[email protected]> Co-authored-by: Gloria Hornero <[email protected]>
Pinging @elastic/security-solution (Team: SecuritySolution) |
Summary
Checklist