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 mysqlbeat #1545

Merged
merged 3 commits into from
May 2, 2016
Merged

added mysqlbeat #1545

merged 3 commits into from
May 2, 2016

Conversation

adibendahan
Copy link
Contributor

added mysqlbeat to community beats

@elasticsearch-release
Copy link

Jenkins standing by to test this. If you aren't a maintainer, you can ignore this comment. Someone with commit access, please review this and clear it for Jenkins to run; then say 'jenkins, test it'.

@ruflin
Copy link
Member

ruflin commented May 2, 2016

LGTM

@adibendahan Do you mind signing the CLA? https://www.elastic.co/contributor-agreement/

@ruflin
Copy link
Member

ruflin commented May 2, 2016

@abdonpijpelink One small correction I just realised: We sorted the beats alphabetically. Could you put mysqlbeat at the right place?

@adibendahan
Copy link
Contributor Author

@ruflin I just signed it

@ruflin
Copy link
Member

ruflin commented May 2, 2016

@adibendahan CLA looks good. Could you update the PR with the ordering? Also let me know if I should do it for you in an other PR.

@adibendahan
Copy link
Contributor Author

@ruflin done.

@ruflin ruflin merged commit 97a7f83 into elastic:master May 2, 2016
@ruflin
Copy link
Member

ruflin commented May 2, 2016

@adibendahan Thanks a lot. Merged.

@adibendahan adibendahan deleted the adibendahan-add-mysqlbeat branch May 2, 2016 15:32
dedemorton pushed a commit to dedemorton/beats that referenced this pull request May 2, 2016
ruflin pushed a commit that referenced this pull request May 3, 2016
* Mention hwsensorsbeat on Community Beats page (#1506)
* Added mysqlbeat to community beats (#1545)
* Add link to Beats repo (#1553)

Novice users might not realize that they need to create the PR against the Beats repo, so I'm adding a link here.
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.

3 participants