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

Added Filebeat Module overview and tutorial #3592

Merged
merged 2 commits into from
Feb 17, 2017

Conversation

tsg
Copy link
Contributor

@tsg tsg commented Feb 14, 2017

Went for documenting FBM in a tutorial fashion. I'll follow up with a PR to
create per-module docs.

Part of #3159.

Went for documenting FBM in a tutorial fashion. I'll follow up with a PR to
create per-module docs.

Part of elastic#3159.
@tsg tsg added docs needs_backport PR is waiting to be backported to other branches. review v5.3.0 labels Feb 14, 2017
@tsg tsg mentioned this pull request Feb 14, 2017
22 tasks
@tsg
Copy link
Contributor Author

tsg commented Feb 14, 2017

@dedemorton fyi, I started on the docs for the FBM. Happy to either get your comments here or as edits in a future PR.

@dedemorton dedemorton mentioned this pull request Feb 14, 2017
12 tasks
@tsg tsg mentioned this pull request Feb 15, 2017

NOTE: At the moment, Filebeat modules require using the Elasticsearch
{elasticsearch}/ingest.html[Ingest Node]. In the future, Filebeat Modules will
be able to also configure Logstash as a more powerful alternative to Ingest
Copy link
Contributor

Choose a reason for hiding this comment

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

I would remove the Logstash part.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I hope that by the time we release this, we'll be able to say that we have sample Logstash configs which we can mention instead: elastic/logstash#6542

Until then, and in case that doesn't happen on time, I think it's important to let people know that we do plan to allow for migrating from Ingest Node to LS.


This also assumes you have Nginx installed and writing logs in the default
location and format. If you want to monitor another service for which a module
exists, feel free to adjust the tutorial.
Copy link
Contributor

Choose a reason for hiding this comment

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

for which a module exists, adjust the commands in the tutorial accordingly.

Otherwise it kind of sounds like someone should thanks this tutorial

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Good point, although I'll do the fix in #3598 to simplify my rebases.

@ruflin ruflin merged commit 5b39cc5 into elastic:master Feb 17, 2017
tsg added a commit to tsg/beats that referenced this pull request Feb 21, 2017
Went for documenting FBM in a tutorial fashion. I'll follow up with a PR to
create per-module docs.

Part of elastic#3159.
(cherry picked from commit 5b39cc5)
ruflin pushed a commit that referenced this pull request Feb 21, 2017
Went for documenting FBM in a tutorial fashion. I'll follow up with a PR to
create per-module docs.

Part of #3159.
(cherry picked from commit 5b39cc5)
@tsg tsg removed the needs_backport PR is waiting to be backported to other branches. label Feb 21, 2017
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.

2 participants