Skip to content

Give the boot translations sync more time #12017

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

Merged
iHiD merged 1 commit into from
Oct 28, 2026
Merged

Give the boot translations sync more time #12017

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
7 changes: 5 additions & 2 deletions bin/start_webserver
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -28,9 +28,12 @@ FileUtils.chdir APP_ROOT do
# as safe here as well as in the glob above.
# Slightly stale translations are far better than a webserver that will not boot,
# so the command reports failures to Sentry and returns rather than raising. The
# outer timeout covers a git process that hangs instead of failing.
# outer timeout covers a git process that hangs instead of failing. A pull that
# changes many files can take several minutes on EFS, and killing one part way
# through leaves files behind that block later pulls, so the timeout is generous.
# It still sits well inside the ECS health check grace period of 600 seconds.
system! "git config --global --add safe.directory /mnt/efs/repos/i18n"
system("timeout -k 10 180 bundle exec bin/rails runner 'TranslationRepo::SyncOnBoot.()'") ||
system("timeout -k 10 360 bundle exec bin/rails runner 'TranslationRepo::SyncOnBoot.()'") ||
warn("\n== Translations were not synced on boot ==")

# Warm the local cache of EFS to avoid big read spikes as users start to hit the website.
Expand Down
Loading