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

StatsD Support Not as Stated #67

Open
louisburton opened this issue Jul 28, 2020 · 1 comment
Open

StatsD Support Not as Stated #67

louisburton opened this issue Jul 28, 2020 · 1 comment
Assignees
Labels
Milestone

Comments

@louisburton
Copy link

louisburton commented Jul 28, 2020

In the latest release, v4.1.2, and lower releases - this plugin is named emqx_statsd and documentation suggests it supports pushing to a statsd endpoint. In fact, I see the repository renaming was only recent and as yet unreleased.
https://docs.emqx.io/broker/latest/en/advanced/metrics-and-stats.html

However, upon trying to configure this to push to a GoStatsD endpoint (port 8125/udp) that I use to bridge to NewRelic, I can't get the integration to work.
Upon digging, it looks like the code is very specifically tied to the prometheus pushgateway metric endpoint format, and looks like it tries to push via HTTP to endpoints like http://pushgateway.example.org:9091/metrics/job/some_job.
I can expose HTTP ingestion endpoints, but not at paths like /metrics/job/some_job which I believe are dynamic to the node name?

https://github.com/emqx/emqx-prometheus/blob/v4.1.2/src/emqx_statsd.erl#L85-L87

Is pushing to statsd supported?
I noticed the dependent library seem to allow pushing via UDP to a statsd endpoint, but is this used in this fashion? https://github.com/lpgauth/statsderl

Would love to get statsd metrics into my non-prometheus system 🙏

@tigercl
Copy link

tigercl commented Jul 29, 2020

@louisburton For some historical reasons, the emqx-statsd plugin (renamed emqx-prometheus) does not support statsd, and the previous name may misunderstand users, so we have corrected it now and will support the standard statsd and dogstatsd in emqx v5.0.

@tigercl tigercl added this to the 5.0-beta.1 milestone Jul 29, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants