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.

Sign up for GitHub

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

Jump to bottom

BUG: test-reporter does not load all the artifacts #166

Open

STRRL opened this issue Mar 31, 2022 · 2 comments

STRRL commented Mar 31, 2022 •

edited

Loading

test-reporter would only fetch the first page of artifacts, so it would loss some when the number of artifacts > 30.

GitHub API Reference: https://docs.github.com/en/rest/reference/actions#list-workflow-run-artifacts

Failed CI: https://github.com/chaos-mesh/chaos-mesh/actions/runs/2069421829

Related GitHub API: https://api.github.com/repos/chaos-mesh/chaos-mesh/actions/runs/2069365380/artifacts

Related Codes:

https://github.com/dorny/test-reporter/blob/main/src/input-providers/artifact-provider.ts#L53-L56

STRRL mentioned this issue

remove CI e2e-test-reporter chaos-mesh/chaos-mesh#3085

Merged

9 tasks

Author

STRRL commented Mar 31, 2022

I noticed that here is already a PR relates to this issue: #165

macrogreg commented May 17, 2022

+1
I am also facing this issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment