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

New Filebeat module: Kafka logs #4885

Merged
merged 1 commit into from
Aug 16, 2017

Conversation

kvch
Copy link
Contributor

@kvch kvch commented Aug 11, 2017

This module is based on https://www.elastic.co/blog/monitoring-kafka-with-elastic-stack-1-filebeat written by @urso.

This PR only includes the log fileset.

@kvch kvch added Filebeat Filebeat review labels Aug 11, 2017
@kvch kvch force-pushed the feature/filebeat-kafka-module-log branch from 450129a to b17ed4b Compare August 11, 2017 18:21
@@ -0,0 +1,14 @@
module_version: 1.0

kafka.home: /opt/kafka[_-\.0-9]*
Copy link
Contributor

Choose a reason for hiding this comment

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

I think this could be made also a var, which can be referred from the paths array. This way, the user can only set that variable. Unless I missremember, the variables can refer to variables defined before them using the syntax {{.home}}.

@tsg
Copy link
Contributor

tsg commented Aug 14, 2017

Nice work so far, could you also add a screenshot of the Kibana dashboard?

@kvch kvch force-pushed the feature/filebeat-kafka-module-log branch from 19cfa17 to a45d829 Compare August 14, 2017 14:23
@kvch
Copy link
Contributor Author

kvch commented Aug 14, 2017

I added the Kibana dashboard along with some missing documentation. I also made kafka_home configurable. Please, check if I did the right thing. :)

And for the record the GC logs will come in a separate PR. So it can be merged, after it passes the review and the tests.

- /opt/kafka[_-\.0-9]*a
- name: paths
default:
- ${kafka_home}/logs/controller.log*
Copy link
Contributor

Choose a reason for hiding this comment

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

Does this work? I would have expected this to use go template syntax, so something like {{.kafka_home}}.

@kvch kvch force-pushed the feature/filebeat-kafka-module-log branch from a45d829 to 774248f Compare August 14, 2017 16:45
@tsg
Copy link
Contributor

tsg commented Aug 15, 2017

jenkins, retest it pls


# Set custom paths for Kafka. If left empty,
# Filebeat will look under /opt.
#var.kafka_home:
Copy link
Contributor

Choose a reason for hiding this comment

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

I think we can put the default in the commented line here, so it's clear for the user how to edit. I.e.:

      #var.kafka_home: /opt/kafka*

Btw, it's /opt/kafka*, not just /opt, right?

#var.kafka_home:

# Set custom paths for the log files. If left empty,
# Filebeat will choose the paths depending on your OS.
Copy link
Contributor

Choose a reason for hiding this comment

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

This comment is out-of-date now, I think. It should say it chooses the paths based on the kibana_path value.

@tsg
Copy link
Contributor

tsg commented Aug 15, 2017

There seems to be some issue with the Kibana dashboard:

screen shot 2017-08-15 at 12 16 31

I'm not sure if something changed on the Kibana side that broke our saving/loading of dashboards..

@tsg
Copy link
Contributor

tsg commented Aug 15, 2017

I see now that this also affects the Postgres dashboards, but not other dashboards.

@kvch
Copy link
Contributor Author

kvch commented Aug 15, 2017

jenkins, test it

@monicasarbu monicasarbu added the in progress Pull request is currently in progress. label Aug 15, 2017
@kvch kvch force-pushed the feature/filebeat-kafka-module-log branch from 774248f to 2c57cb9 Compare August 15, 2017 14:41
@kvch kvch removed the in progress Pull request is currently in progress. label Aug 15, 2017
@kvch
Copy link
Contributor Author

kvch commented Aug 15, 2017

I updated the dashboards and now it is working. I will open a separate PR for the PostgreSQL dashboards.

@tsg tsg merged commit 7b56380 into elastic:master Aug 16, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants