Skip to content

Add Sangfor AD exporter#2832

Closed
shaxiaozz wants to merge 1 commit intoprometheus:mainfrom
shaxiaozz:add-sangfor-ad-exporter
Closed

Add Sangfor AD exporter#2832
shaxiaozz wants to merge 1 commit intoprometheus:mainfrom
shaxiaozz:add-sangfor-ad-exporter

Conversation

@shaxiaozz
Copy link

https://groups.google.com/g/prometheus-developers/c/jHROaPs0g0U

Signed-off-by: 傻笑zz <43721571+shaxiaozz@users.noreply.github.com>
@jan--f
Copy link
Contributor

jan--f commented Feb 10, 2026

Hello from the bug scrub!
Thank you for your contribution. I'm afraid we need to decline this for now. We are happy to feature exporters that are actively developed and used. Unfortunately this one has so far only a limited community behind it and seemingly low adoption. Please try again at a later stage.
The exporter itself could use a few improvements with regards to the Prometheus data model (e.g. don't export rates but rather counters and let Prometheus calculate rates, see https://prometheus.io/docs/instrumenting/writing_exporters/).

@jan--f jan--f closed this Feb 10, 2026
@shaxiaozz
Copy link
Author

@jan--f okk,thank you.

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.

2 participants