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

discussion: a metrics plugin using github.com/VictoriaMetrics/metrics #875

Open
vrischmann opened this issue Dec 14, 2024 · 7 comments
Open
Labels
has pr plugin PRs related to plugins

Comments

@vrischmann
Copy link

Hi,

this is not really an issue, I'm opening a discussion.

We use VictoriaMetrics at work and its metrics library to expose metrics from Go programs. It's a lightweight alternative to the official prometheus client.
It's not necessary to use VictoriaMetrics, it's just another, more efficient way to expose metrics in the Prometheus text format (and its histogram is easier to use).

I'm wondering if you would be open to include a plugin like kvictoria or something that would expose metrics using the metrics library, that way we could benefit from the advantages of the metrics library.

I'm willing to work on this. My plan would be to essentially copy and adapt the kprom plugin.

Let me know what you think.

@vtolstov
Copy link
Contributor

vtolstov commented Dec 14, 2024

metrics library does not support prometheus histograms

@vrischmann
Copy link
Author

metrics library does not support prometheus histograms

Do you mean that Prometheus (the database) wouldn't know what to do with a Histogram from the metrics library ?
That's probably true but this plugin would be for users of VictoriaMetrics (the database).

@vtolstov
Copy link
Contributor

i'm use metrics library in some of my projects, but without prometheus like histograms metrics can't be collected by prometheus server. So this plugin can be useful only with victoriametrics server that collects metrcis

@vrischmann
Copy link
Author

Yes. That's my use case and it might also benefit other users of VictoriaMetrics server so I'd argue a plugin is still useful.

@vtolstov
Copy link
Contributor

nice to hear!

@twmb
Copy link
Owner

twmb commented Dec 28, 2024

I'm open to this, and will review/tag a release if you open a PR. Please let me know in the PR message if this involves changes to kgo because that'll need to be merged/tagged independently.

@twmb twmb added the waiting label Dec 28, 2024
@vrischmann
Copy link
Author

Hi, thanks for the feedback.

I've started working on this and I realized that I'll have the same problem that is addressed in #820, in fact I've had this problem at work because we sometimes use multiple clients per application.

As far as I can tell the changes in #820 will work for me too.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
has pr plugin PRs related to plugins
Projects
None yet
Development

No branches or pull requests

3 participants