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

Fix go plugins not loaded when beat starts #4799

Merged
merged 1 commit into from
Aug 1, 2017

Conversation

vjsamuel
Copy link
Contributor

No description provided.

@elasticmachine
Copy link
Collaborator

Can one of the admins verify this patch?

@elasticmachine
Copy link
Collaborator

Since this is a community submitted pull request, a Jenkins build has not been kicked off automatically on build-eu-00. Can an Elastic organization member please verify the contents of this patch and then kick off a build manually?

Copy link
Contributor

@tsg tsg left a comment

Choose a reason for hiding this comment

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

LGTM, waiting for a review from @exekias as well.

@exekias exekias merged commit b7dbf03 into elastic:master Aug 1, 2017
@exekias exekias added backport needs_backport PR is waiting to be backported to other branches. v6.0.0-GA and removed backport labels Aug 1, 2017
@vjsamuel vjsamuel deleted the gix_go_plugins branch August 1, 2017 16:59
exekias pushed a commit to exekias/beats that referenced this pull request Aug 16, 2017
exekias pushed a commit to exekias/beats that referenced this pull request Aug 16, 2017
@exekias exekias removed the needs_backport PR is waiting to be backported to other branches. label Aug 16, 2017
andrewkroh pushed a commit that referenced this pull request Aug 16, 2017
…4923)

* Fix go plugins not loaded when beat starts (#4799)

(cherry picked from commit b7dbf03)
leweafan pushed a commit to leweafan/beats that referenced this pull request Apr 28, 2023
…starts (elastic#4923)

* Fix go plugins not loaded when beat starts (elastic#4799)

(cherry picked from commit 67998cd)
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