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

APM Server support #49

Open
wants to merge 5 commits into
base: master
Choose a base branch
from
Open

Conversation

xco-sk
Copy link

@xco-sk xco-sk commented Oct 8, 2021

This PR adds the full (as of APM server v7.10.1) support with all the APM Server metrics exported. Alongside with the addition of APM-server-specific metrics, the general issue with the APM server beat name (name containing "-" between apm and server) has been fixed.

Feature successfully tested with APM server v7.10.1, other versions are yet to be tested, but considering the size of this change, I would like to start with feedback loops.
I will also test the metric and filebeats for potential regression. I will post the results here.

Closes #42

@erik-bizzdo
Copy link

I am the author of issue #42. I built a local image of beat-exporter with this pull request applied and deployed the image as a second container inside my apm-server pod. Beat-exporter now extracts metrics from apm-server and exposes the metrics as expected to Prometheus. My Prometheus scrapes them without any problem.

Thank you for this. Now I can finally get to the bottom of my apm-server performance issues.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Crash while trying to export metrics for apm-server.
2 participants