Skip to content

feat: add retry logic to GitHub API requests - #1219

Merged
woodruffw merged 2 commits into
mainfrom
ww/retry
Oct 6, 2025
Merged

woodruffw merged 2 commits into
mainfrom
ww/retry

Conversation

@woodruffw

Copy link
Copy Markdown
Member

This uses reqwest's semi-new retry policy APIs. These aren't the most feature-rich at the moment (notably they don't have backoff support), but they're good enough for our purpose 🙂

Closes #1216.

Signed-off-by: William Woodruff <william@yossarian.net>
@woodruffw woodruffw self-assigned this Oct 6, 2025
@woodruffw woodruffw added the enhancement New feature or request label Oct 6, 2025
@kusari-inspector

kusari-inspector Bot commented Oct 6, 2025 •

Copy link
Copy Markdown

Kusari Inspector

Kusari Analysis Results:

Proceed with these changes

✅ No Flagged Issues Detected
All values appear to be within acceptable risk parameters.

No pinned version dependency changes, code issues or exposed secrets detected!

Note

View full detailed analysis result for more information on the output and the checks that were run.


@kusari-inspector rerun - Trigger a re-analysis of this PR
@kusari-inspector feedback [your message] - Send feedback to our AI and team
See Kusari's documentation for setup and configuration.
Commit: 10e335b, performed at: 2025-10-06T20:45:59Z

Found this helpful? Give it a 👍 or 👎 reaction!

Signed-off-by: William Woodruff <william@yossarian.net>
@woodruffw

Copy link
Copy Markdown
Member Author

(I don't have a great way to test this at the moment.)

@kusari-inspector

Copy link
Copy Markdown

Kusari PR Analysis rerun based on - 10e335b performed at: 2025-10-06T20:45:59Z - link to updated analysis

@github-actions

github-actions Bot commented Oct 6, 2025 •

Copy link
Copy Markdown
Contributor

🐰 Bencher Report

Branchww/retry
Testbedubuntu-latest

⚠️ WARNING: No Threshold found!

Without a Threshold, no Alerts will ever be generated.

Click here to create a new Threshold
For more information, see the Threshold documentation.
To only post results if a Threshold exists, set the --ci-only-thresholds flag.

Click to view all benchmark results
BenchmarkLatencymilliseconds (ms)
zizmor::grafana-9f212d11d0ac📈 view plot
⚠️ NO THRESHOLD
316.84 ms
🐰 View full continuous benchmarking report in Bencher

@woodruffw
woodruffw merged commit bee86b1 into main Oct 6, 2025
10 checks passed
@woodruffw
woodruffw deleted the ww/retry branch October 6, 2025 20:52

This branch was previously deployed

1 inactive deployment
bencher — 10e335b9 Deployed Oct 6, 2025 by woodruffw via Continuous Benchmarking PRs with Bencher #307
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

CI flake: Error 502 for https://api.github.com

1 participant