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

dashboard/app: export coverage json #5726

Merged
merged 2 commits into from
Feb 6, 2025

Conversation

tarasmadan
Copy link
Collaborator

@tarasmadan tarasmadan commented Jan 30, 2025

Covers min #5591 requirements.

Json size is 66MB. Gzipped version 4,5MB.
AppEngine can stream only the gzipped version because of its response size limitations.
Repo and commit are streamed last because it is a bit easier.

The response time is 10-15s for the upstream namespace.
It returns the previous (not current) month coverage.

The data is available at /upstream/graph/coverage_heatmap?json=1.
Landing the #5716 we'll be able to serve it from /coverage?json=1.

coverage.json.gz

@tarasmadan tarasmadan force-pushed the export_coverage_json branch 13 times, most recently from d38d201 to 8df0f26 Compare January 31, 2025 12:26
@tarasmadan tarasmadan marked this pull request as ready for review January 31, 2025 12:44
@tarasmadan tarasmadan requested a review from a-nogikh January 31, 2025 13:35
@a-nogikh
Copy link
Collaborator

Note that there are limits on the maximum size of dynamically generated content in GAE: 32MB
https://cloud.google.com/appengine/docs/standard/how-requests-are-handled?tab=go#quotas_and_limits

@tarasmadan
Copy link
Collaborator Author

This PR is waiting for #5743 .

@tarasmadan tarasmadan force-pushed the export_coverage_json branch from 8df0f26 to 14adc61 Compare February 5, 2025 08:45
@tarasmadan
Copy link
Collaborator Author

We can merge it before #5743 to let others start using the data.

@tarasmadan tarasmadan force-pushed the export_coverage_json branch 2 times, most recently from 5feece5 to ae8d1b5 Compare February 5, 2025 15:53
@tarasmadan tarasmadan force-pushed the export_coverage_json branch from ae8d1b5 to f86ad11 Compare February 5, 2025 19:01
@tarasmadan tarasmadan added this pull request to the merge queue Feb 6, 2025
Merged via the queue into google:master with commit 3d54a78 Feb 6, 2025
18 checks passed
@tarasmadan tarasmadan deleted the export_coverage_json branch February 12, 2025 09:35
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