Skip to content

docs: call out compute private beta limitations#3479

Merged
nicktrn merged 1 commit intomainfrom
feat/compute-private-beta-limitations
Apr 30, 2026
Merged

docs: call out compute private beta limitations#3479
nicktrn merged 1 commit intomainfrom
feat/compute-private-beta-limitations

Conversation

@nicktrn
Copy link
Copy Markdown
Collaborator

@nicktrn nicktrn commented Apr 30, 2026

Updates the private beta page with current caveats so beta orgs aren't surprised.

Refs TRI-8900.

@changeset-bot
Copy link
Copy Markdown

changeset-bot Bot commented Apr 30, 2026

⚠️ No Changeset found

Latest commit: 7429261

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented Apr 30, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

Run ID: 93c17eb2-1613-4aee-bc51-2f98ad79cf88

📥 Commits

Reviewing files that changed from the base of the PR and between 04b4d85 and 7429261.

📒 Files selected for processing (1)
  • docs/compute-private-beta.mdx
📜 Recent review details
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (4)
  • GitHub Check: Mintlify Deployment
  • GitHub Check: Analyze (actions)
  • GitHub Check: Analyze (python)
  • GitHub Check: Analyze (javascript-typescript)
🧰 Additional context used
📓 Path-based instructions (1)
docs/**/*.mdx

📄 CodeRabbit inference engine (docs/CLAUDE.md)

docs/**/*.mdx: MDX documentation pages must include frontmatter with title (required), description (required), and sidebarTitle (optional) in YAML format
Use Mintlify components for structured content: , , , , , , /, /
Always import from @trigger.dev/sdk in code examples (never from @trigger.dev/sdk/v3)
Code examples must be complete and runnable where possible
Use language tags in code fences: typescript, bash, json

Files:

  • docs/compute-private-beta.mdx
🧠 Learnings (4)
📓 Common learnings
Learnt from: nicktrn
Repo: triggerdotdev/trigger.dev PR: 3200
File: docs/config/config-file.mdx:353-368
Timestamp: 2026-03-10T12:44:19.869Z
Learning: In the triggerdotdev/trigger.dev repository, docs PRs are often written as companions to implementation PRs (e.g., PR `#3200` documents features being added in PR `#3196`). When reviewing docs PRs, the documented features may exist in a companion/companion PR branch rather than main. Always check companion PRs referenced in the PR description before flagging missing implementations.
Learnt from: nicktrn
Repo: triggerdotdev/trigger.dev PR: 3366
File: apps/webapp/app/v3/services/worker/workerGroupService.server.ts:248-257
Timestamp: 2026-04-13T10:09:26.822Z
Learning: In `apps/webapp/app/v3/services/worker/workerGroupService.server.ts`, the MICROVM compute-access check in `getDefaultWorkerGroupForProject` intentionally applies only to the `regionOverride` path. When no `regionOverride` is given, `project.defaultWorkerGroup` and the global default worker group are always returned without compute-access gating — this is by design and should not be flagged as a bypass.
📚 Learning: 2026-03-25T15:29:25.889Z
Learnt from: CR
Repo: triggerdotdev/trigger.dev PR: 0
File: .cursor/rules/writing-tasks.mdc:0-0
Timestamp: 2026-03-25T15:29:25.889Z
Learning: Applies to **/trigger/**/*.{ts,tsx,js,jsx} : Specify CPU/RAM requirements using the `machine` option with a `preset` value (e.g., 'large-1x')

Applied to files:

  • docs/compute-private-beta.mdx
📚 Learning: 2026-04-13T10:09:26.822Z
Learnt from: nicktrn
Repo: triggerdotdev/trigger.dev PR: 3366
File: apps/webapp/app/v3/services/worker/workerGroupService.server.ts:248-257
Timestamp: 2026-04-13T10:09:26.822Z
Learning: In `apps/webapp/app/v3/services/worker/workerGroupService.server.ts`, the MICROVM compute-access check in `getDefaultWorkerGroupForProject` intentionally applies only to the `regionOverride` path. When no `regionOverride` is given, `project.defaultWorkerGroup` and the global default worker group are always returned without compute-access gating — this is by design and should not be flagged as a bypass.

Applied to files:

  • docs/compute-private-beta.mdx
📚 Learning: 2026-03-10T12:44:14.176Z
Learnt from: nicktrn
Repo: triggerdotdev/trigger.dev PR: 3200
File: docs/config/config-file.mdx:353-368
Timestamp: 2026-03-10T12:44:14.176Z
Learning: In the trigger.dev repo, docs PRs are often companions to implementation PRs. When reviewing docs PRs (MDX files under docs/), check the PR description for any companion/related PR references and verify that the documented features exist in those companion PRs before flagging missing implementations. This ensures docs stay in sync with code changes across related PRs.

Applied to files:

  • docs/compute-private-beta.mdx
🔇 Additional comments (2)
docs/compute-private-beta.mdx (2)

51-53: Accurate and actionable default-region caveat.

Clear warning and concrete mitigation (cancel + re-trigger) should prevent confusion during region switches.


59-65: Strong beta limitations callout for machine sizes and disk cap.

These updates set expectations well and reduce surprise by explicitly documenting current support boundaries and planned improvements.


Walkthrough

The pull request modifies the compute private beta documentation to clarify operational behavior when changing the project-wide default region, specifying that in-queue runs do not automatically migrate. It updates machine sizing guidance regarding snapshot generation expectations, clarifies support status for small-2x and medium-* sizes, and documents a current 1GB disk limitation across all sizes despite the documented 10GB target. The scope of upcoming CLI and SDK changes is expanded to include lifting the disk cap and enabling larger presets.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~4 minutes

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Description check ⚠️ Warning The description is minimal and missing several required template sections including Testing, Changelog, and Checklist. Fill out the required template sections: add Testing steps, complete the Changelog section with details, and check off the Checklist items to confirm compliance with contributing guidelines.
✅ Passed checks (4 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly and concisely summarizes the main change: documenting compute private beta limitations.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch feat/compute-private-beta-limitations

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.

❤️ Share
Review rate limit: 7/8 reviews remaining, refill in 7 minutes and 30 seconds.

Comment @coderabbitai help to get the list of available commands and usage tips.

@mintlify
Copy link
Copy Markdown
Contributor

mintlify Bot commented Apr 30, 2026

Preview deployment for your docs. Learn more about Mintlify Previews.

Project Status Preview Updated (UTC)
trigger 🟢 Ready View Preview Apr 30, 2026, 4:39 PM

💡 Tip: Enable Workflows to automatically generate PRs for you.

Copy link
Copy Markdown
Contributor

@devin-ai-integration devin-ai-integration Bot left a comment

Choose a reason for hiding this comment

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

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no bugs or issues to report.

Open in Devin Review

@nicktrn nicktrn merged commit 24de77c into main Apr 30, 2026
12 checks passed
@nicktrn nicktrn deleted the feat/compute-private-beta-limitations branch April 30, 2026 16:40
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