Repository navigation
ci: move the production docs deploy to Blacksmith and its build cache - #8127
Conversation
#8101 missed the production docs deploy: it stayed on a GitHub runner with no Next build cache, and the preview docs job lost its cache step on the belief that production had none. Docs now runs on Blacksmith and writes the cache that the preview reads, like the other four apps. The five production jobs also install the Vercel CLI with bun, as the previews do.
|
Navigate logical layers of code changes, visualize relationships, and explore their blast radius. Note Currently processing new changes in this PR. This may take a few minutes, please wait... ⚙️ Run configuration
📒 Files selected for processing (2)
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
🧹 Preview Cleanup CompleteThe following preview resources have been cleaned up:
Thank you for your contribution! 🎉 |
There was a problem hiding this comment.
No issues found across 2 files
Architecture diagram
sequenceDiagram
participant GH as GitHub Actions
participant BS as Blacksmith Runner
participant CA as Cache Store
participant VE as Vercel CLI
participant VAPI as Vercel API
Note over GH,BS: Production Deploy (push to main)
GH->>BS: Trigger deploy-docs job
BS->>BS: Setup Bun runtime
BS->>VE: Install Vercel CLI (bun add --global)
BS->>CA: Save Next.js build cache (key: next-docs-{sha})
BS->>BS: Build docs app
BS->>VE: Run vercel deploy --prod
VE->>VAPI: Deploy docs to production
VAPI-->>VE: Deployment URL
VE-->>BS: Success
Note over GH,BS: Preview Deploy (PR)
GH->>BS: Trigger preview docs job
BS->>BS: Setup Bun runtime
BS->>VE: Install Vercel CLI (bun add --global)
BS->>CA: Restore Next.js build cache (restore-keys: next-docs-)
alt Cache hit
CA-->>BS: Restored cache
else Cache miss
CA-->>BS: No cache found (cold build)
end
BS->>BS: Build docs app
BS->>VE: Run vercel deploy --preview
VE->>VAPI: Deploy docs preview
VAPI-->>VE: Preview URL
VE-->>BS: Success
Why
#8101 missed the production docs deploy. It stayed on a GitHub runner with no Next build cache, and the preview docs job lost its cache step because I wrongly believed production had no docs deploy.
What changes
Deploy Docs to Vercelruns on Blacksmith 4 vCPU and writes the Next build cache, like the other four apps.Verified
main, so it also measures the warm path for API, Web, Marketing and Admin.Not verified
main.Summary by cubic
Finishes the production deploy move that #8101 started, which missed the docs job. Docs now deploys on Blacksmith and writes a Next build cache that the preview job restores, and all five production deploy jobs install the Vercel CLI with bun instead of npm.
The production side only runs on push to
main, so the warm cache path is only verified by the next production run.Written for commit f2a7a64. Summary will update on new commits.
Summary by CodeRabbit