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] Adding aggregations for endpoint events #72705

Merged

Conversation

jonathan-buttner
Copy link
Contributor

@jonathan-buttner jonathan-buttner commented Jul 21, 2020

Summary

This PR adds support for aggregating totals for endpoint data and endgame data. We should probably do some cleanup on the variable names to switch them over to endpoint or some combination of endpoint and endgame. This is a bug fix so I have not done the refactoring in this PR.

Endpoint only data
image

Endgame Data
image

Both Endpoint and Endgame Data
image

@jonathan-buttner jonathan-buttner added Feature:Endpoint Elastic Endpoint feature release_note:skip Skip the PR/issue when compiling release notes Team:Endpoint Data Visibility Team managing the endpoint resolver v7.9.0 v8.0.0 labels Jul 21, 2020
bool: {
should: [
{
term: {
Copy link
Contributor Author

Choose a reason for hiding this comment

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

We're treating library and driver events as image load.

bool: {
filter: [
{
bool: {
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Network is anything but dns

should: [
{
bool: {
filter: [
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Security events for endpoint should be an array of two values for event.category == [session, authentication]

@kibanamachine
Copy link
Contributor

💚 Build Succeeded

Build metrics

✅ unchanged

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

@michaelolo24
Copy link
Contributor

Can you make an issue for cleaning up the names that we can come back to later on?

@jonathan-buttner jonathan-buttner marked this pull request as ready for review July 22, 2020 14:25
@jonathan-buttner jonathan-buttner requested review from a team as code owners July 22, 2020 14:25
@elasticmachine
Copy link
Contributor

Pinging @elastic/endpoint-app-team (Feature:Endpoint)

@elasticmachine
Copy link
Contributor

Pinging @elastic/endpoint-data-visibility-team (Team:Endpoint Data Visibility)

@jonathan-buttner jonathan-buttner merged commit b346253 into elastic:master Jul 22, 2020
@jonathan-buttner jonathan-buttner deleted the include-endpoint-totals branch July 22, 2020 15:32
jonathan-buttner added a commit to jonathan-buttner/kibana that referenced this pull request Jul 22, 2020
jonathan-buttner added a commit to jonathan-buttner/kibana that referenced this pull request Jul 22, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature:Endpoint Elastic Endpoint feature release_note:skip Skip the PR/issue when compiling release notes Team:Endpoint Data Visibility Team managing the endpoint resolver v7.9.0 v8.0.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants