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 pie chart for highest build time by package #452

Merged
merged 1 commit into from
Apr 10, 2023

Conversation

jjnesbitt
Copy link
Collaborator

image (2)

I'll no doubt be making more changes to this, but just want to encode what I have to start.

@tgamblin
Copy link
Member

tgamblin commented Apr 6, 2023

This looks great! Stuff we'll also want to know:

  1. Total build time (the graph probably needs an "other" wedge, or it needs to say what percent of total build time the top packages are)
  2. total number of jobs (we already have this elsewhere)
  3. how many jobs comprise each wedge
  4. min/max/average/median build time in each wedge
  5. what percent of each wedge was retries vs. non-retried builds

@jjnesbitt
Copy link
Collaborator Author

This looks great! Stuff we'll also want to know:

  1. Total build time (the graph probably needs an "other" wedge, or it needs to say what percent of total build time the top packages are)
  2. total number of jobs (we already have this elsewhere)
  3. how many jobs comprise each wedge
  4. min/max/average/median build time in each wedge
  5. what percent of each wedge was retries vs. non-retried builds

Since a lot of this would depend on other changes to our data analysis infrastructure (e.g. setting up a replica gitlab db), I think we should merge this as is, and I'll create a new issue for your comment.

@jjnesbitt jjnesbitt merged commit 57557f9 into main Apr 10, 2023
@jjnesbitt jjnesbitt deleted the package-total-build-time branch April 10, 2023 17:12
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.

None yet

3 participants