Skip to content

fix(ci): resolve OOM in Algolia index workflow#17283

Merged
sergical merged 1 commit intomasterfrom
fix/algolia-index-oom
Apr 9, 2026
Merged

fix(ci): resolve OOM in Algolia index workflow#17283
sergical merged 1 commit intomasterfrom
fix/algolia-index-oom

Conversation

@sergical
Copy link
Copy Markdown
Member

@sergical sergical commented Apr 8, 2026

DESCRIBE YOUR PR

The Algolia index workflow has been consistently OOM-ing when user docs change. The runner loses communication with the server during next build on ubuntu-latest (7GB RAM).

Root cause: This is the only workflow running pnpm build without .next/cache caching — lint-404s.yml and test.yml both cache it and succeed on the same runner.

Changes:

  • Cache .next/cache between runs (matches existing pattern from lint-404s.yml)
  • Skip generate-md-exports — unnecessary for Algolia (script only reads .next/server/app/*.html)
  • Set NODE_OPTIONS=--max-old-space-size=6144 as safety net for cold builds

IS YOUR CHANGE URGENT?

Help us prioritize incoming PRs by letting us know when the change needs to go live.

  • Urgent deadline (GA date, etc.):
  • Other deadline:
  • None: Not urgent, can wait up to 1 week+

SLA

  • Teamwork makes the dream work, so please add a reviewer to your PRs.
  • Please give the docs team up to 1 week to review your PR unless you've added an urgent due date to it.
    Thanks in advance for your help!

PRE-MERGE CHECKLIST

Make sure you've checked the following before merging your changes:

  • Checked Vercel preview for correctness, including links
  • PR was reviewed and approved by any necessary SMEs (subject matter experts)
  • PR was reviewed and approved by a member of the Sentry docs team

The `next build` step was running out of memory on `ubuntu-latest` (7GB RAM),
causing the runner to lose communication with the server. Three changes:

1. Cache `.next/cache` between runs (matches pattern from lint-404s.yml/test.yml)
   so incremental builds recompile only changed pages
2. Skip `generate-md-exports` which is unnecessary for Algolia indexing
   (the script only reads `.next/server/app/*.html`)
3. Set `NODE_OPTIONS=--max-old-space-size=6144` as a safety net for cold builds

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@vercel
Copy link
Copy Markdown

vercel bot commented Apr 8, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
develop-docs Ready Ready Preview, Comment Apr 8, 2026 8:23pm
sentry-docs Ready Ready Preview, Comment Apr 8, 2026 8:23pm

Request Review

@sergical sergical merged commit 8ea3b22 into master Apr 9, 2026
18 checks passed
@sergical sergical deleted the fix/algolia-index-oom branch April 9, 2026 11:12
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