-
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
Sustainable Kibana Architecture: Move modules owned by @elastic/security-solution
#202851
base: main
Are you sure you want to change the base?
Sustainable Kibana Architecture: Move modules owned by @elastic/security-solution
#202851
Conversation
A documentation preview will be available soon. Request a new doc build by commenting
If your PR continues to fail for an unknown reason, the doc build pipeline may be broken. Elastic employees can check the pipeline status here. |
Pinging @elastic/kibana-core (Team:Core) |
Pinging @elastic/security-solution (Team: 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.
Finally got a successful build, almost there! LGTM
5b9a02a
to
5152866
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.
Security Detection Rule Management changes LGTM 👍
- The folder structure looks good
- The app runs locally without issues, the features we own work
- The API integration tests we own run locally
- The OpenAPI code generation works locally and is properly integrated with CI
- The OpenAPI specs bundling works locally and is properly integrated with CI
Should be good to merge and fix leftover issues separately:
- Sustainable Kibana Architecture: Move modules owned by
@elastic/security-solution
#202851 (comment) - Paths in the CODEOWNERS file
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
9e88c52
to
c5c64be
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.
Fleet changes 🚀
I’ve noticed that some Also there are issues with reporters in Cypress tests - probably just wrong paths to them in cypress config files. . |
💛 Build succeeded, but was flaky
Failed CI StepsTest Failures
Metrics [docs]
History
|
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 for the Threat Hunting Investigations team.
Code change looks good and I desk tested the following:
- timeline, alerts and intelligence pages
- interaction with the Cases plugin
- expandable flyout
- notes
Looking at Buildkite, all the Jest and Cypress tests seem to all be running.
I noticed a issue related to Storybook where the Threat Intelligence plugin seems to be bundled under yarn storybook security_solution_packages
but that can be fixed in a follow up PR!
Summary
This PR aims at relocating some of the Kibana modules (plugins and packages) into a new folder structure, according to the Sustainable Kibana Architecture initiative.
Important
packages[\/\\]
andplugins[\/\\]
patterns in the source code (Babel and Eslint config files), and update them appropriately..buildkite/scripts/pipelines/pull_request/pipeline.ts
to ensure that any CI pipeline customizations continue to be correctly applied after the changed path names.ts
and.js
files listed in the sections below, as some of them contain relative paths that have been updated.Note
3 plugin(s) are going to be relocated:
@kbn/security-solution-ess
x-pack/solutions/security/plugins/security_solution_ess
@kbn/security-solution-plugin
x-pack/solutions/security/plugins/security_solution
@kbn/security-solution-serverless
x-pack/solutions/security/plugins/security_solution_serverless
2 packages(s) are going to be relocated:
@kbn/deeplinks-security
src/platform/packages/shared/deeplinks/security
@kbn/serverless-security-settings
src/platform/packages/shared/serverless/settings/security_project
Script errors
Updated references
Updated relative paths