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

Ingest structured ES slow logs #10447

Merged
merged 3 commits into from
Feb 1, 2019

Conversation

ycombinator
Copy link
Contributor

This PR teaches the elasticsearch/slowlog fileset to ingest structured Elasticsearch search and indexing slow logs.

This PR takes the same approach as #10352, in that it creates an entrypoint pipeline, pipeline.json, that delegates further processing of a log entry depending on what it sees as the first character of the entry:

  • If the first character is {, it assumes the log line is structured as JSON and delegates further processing to pipeline-json.json.
  • Else, it assumes the log line is plaintext and delegates further processing to pipeline-plaintext.json.

@elasticmachine
Copy link
Collaborator

Pinging @elastic/stack-monitoring

@ycombinator ycombinator force-pushed the fb-es-slowlog-structured branch from 1b9b1a3 to 35abb5f Compare January 31, 2019 02:01
@cachedout
Copy link
Contributor

Looking good so far!

@ycombinator ycombinator force-pushed the fb-es-slowlog-structured branch 2 times, most recently from b8f6196 to 874a9cf Compare January 31, 2019 15:15
@ycombinator ycombinator changed the title [WIP] Ingest structured ES slow logs Ingest structured ES slow logs Feb 1, 2019
@ycombinator ycombinator added review and removed in progress Pull request is currently in progress. labels Feb 1, 2019
}
},
{
"dot_expander": {
Copy link
Contributor

Choose a reason for hiding this comment

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

Kind of odd that we need 2 steps here, but well.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Agreed. This will help: elastic/elasticsearch#36950.

@@ -6,7 +6,7 @@ paths:
exclude_files: [".gz$"]

multiline:
pattern: '^\[?[0-9]{4}-[0-9]{2}-[0-9]{2}'
pattern: '^(\[?[0-9]{4}-[0-9]{2}-[0-9]{2}|{)'
Copy link
Contributor

Choose a reason for hiding this comment

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

This is kind of interesting. In the event for json it's not a multiline anymore but here we still need the multiline config 🤔

Good for now, jus thought I mention it.

@ycombinator ycombinator force-pushed the fb-es-slowlog-structured branch from 874a9cf to 7d34e86 Compare February 1, 2019 12:53
@ycombinator ycombinator force-pushed the fb-es-slowlog-structured branch from 7d34e86 to 6a6b3fa Compare February 1, 2019 20:29
@ycombinator ycombinator merged commit b3f0c31 into elastic:master Feb 1, 2019
@ycombinator ycombinator deleted the fb-es-slowlog-structured branch February 1, 2019 21:28
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.

4 participants