Repository navigation
Conversation
Replace the goal table and closing Platform paragraph with card sections grouped by reader goal. Cards cover every vCluster integration page and the related Platform integrations, with tags for worker node models, license tier, and product. Add an optional tags prop to LinkCard to show those labels. Platform cards link to /docs/platform/next paths until the Platform 5.0 release makes them stable. A comment in the page marks the switch. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
The NVIDIA Dynamo Certified Stack page landed on next during the rebase. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
✅ Deploy Preview for next-vcluster-docs-site ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
Introduce the section before explaining why KubeVirt appears twice. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
- Split sentences over 25 words in the intro. - Explain untagged cards, and drop the inaccurate claim that every Platform integration runs in the management plane. - Move the Platform docs pointer from Extend vCluster into the intro. - Note that preview environments use the platform's sleep mode. - Spell out HPA and name NVIDIA DCGM on the autoscaling card. - Shorten the Run:ai card description. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
|
@claude review this PR |
|
Claude finished @feorlen's task in 1m 11s —— View job PR review: rebuild the integrations overview with cards
This PR is in good shape. I checked every card tag against the
Every Platform Should fix1. The metrics server move only half matches its guide.
Line 15 of the guide calls the reuse method the recommended one. It reads: "In case the above recommended method of getting metrics in vCluster using the metrics server proxy does not fulfill your requirements…". So the original placement under "Reuse a tool on the control plane cluster" was arguably right, and the card text ("Install a dedicated metrics server…") describes only the fallback. Two options:
The guide also describes the recommended method as "above" when it's actually at the bottom of the page. That's an existing problem and out of scope here, but worth a follow-up ticket. Consider2. Preview environments needs the platform, but the card has no tag saying so. The card description says "with the platform's sleep mode", and the guide creates a cluster template in vCluster Platform. The page has no tier badge, so by your rule ("tags repeat what each guide states") having no tier tag is consistent. Still, a reader who filters on tags will take this for a plain vCluster guide. This probably needs a badge on the destination page, not just a tag on the card. That could be a follow-up. 3. "idle environments use no resources" overstates what sleep mode does. The guide says the same thing (line 14). Sleep mode scales workloads down, but persistent volumes and the cluster's stored state remain. Something like "so idle environments stop using compute" is more accurate. It also fits the repositioning guidance in CLAUDE.md to move away from cost framing. 4. Tag text is part of each link's accessible name. Screen readers will read a card as "Flux Shared nodes Private nodes Manage clusters as code…". That's acceptable. If you want the tags announced as a separate unit, add a visually hidden prefix such as "Supports:" or wrap the tags in Not an issue
Not run: I didn't run a local build or Vale. I'm relying on the Netlify preview and the checks listed in the PR body. |
The metrics server guide recommends reusing the control plane cluster's metrics server, with a dedicated install as the fallback. Move it back to the reuse examples and cover both methods on its card. Sleep mode reduces compute but keeps stored state, so the preview environments card no longer says idle environments use no resources. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Content description
Rebuilds the vCluster integrations overview with the card layout from Start here, grouped by reader goal, so readers can find integrations across both the vCluster and Platform docs.
LinkCardgets an optionaltagsprop. Existing cards are unchanged.Platform links on
next: Platform cards use/docs/platform/next/...paths. Until the release, this build's stable Platform version is 4.12, where the 5.0 paths don't exist, soproduct="platform"links would fail the build. An MDX comment at the top of the page marks the release-day switch toproduct="platform".KubeVirt node provider badge: the "KubeVirt: VMs as nodes" card is tagged Free, matching its destination page. This was confirmed against the plan definitions (details in DOC-1974). The Enterprise badge on the vMetal virtual machines overview covers a separate feature, so it doesn't conflict.
Testing: the Netlify deploy preview builds,
validate-mdx-linkspasses, and Vale has one warning left (the "Networking" heading, which names a group of cards). A local production build confirmed every card link resolves. In the browser, the grid is 3 columns on desktop and 1 column at narrow width, card clicks across products land on the right pages, and Tab and Enter work with a visible focus ring.Preview link
https://deploy-preview-3126--next-vcluster-docs-site.netlify.app/docs/vcluster/next/integrations
Internal reference
Closes DOC-1974
Follow-ups from the AI review: DOC-1977 (metrics server guide order) and DOC-1978 (preview environments platform requirement).
AI review: mention
@claudein a comment to request a review. Claude reads the PR and posts review comments. See CONTRIBUTING.md for example commands.🤖 Generated with Claude Code
@netlify /docs