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 etcdbeat to the community beats #4915

Merged
merged 1 commit into from Aug 18, 2017

Conversation

Projects
None yet
4 participants
@berfinsari
Contributor

berfinsari commented Aug 16, 2017

No description provided.

@elasticmachine

This comment has been minimized.

elasticmachine commented Aug 16, 2017

Can one of the admins verify this patch?

@tsg

This comment has been minimized.

Collaborator

tsg commented Aug 17, 2017

@berfinsari Nice! Two things:

  • Could you sign our CLA, please? It should only take a couple of minutes.
  • What do you think about also contributing the etcd support as a module in Metricbeat? I think it fits very well into Metricbeat.
@berfinsari

This comment has been minimized.

Contributor

berfinsari commented Aug 17, 2017

I signed the CLA. (but I signed after I made a pull request)
I want to contribute the module. My currently aim is contributing the etcd support as a module in Metricbeat and now i'm working on a module for metricbeat.

@exekias

This comment has been minimized.

Member

exekias commented Aug 17, 2017

@berfinsari

This comment has been minimized.

Contributor

berfinsari commented Aug 18, 2017

Is there anything that I can do for this unsuccessful check?

@exekias exekias merged commit 4302f53 into elastic:master Aug 18, 2017

4 of 5 checks passed

continuous-integration/travis-ci/pr The Travis CI build could not complete due to an error
Details
CLA Commit author has signed the CLA
Details
codecov/patch Coverage not affected when comparing 7b56380...625faf4
Details
codecov/project 58.3% (+0.39%) compared to 7b56380
Details
continuous-integration/appveyor/pr AppVeyor build succeeded
Details
@exekias

This comment has been minimized.

Member

exekias commented Aug 18, 2017

merged it, test errors had nothing to do with this

exekias added a commit to exekias/beats that referenced this pull request Aug 29, 2017

dedemorton added a commit to dedemorton/beats that referenced this pull request Sep 6, 2017

dedemorton added a commit to dedemorton/beats that referenced this pull request Sep 6, 2017

exekias added a commit that referenced this pull request Sep 7, 2017

Backport: Cherrypick community beats updates into 5.6 (#5114)
* Add Githubbeats to community beats (#4723)

* add cloudwatchlogsbeat to community beats (#4829)

* Add etcdbeat to the community beats (#4915)

* Edit githubbeat description

exekias added a commit that referenced this pull request Sep 7, 2017

Backport: Cherrypick community beats updates into 6.0 (#5115)
* Add etcdbeat to the community beats (#4915)

* Edit githubbeat description

ramon-garcia added a commit to ramon-garcia/beats that referenced this pull request Dec 5, 2017

athom added a commit to athom/beats that referenced this pull request Jan 25, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment