Skip to content

[ML] Adds new SIEM auditbeat, winlogbeat and packetbeat modules#47848

Merged
peteharverson merged 3 commits intoelastic:masterfrom
peteharverson:ml-siem-auth-packetbeat-modules
Oct 11, 2019
Merged

[ML] Adds new SIEM auditbeat, winlogbeat and packetbeat modules#47848
peteharverson merged 3 commits intoelastic:masterfrom
peteharverson:ml-siem-auth-packetbeat-modules

Conversation

@peteharverson
Copy link
Contributor

@peteharverson peteharverson commented Oct 10, 2019

Summary

Adds files for three new SIEM data recognizer modules for auditbeat, winlogbeat and packetbeat data:

  • SIEM Auditbeat Authentication (detecting suspicious authentication events in Auditbeat data)
  • SIEM Winlogbeat Authentication (detecting suspicious authentication events in Winlogbeat data)
  • SIEM Packetbeat (detecting suspicious network activity in Packetbeat data)

Also includes some edits to the descriptions and filters from the existing Auditbeat and Winlogbeat modules, removing the ecs suffix from the module IDs and folders.

image

image

image

Contains:

  • Module manifest.json
  • Module logo.json
  • ML Job configurations
  • ML Datafeed configurations

Checklist

For maintainers

@elasticmachine
Copy link
Contributor

Pinging @elastic/ml-ui (:ml)

@elasticmachine
Copy link
Contributor

💚 Build Succeeded

Copy link
Contributor

@blaklaybul blaklaybul left a comment

Choose a reason for hiding this comment

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

Looks good. However let's pull windows_rare_country_for_user and linux_rare_country_for_user for now. This detector has only been tested on okta data, not yet on auditbeat or winlogbeat. @cwurm began ingesting country info on our siem development cluster, but these jobs have yet to produce results there.

@elasticmachine
Copy link
Contributor

💚 Build Succeeded

Copy link
Member

@jgowdyelastic jgowdyelastic left a comment

Choose a reason for hiding this comment

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

LGTM

@peteharverson peteharverson merged commit cd735fd into elastic:master Oct 11, 2019
peteharverson added a commit to peteharverson/kibana that referenced this pull request Oct 11, 2019
…tic#47848)

* [ML] Adds new SIEM auditbeat, winlogbeat and packetbeat modules

* [ML] Removed rare_country_for_user jobs

* [ML] Removed rare_country_for_user jobs from manifests
peteharverson added a commit that referenced this pull request Oct 11, 2019
…) (#47945)

* [ML] Adds new SIEM auditbeat, winlogbeat and packetbeat modules

* [ML] Removed rare_country_for_user jobs

* [ML] Removed rare_country_for_user jobs from manifests
droberts195 pushed a commit that referenced this pull request Oct 15, 2019
This change augments the SIEM jobs and datafeeds that were
added in #47848 with the allow_lazy_open and max_empty_searches
options that were added in elastic/elasticsearch#47726 and
elastic/elasticsearch#47922 respectively.
spong added a commit that referenced this pull request Oct 15, 2019
## Summary

This PR updates the list of ML Modules that the SIEM App should display jobs from within the Anomaly Detection UI. ML PR: #47848

Modules include:
```
  'siem_auditbeat',
  'siem_auditbeat_auth',
  'siem_packetbeat',
  'siem_winlogbeat',
  'siem_winlogbeat_auth',
```

Anomaly Detection UI now shows `23` jobs in total:

![image](https://user-images.githubusercontent.com/2946766/66853948-a31ad900-ef3d-11e9-8be9-dc7224a0f975.png)



Summarize your PR. If it involves visual changes include a screenshot or gif.

### Checklist

Use ~~strikethroughs~~ to remove checklist items you don't feel are applicable to this PR.

- [ ] ~This was checked for cross-browser compatibility, [including a check against IE11](https://github.com/elastic/kibana/blob/master/CONTRIBUTING.md#cross-browser-compatibility)~
- [ ] ~Any text added follows [EUI's writing guidelines](https://elastic.github.io/eui/#/guidelines/writing), uses sentence case text and includes [i18n support](https://github.com/elastic/kibana/blob/master/packages/kbn-i18n/README.md)~
- [ ] [Documentation](https://github.com/elastic/kibana/blob/master/CONTRIBUTING.md#writing-documentation) was added for features that require explanation or tutorials
  * will work with @benskelker on latest ML doc updates
- [ ] ~[Unit or functional tests](https://github.com/elastic/kibana/blob/master/CONTRIBUTING.md#cross-browser-compatibility) were updated or added to match the most common scenarios~
- [ ] ~This was checked for [keyboard-only and screenreader accessibility](https://developer.mozilla.org/en-US/docs/Learn/Tools_and_testing/Cross_browser_testing/Accessibility#Accessibility_testing_checklist)~

### For maintainers

- [ ] ~This was checked for breaking API changes and was [labeled appropriately](https://github.com/elastic/kibana/blob/master/CONTRIBUTING.md#release-notes-process)~
- [ ] ~This includes a feature addition or change that requires a release note and was [labeled appropriately](https://github.com/elastic/kibana/blob/master/CONTRIBUTING.md#release-notes-process)~
spong added a commit to spong/kibana that referenced this pull request Oct 16, 2019
## Summary

This PR updates the list of ML Modules that the SIEM App should display jobs from within the Anomaly Detection UI. ML PR: elastic#47848

Modules include:
```
  'siem_auditbeat',
  'siem_auditbeat_auth',
  'siem_packetbeat',
  'siem_winlogbeat',
  'siem_winlogbeat_auth',
```

Anomaly Detection UI now shows `23` jobs in total:

![image](https://user-images.githubusercontent.com/2946766/66853948-a31ad900-ef3d-11e9-8be9-dc7224a0f975.png)



Summarize your PR. If it involves visual changes include a screenshot or gif.

### Checklist

Use ~~strikethroughs~~ to remove checklist items you don't feel are applicable to this PR.

- [ ] ~This was checked for cross-browser compatibility, [including a check against IE11](https://github.com/elastic/kibana/blob/master/CONTRIBUTING.md#cross-browser-compatibility)~
- [ ] ~Any text added follows [EUI's writing guidelines](https://elastic.github.io/eui/#/guidelines/writing), uses sentence case text and includes [i18n support](https://github.com/elastic/kibana/blob/master/packages/kbn-i18n/README.md)~
- [ ] [Documentation](https://github.com/elastic/kibana/blob/master/CONTRIBUTING.md#writing-documentation) was added for features that require explanation or tutorials
  * will work with @benskelker on latest ML doc updates
- [ ] ~[Unit or functional tests](https://github.com/elastic/kibana/blob/master/CONTRIBUTING.md#cross-browser-compatibility) were updated or added to match the most common scenarios~
- [ ] ~This was checked for [keyboard-only and screenreader accessibility](https://developer.mozilla.org/en-US/docs/Learn/Tools_and_testing/Cross_browser_testing/Accessibility#Accessibility_testing_checklist)~

### For maintainers

- [ ] ~This was checked for breaking API changes and was [labeled appropriately](https://github.com/elastic/kibana/blob/master/CONTRIBUTING.md#release-notes-process)~
- [ ] ~This includes a feature addition or change that requires a release note and was [labeled appropriately](https://github.com/elastic/kibana/blob/master/CONTRIBUTING.md#release-notes-process)~
spong added a commit that referenced this pull request Oct 16, 2019
## Summary

This PR updates the list of ML Modules that the SIEM App should display jobs from within the Anomaly Detection UI. ML PR: #47848

Modules include:
```
  'siem_auditbeat',
  'siem_auditbeat_auth',
  'siem_packetbeat',
  'siem_winlogbeat',
  'siem_winlogbeat_auth',
```

Anomaly Detection UI now shows `23` jobs in total:

![image](https://user-images.githubusercontent.com/2946766/66853948-a31ad900-ef3d-11e9-8be9-dc7224a0f975.png)



Summarize your PR. If it involves visual changes include a screenshot or gif.

### Checklist

Use ~~strikethroughs~~ to remove checklist items you don't feel are applicable to this PR.

- [ ] ~This was checked for cross-browser compatibility, [including a check against IE11](https://github.com/elastic/kibana/blob/master/CONTRIBUTING.md#cross-browser-compatibility)~
- [ ] ~Any text added follows [EUI's writing guidelines](https://elastic.github.io/eui/#/guidelines/writing), uses sentence case text and includes [i18n support](https://github.com/elastic/kibana/blob/master/packages/kbn-i18n/README.md)~
- [ ] [Documentation](https://github.com/elastic/kibana/blob/master/CONTRIBUTING.md#writing-documentation) was added for features that require explanation or tutorials
  * will work with @benskelker on latest ML doc updates
- [ ] ~[Unit or functional tests](https://github.com/elastic/kibana/blob/master/CONTRIBUTING.md#cross-browser-compatibility) were updated or added to match the most common scenarios~
- [ ] ~This was checked for [keyboard-only and screenreader accessibility](https://developer.mozilla.org/en-US/docs/Learn/Tools_and_testing/Cross_browser_testing/Accessibility#Accessibility_testing_checklist)~

### For maintainers

- [ ] ~This was checked for breaking API changes and was [labeled appropriately](https://github.com/elastic/kibana/blob/master/CONTRIBUTING.md#release-notes-process)~
- [ ] ~This includes a feature addition or change that requires a release note and was [labeled appropriately](https://github.com/elastic/kibana/blob/master/CONTRIBUTING.md#release-notes-process)~
droberts195 pushed a commit that referenced this pull request Oct 16, 2019
…#48372)

This change augments the SIEM jobs and datafeeds that were
added in #47848 with the allow_lazy_open and max_empty_searches
options that were added in elastic/elasticsearch#47726 and
elastic/elasticsearch#47922 respectively.
@elasticmachine
Copy link
Contributor

Pinging @elastic/siem (Team:SIEM)

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants