Skip to content

[Security Solution] Cleanup graphql #78403

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 5 commits into from
Sep 24, 2020
Merged

[Security Solution] Cleanup graphql #78403

merged 5 commits into from
Sep 24, 2020

Conversation

@patrykkopycinski
Copy link
Contributor

@patrykkopycinski patrykkopycinski commented Sep 24, 2020 •
edited
Loading

Summary

This PR cleans up the logic that has been moved to the Search Strategy
#76928
#77205
#77410
#77606
#76603
#76252
#76250
#76249
#76244
#76243
#76241
#75591

Checklist

@patrykkopycinski patrykkopycinski added Team:SIEM v8.0.0 release_note:skip Skip the PR/issue when compiling release notes v7.10.0 labels Sep 24, 2020
@patrykkopycinski patrykkopycinski self-assigned this Sep 24, 2020
@patrykkopycinski patrykkopycinski marked this pull request as ready for review September 24, 2020 10:55
@patrykkopycinski patrykkopycinski requested review from a team as code owners September 24, 2020 10:55
@elasticmachine
Copy link
Contributor

elasticmachine commented Sep 24, 2020

Pinging @elastic/siem (Team:SIEM)

@patrykkopycinski
Copy link
Contributor Author

patrykkopycinski commented Sep 24, 2020

@elasticmachine merge upstream

@kibanamachine
Copy link
Contributor

kibanamachine commented Sep 24, 2020 •
edited
Loading

💔 Build Failed

Failed CI Steps

Metrics [docs]

@kbn/optimizer bundle module count

id value diff baseline
securitySolution 1970 -1 1971

async chunks size

id value diff baseline
securitySolution 10.1MB -25.7KB 10.2MB

page load bundle size

id value diff baseline
securitySolution 806.3KB -4.0KB 810.3KB

distributable file count

id value diff baseline
default 45810 -68 45878

History

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

@patrykkopycinski patrykkopycinski merged commit 179c7d9 into elastic:master Sep 24, 2020
@patrykkopycinski patrykkopycinski deleted the chore/rest-graphql-cleanup branch September 24, 2020 16:23
@patrykkopycinski patrykkopycinski mentioned this pull request Sep 24, 2020
Merged
patrykkopycinski added a commit to patrykkopycinski/kibana that referenced this pull request Sep 24, 2020
@patrykkopycinski @elasticmachine
[Security Solution] Cleanup graphql (elastic#78403)
24b733b 
* [Security Solution] Cleanup graphql

* cleanup

* cleanup

* fix types

Co-authored-by: Elastic Machine <[email protected]>
patrykkopycinski added a commit that referenced this pull request Sep 25, 2020
@patrykkopycinski
[Security Solution] Cleanup graphql (#78403) (#78489)
de8887a
@MindyRS MindyRS added the Team: SecuritySolution Security Solutions Team working on SIEM, Endpoint, Timeline, Resolver, etc. label Sep 23, 2021
@elasticmachine
Copy link
Contributor

elasticmachine commented Sep 23, 2021

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@angorayc angorayc angorayc approved these changes

Assignees

@patrykkopycinski patrykkopycinski

Labels

release_note:skip Skip the PR/issue when compiling release notes Team: SecuritySolution Security Solutions Team working on SIEM, Endpoint, Timeline, Resolver, etc. Team:SIEM v7.10.0 v8.0.0

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@patrykkopycinski @elasticmachine @kibanamachine @angorayc @MindyRS