You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
It's not yet clear what exactly caused this. It looks like the failure has only shown up when publishing check results (test report, checkstyle report, spotbugs report). It's possible we've hit a limit just with those, or it might be related to our queries in the Check Build Status workflow.
The checks we've used haven't logged out the full response from GitHub. We could try making a call explicitly to see the full output.
If we move to a single workflow with modules on different jobs instead of workflows, we could maybe aggregate the checks together into a single report at the end. That would be covered by another issue: #186
The text was updated successfully, but these errors were encountered:
Re-running those jobs half an hour later, it seems to be fine now. It might not be anything to worry about. Probably best to wait and see if it happens again.
This seems to be down to the library we're using not supporting the calls we want to make to get the recently run workflows, ie. the branch parameter on this endpoint, or more control over paging:
Here are some builds that failed because we hit a rate limit:
https://github.com/gchq/sleeper/actions/runs/3235165786/jobs/5299217395
https://github.com/gchq/sleeper/actions/runs/3235165799/jobs/5299217705
https://github.com/gchq/sleeper/actions/runs/3235165823/jobs/5299217643
It's not yet clear what exactly caused this. It looks like the failure has only shown up when publishing check results (test report, checkstyle report, spotbugs report). It's possible we've hit a limit just with those, or it might be related to our queries in the Check Build Status workflow.
The checks we've used haven't logged out the full response from GitHub. We could try making a call explicitly to see the full output.
If we move to a single workflow with modules on different jobs instead of workflows, we could maybe aggregate the checks together into a single report at the end. That would be covered by another issue: #186
The text was updated successfully, but these errors were encountered: