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

Add feature to filter project ID at call-time #164

Merged

Conversation

xairos
Copy link
Contributor

@xairos xairos commented Jul 18, 2022

Hey @SuperQ ! This is an implementation of the feature proposal in #145, TLDR to allow filtering by project ID at scrape-time. Our use case is multi-tenancy, where each tenant can scrape a centrally hosted SD exporter with their own project IDs as scrape parameters.

Closes #145.

Breaking changes

Call-time filtering via the collect parameter now also matches if any preconfigured metric prefix is a string prefix for the call-time filter.

Example

Running stackdriver exporter with the prefix dataflow.googleapis.com/job/ would allow you to use the following collect config:

collect:
- dataflow.googleapis.com/job/is_failed
- dataflow.googleapis.com/job/elapsed_time

@xairos xairos force-pushed the project-id-filtering-feature branch from 6f80a05 to a5673c9 Compare July 25, 2022 17:25
@SuperQ SuperQ self-requested a review July 25, 2022 19:56
@SuperQ
Copy link
Contributor

SuperQ commented May 25, 2023

This needs a rebase.

Copy link
Contributor

@SuperQ SuperQ left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This needs a rebase.

Signed-off-by: Ben Rottke <benjaminr@ecobee.com>
Signed-off-by: Ben Rottke <benjaminr@ecobee.com>
@xairos xairos force-pushed the project-id-filtering-feature branch from a5673c9 to 12b1048 Compare March 11, 2024 19:50
@xairos xairos requested a review from SuperQ March 11, 2024 19:51
Signed-off-by: Ben Rottke <benjaminr@ecobee.com>
@SuperQ SuperQ merged commit bc18b73 into prometheus-community:master Mar 11, 2024
4 checks passed
xairos added a commit to xairos/stackdriver_exporter that referenced this pull request Mar 13, 2024
…munity#164)"

This reverts commit bc18b73.

Signed-off-by: Ben Rottke <benjaminr@ecobee.com>
SuperQ pushed a commit that referenced this pull request Mar 13, 2024
This reverts commit bc18b73.

Signed-off-by: Ben Rottke <benjaminr@ecobee.com>
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.

Enhancement proposal: Optionally filter projectIDs at scrape-time
2 participants