Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Add a filter option to the graphite bridge.#62

Closed
iksaif wants to merge 1 commit intoprometheus:masterprometheus/client_python:masterfrom
iksaif:masteriksaif/client_python:masterCopy head branch name to clipboard
Closed

Add a filter option to the graphite bridge.#62
iksaif wants to merge 1 commit intoprometheus:masterprometheus/client_python:masterfrom
iksaif:masteriksaif/client_python:masterCopy head branch name to clipboard

Conversation

@iksaif
Copy link

@iksaif iksaif commented Oct 19, 2015

This makes it easy to only export a set of the metrics to Graphite
which is useful when you need only part of your metrics in Graphite.

This makes it easy to only export a set of the metrics to Graphite
which is useful when you need only part of your metrics in Graphite.
@brian-brazil
Copy link
Contributor

If you want to only export some metrics you should use a custom registry, rather than using a filter.

@iksaif
Copy link
Author

iksaif commented Oct 19, 2015

That means that graphite specific choices will influence the way I use prometheus.

In this case, I'd like to use prometheus as usual, putting all my metrics into /metrics, which is totally what I want but I would also like to export a subset of these metrics to graphite.

@brian-brazil
Copy link
Contributor

That should work fine with custom registries, a collector can be in more than one registry.

@iksaif
Copy link
Author

iksaif commented Oct 20, 2015

Right, I guess one way to not affect my program too much is to have two registries. The default one when I store everything and another one that I use only for what I want in graphite. Is that what you meant ?

I guess I won't have too much data so it should be ok to duplicate it.

@brian-brazil
Copy link
Contributor

Yes, that's what I meant.

Using a regex is not the ideal solution as you're dealing with the samples as munged strings. rather than time series with labels. There'll be a feature in future for http exposition allowing requesting a subset of metrics, and it'd be best to tie this sort of thing into that rather than a graphite-specific solution.

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

Morty Proxy This is a proxified and sanitized view of the page, visit original site.