docs: rewrite the CoreWeave (CKS) guide as a full 0-100 deployment walkthrough#12
Merged
Conversation
…lkthrough Cluster + node pool creation, enabling the sandbox runner, sandbox profile authoring/binding via cwic file import, the CoreWeave Traefik/cert-manager ingress stack, CAIOS bucket setup, the Helm install, and wildcard kernel TLS - each step opinionated on one path with links to the official CoreWeave docs, plus optional add-ons (auto CAIOS creds, GPU profiles, custom domain). Also fix docs-wide inaccuracies found along the way: the health endpoint is /api/health (there is no /healthz route), and MARIMOHUB_COMPUTE_COREWEAVE_PROFILE selects runner binding names, not ingress/egress modes.
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
…nfig registry docs/configuration.md is generated from spec.ts; the hand-edited row broke the docs-sync test.
…guard generated docs in pre-commit vp staged now runs the config docs-sync test and the wizard snapshot tests when their generated sources are staged, so drift fails locally instead of in CI.
There was a problem hiding this comment.
Review completed against the latest diff
Reply with feedback, questions, or to request a fix.
Re-trigger cubic
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Cluster + node pool creation, enabling the sandbox runner, sandbox profile
authoring/binding via cwic file import, the CoreWeave Traefik/cert-manager
ingress stack, CAIOS bucket setup, the Helm install, and wildcard kernel TLS
plus optional add-ons (auto CAIOS creds, GPU profiles, custom domain).
Also fix docs-wide inaccuracies found along the way: the health endpoint is
/api/health (there is no /healthz route), and MARIMOHUB_COMPUTE_COREWEAVE_PROFILE
selects runner binding names, not ingress/egress modes.
Summary by cubic
Rewrote the CoreWeave (CKS) deployment guide into a complete 0→100 walkthrough, covering cluster and node pools, enabling the managed sandbox runner, profile authoring/binding via
cwic, ingress stack, CAIOS storage, Helm install, and wildcard TLS for kernels. Also updated docs to use/api/health, alignedMARIMOHUB_COMPUTE_COREWEAVE_PROFILE, and added pre-commit checks to keep generated docs and wizard snapshots in sync.New Features
cwicwith per-sandbox hostnames; bind underprofile_namemarimohub.traefikandcert-managerwith automatic DNS-01 for*.coreweave.app.TLSStoredefault for kernel hosts; plus validation, troubleshooting, and optional add-ons.Bug Fixes
/api/health(there is no/healthz).MARIMOHUB_COMPUTE_COREWEAVE_PROFILEusage and examples across the config registry, docs, and the deployment wizard snapshots (nowmarimohub), and added pre-commit checks to run docs-sync and wizard snapshot tests to catch drift locally.Written for commit 6bdc31d. Summary will update on new commits.