-
Notifications
You must be signed in to change notification settings - Fork 152
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
Option to aggregate PR-builds from same place #500
Comments
Appears highly related to: |
Hi, thank you for the issue. I'll have to investigate how this is possible. Sounds like a valid use case. So should this only be applied to PullRequests? Or should it be more like a general grouping by.. I don't know regex or something? |
Maybe it could regex on PR builds, but it might be more simple to generalize it. We might want metrics for all folders of builds, to show aggregation at the directory level. The use case is slightly different, but the logic might be more simple to implement. Like for builds:
We could want to:
The directory statistics could be very powerful for groups of teams. Our specific use case is for multibranch pipeline jobs, but this might also work to generalize outside that specific use case. |
I see. I cannot promise that I'll be able to implement it fast but I'll see what I can do. Think it could be useful :) |
Stale issue message |
"activity" |
I have the same issue when I'm using this plugin with the workflow-multibranch plugin. A quick win could be to detect builds attached to a multibranch workflow and aggregate them together. @Waschndolos, I have some free time to work on it if you want. |
@seguins. Hey thank you for reaching out. My plan was to implement this feature when I'm done testing my refactoring in the refactoring branch (https://github.com/jenkinsci/prometheus-plugin/tree/refactoring). But I guess I'm able to merge it when you've already have an idea to implement this in the current code. Otherwise I'll try to finish testing this week on my companies Jenkins instance and after that feel free to help :) |
Stale issue message |
Quick update. It's not forgotten :) I'm working on it. Didn't have a good idea yet but I might be making progress right now. |
Stale issue message |
Stale issue message |
My idea didn't work. Don't have a good idea on how to implement it. Does anybody have one? |
@Waschndolos what was your idea? I might be able to take on this work. |
@drewandersen. Don't have a specific one but probably it's best to let the plugin calculate all individual metrics and then before writing it out to group the relevant metrics somehow. Maybe it's possible to write a custom CollectorRegistry which can do that? |
What feature do you want to see added?
With projects that are large, with many PRs, metrics are potentially too large, giving too much detail between different PRs, where differences between PRs is not helpful for metrics given the extra storage and bandwidth required.
As an SRE managing metrics and dashboards for Jenkins builds,
I would like to simplify and simplify the aggregation across PRs of the same build,
so that I don't have to store and transmit so much data and Prometheus doesn't have to work so hard to aggregate it at query time.
For example, if I have a build with 2 PRs, and it shows statistics like this:
with this option enabled, it would aggregate like so:
(and for all other per-build metrics)
We will want to continue to differentiate on the other labels. "buildable" and "repo"
Upstream changes
No response
The text was updated successfully, but these errors were encountered: