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

Add Filebeat input plugin #4911

Closed
wants to merge 1 commit into from
Closed

Conversation

dmitryilyin
Copy link
Contributor

Add a new Filebeat 6 Monitoring API input plugin.

  • Signed CLA.
  • Associated README.md updated.
  • Has appropriate unit tests.

@gl1ridae
Copy link

Hi! Any information regarding when it will be merged?

@Esity
Copy link

Esity commented Jun 6, 2019

@gl1ridae what do you think about updating this to work for all beats(filebeat, packetbeat, metricbeat, etc) instead of just filebeat. They should all function pretty similar from a collection and metric perspective and it wouldn't require much for code changes

@gl1ridae
Copy link

gl1ridae commented Jun 7, 2019

@Esity Sounds great actually, just checked beats endpoints. I can try to figure it out and create mr.

@dmitryilyin
Copy link
Contributor Author

dmitryilyin commented Jun 14, 2019

@Esity Well, perhaps you're right. I've also made input for kafkabeat #5992 and it's almost identical.

I'll make a single input for "beat" instead.

@nferch
Copy link
Contributor

nferch commented Nov 7, 2019

Since this seemed pretty straightforward, started on merging the two, see nferch@416bb8e . Seems to work with my Filebeat instance, don't have a Kafka instance to test it on.

What do y'all think? @gl1ridae @dmitryilyin

@nferch nferch mentioned this pull request Nov 13, 2019
3 tasks
@sjwang90
Copy link
Contributor

Closing this as we'd prefer the general #6653 Beat input plugin. Will re-open if we need to separate them (file, kafka, etc.) out.

Please provide any feedback to PR #6653 and that will help us get it merged in.

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.

6 participants