Skip to content

fix: retry failed downloads#2688

Merged
simon04 merged 1 commit into
freeCodeCamp:mainfrom
ojeytonwilliams:fix/retry-failed-downloads
May 26, 2026
Merged

fix: retry failed downloads#2688
simon04 merged 1 commit into
freeCodeCamp:mainfrom
ojeytonwilliams:fix/retry-failed-downloads

Conversation

@ojeytonwilliams
Copy link
Copy Markdown
Contributor

This makes the build retry failed downloads a couple of times.

I checked locally and it handles the failures without any obvious issues.

Ref: #2686

@ojeytonwilliams ojeytonwilliams requested a review from a team as a code owner May 26, 2026 09:14
Copy link
Copy Markdown
Contributor

@simon04 simon04 left a comment

Choose a reason for hiding this comment

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

Amazing, thank you for implementing this workaround.

@simon04 simon04 merged commit 3ab3d76 into freeCodeCamp:main May 26, 2026
2 checks passed
@ojeytonwilliams ojeytonwilliams deleted the fix/retry-failed-downloads branch May 26, 2026 09:40
@ojeytonwilliams
Copy link
Copy Markdown
Contributor Author

No worries, it looks like it was enough.

It's a little hard to figure out what's wrong because it seems the bucket sometimes simply doesn't respond. I'll see if I can find out why.

@ojeytonwilliams
Copy link
Copy Markdown
Contributor Author

Rather curiously, it only needed to retry once in the latest deploy.

@simon04
Copy link
Copy Markdown
Contributor

simon04 commented May 26, 2026

Deployments used to take only approx 2min in 2025-12; since then the times have steadily increased: https://github.com/freeCodeCamp/devdocs/actions/workflows/build.yml

@ojeytonwilliams
Copy link
Copy Markdown
Contributor Author

Right. We could be seeing failures more often simply because the number of requests keeps growing. Maybe. I'm not 100% sure that's the whole story, though.

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