[ML] Adds ML jobs for access logs to Nginx package#912
Merged
peteharverson merged 2 commits intoelastic:masterfrom Apr 19, 2021
Merged
[ML] Adds ML jobs for access logs to Nginx package#912peteharverson merged 2 commits intoelastic:masterfrom
peteharverson merged 2 commits intoelastic:masterfrom
Conversation
💚 Build Succeeded
Expand to view the summary
Build stats
Test stats 🧪
Trends 🧪 |
mtojek
reviewed
Apr 14, 2021
Contributor
mtojek
left a comment
There was a problem hiding this comment.
Please rebase it against master as I pushed fix for the missing spec (ML modules).
b7abf63 to
b6c1900
Compare
mtojek
approved these changes
Apr 14, 2021
Contributor
|
I'm really exited to see this. Have been waiting for this moment quite some time: elastic/package-spec#30 ! |
jgowdyelastic
approved these changes
Apr 15, 2021
b6c1900 to
758710b
Compare
6 tasks
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What does this PR do?
Adds an ML module containing anomaly detection jobs for finding unusual activity in HTTP access logs to the Nginx integration. Requires Kibana 7.13.0 or later.
These are the same five jobs that have previously been stored inside the ML Kibana plugin:
Some minor edits have been made to the previous job configurations stored in the ML Kibana plugin:
ngix_data_streamcompared tonginx_ecsfor the legacy moduledata_stream.dataset: nginx.accesscompared toevent.dataset: nginx.accessfor the legacy moduleNginx logs overviewdashboard which is already included in the Nginx package.(ECS)has been removed from the module and job descriptions_nginxis appended to the IDs of the jobs in the modulecreated_byproperty used for telemetry is set toml-module-nginx-access-data-streamcompared toml-module-nginx-accessfor the legacy moduleChecklist
changelog.ymlfile.How to test this PR locally
To test this PR:
logs-*, matching the query in the ML module JSON file:Go to the ML plugin in Kibana, and create a job, selecting the appropriate index (such as

logs-*) and select the card for this new Nginx access logs module:Create and run the jobs from the ML job wizard
Test that the custom URLs to the Nginx logs overview dashboard and the raw data in Discover work
Related issues
elastic/package-spec#148
Screenshots
ML module is now listed in the Kibana assets section for the Nginx package:

List of Nginx jobs in the ML Job list:

Screenshot showing results of Nginx ML jobs in the ML Anomaly Explorer:
