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] Assistant package + plugin dead code removal #200235

Merged
merged 16 commits into from
Nov 18, 2024

Conversation

stephmilovic
Copy link
Contributor

@stephmilovic stephmilovic commented Nov 14, 2024

Summary

Removes dead code from:

  • x-pack/packages/kbn-elastic-assistant
  • x-pack/packages/kbn-elastic-assistant-common
  • x-pack/packages/kbn-langchain
  • x-pack/plugins/elastic_assistant
  • x-pack/plugins/security_solution/public/assistant
  • x-pack/plugins/security_solution/public/attack_discovery
  • x-pack/plugins/security_solution/server/assistant

@stephmilovic stephmilovic added release_note:skip Skip the PR/issue when compiling release notes v9.0.0 Team: SecuritySolution Security Solutions Team working on SIEM, Endpoint, Timeline, Resolver, etc. Team:Security Generative AI Security Generative AI backport:version Backport to applied version labels v8.17.0 labels Nov 14, 2024
@stephmilovic stephmilovic marked this pull request as ready for review November 18, 2024 15:56
@stephmilovic stephmilovic requested review from a team as code owners November 18, 2024 15:56
@elasticmachine
Copy link
Contributor

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

@elasticmachine
Copy link
Contributor

💚 Build Succeeded

Metrics [docs]

Public APIs missing comments

Total count of every public API that lacks a comment. Target amount is 0. Run node scripts/build_api_docs --plugin [yourplugin] --stats comments for more detailed information.

id before after diff
@kbn/elastic-assistant-common 370 369 -1

Async chunks

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

id before after diff
securitySolution 13.4MB 13.4MB -2.4KB
Unknown metric groups

API count

id before after diff
@kbn/elastic-assistant-common 401 400 -1

History

@stephmilovic stephmilovic merged commit cc38d8d into elastic:main Nov 18, 2024
43 checks passed
@kibanamachine
Copy link
Contributor

Starting backport for target branches: 8.x

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

@kibanamachine
Copy link
Contributor

💔 All backports failed

Status Branch Result
8.x Backport failed because of merge conflicts

Manual backport

To create the backport manually run:

node scripts/backport --pr 200235

Questions ?

Please refer to the Backport tool documentation

@stephmilovic
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

stephmilovic added a commit to stephmilovic/kibana that referenced this pull request Nov 18, 2024
…stic#200235)

(cherry picked from commit cc38d8d)

# Conflicts:
#	x-pack/plugins/translations/translations/zh-CN.json
stephmilovic added a commit that referenced this pull request Nov 18, 2024
#200235) (#200630)

# Backport

This will backport the following commits from `main` to `8.x`:
- [[Security solution] Assistant package + plugin dead code removal
(#200235)](#200235)

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

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

<!--BACKPORT [{"author":{"name":"Steph
Milovic","email":"[email protected]"},"sourceCommit":{"committedDate":"2024-11-18T18:17:00Z","message":"[Security
solution] Assistant package + plugin dead code removal
(#200235)","sha":"cc38d8d03b664ae43d05e26c3c08ba68998e4348","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:
SecuritySolution","Team:Security Generative
AI","backport:version","v8.17.0"],"number":200235,"url":"https://github.com/elastic/kibana/pull/200235","mergeCommit":{"message":"[Security
solution] Assistant package + plugin dead code removal
(#200235)","sha":"cc38d8d03b664ae43d05e26c3c08ba68998e4348"}},"sourceBranch":"main","suggestedTargetBranches":["8.x"],"targetPullRequestStates":[{"branch":"main","label":"v9.0.0","labelRegex":"^v9.0.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/200235","number":200235,"mergeCommit":{"message":"[Security
solution] Assistant package + plugin dead code removal
(#200235)","sha":"cc38d8d03b664ae43d05e26c3c08ba68998e4348"}},{"branch":"8.x","label":"v8.17.0","labelRegex":"^v8.17.0$","isSourceBranch":false,"state":"NOT_CREATED"}]}]
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 release_note:skip Skip the PR/issue when compiling release notes Team:Security Generative AI Security Generative AI Team: SecuritySolution Security Solutions Team working on SIEM, Endpoint, Timeline, Resolver, etc. v8.17.0 v9.0.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants