Skip to content

Avoid timeouts for batch requests - #1070

Merged
zetter-rpf merged 1 commit into
mainfrom
stop-import-timeouts
Oct 22, 2026
Merged

zetter-rpf merged 1 commit into
mainfrom
stop-import-timeouts

Conversation

@zetter-rpf

Copy link
Copy Markdown
Contributor

Status

What's changed?

We're seeing increased connection errors from the SchoolClasses#import action and reports of errors when importing students

Looking at the last month, some of these requests take 30 seconds (which I believe may have hit timeouts before, they are just even more likely to now)

The platform team are looking at ways we can speed up the issue as creating passwords for users may be a bottleneck. We could also look at moving the class import into a background job.

@cla-bot cla-bot Bot added the cla-signed label Oct 22, 2026
We're seeing increased connection errors from the SchoolClasses#import action and reports of errors when importing students

Looking at the last month, some of these requests take 30 seconds (which I believe may have hit timeouts before, they are just even more likely to now)

The platform team are looking at ways we can speed up the issue as creating passwords for users may be a bottleneck. We could also look at moving the class import into a background job.
@zetter-rpf
zetter-rpf force-pushed the stop-import-timeouts branch from d8e76f2 to 6a24cdb Compare October 22, 2026 08:56
@github-actions

github-actions Bot commented Oct 22, 2026 •

Copy link
Copy Markdown

Test coverage

94.29% line coverage reported by SimpleCov.
Run: https://github.com/RaspberryPiFoundation/editor-api/actions/runs/39751560363

@zetter-rpf
zetter-rpf marked this pull request as ready for review October 22, 2026 08:57
@raspberrypiherokubot
raspberrypiherokubot temporarily deployed to editor-api-p-stop-impor-hercke October 22, 2026 09:01 Inactive

@cocomarine cocomarine left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@zetter-rpf
zetter-rpf merged commit 2f030f0 into main Oct 22, 2026
9 checks passed
@zetter-rpf
zetter-rpf deleted the stop-import-timeouts branch October 22, 2026 10:26

This branch was previously deployed

1 inactive deployment
editor-api-p-stop-impor-hercke — 6a24cdb5 Deployed Oct 22, 2026 by raspberrypiherokubot
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants