Skip to content

WIP: Get total runs #8695

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

Open
wants to merge 5 commits into
base: master
Choose a base branch
from
Open

Conversation

djangoliv
Copy link
Contributor

No description provided.


def get_summary_groups():
url = "https://treeherder.mozilla.org/api/groupsummary/"
response = requests.get(url, headers={"User-agent": "mach-test-info/1.0"})
Copy link
Collaborator

Choose a reason for hiding this comment

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

I believe this only gets a single day, we will want an entire range of data. Also this takes ~25 seconds to return, which is why there is an artifact that exists with this data already, and it has 30 days of history, so you can choose which days you want.

The only problem is you will need to combine data from different days to summarize it.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

OK, thanks for that precisions.
Can you give me the url of the summary group artifact please? and how to access a particular day?

Copy link
Collaborator

Choose a reason for hiding this comment

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

test-run-info.json: https://searchfox.org/mozilla-central/source/testing/testinfo.py#400

there is code in there to get the data and squash it to combine days into a larger data structure:
https://searchfox.org/mozilla-central/source/testing/testinfo.py#397

as this is run counts split up by day, when looking at a week (7 days) you will need to squash data, otherwise you can pull a specific date (current day - 1) and get results.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I have taken your comments into account, however there is a lint problem with prettier that I am not reproducing locally.

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.

3 participants