Skip to content

Conversation

@ChrisJBurns
Copy link
Contributor

Description

Type of change

  • New documentation
  • Documentation update
  • Bug fix (typo, broken link, etc.)
  • Navigation/structure change

Related issues/PRs

Screenshots

Submitter checklist

Content and formatting

  • I have reviewed the content for technical accuracy
  • I have reviewed the content for spelling, grammar, and style

Navigation

  • New pages include a frontmatter section with title and description at a minimum
  • Sidebar navigation (sidebars.ts) updated for added, deleted, reordered, or renamed files
  • Redirects added to vercel.json for moved, renamed, or deleted pages (i.e., if the URL slug changed)

Reviewer checklist

Content

  • I have reviewed the content for technical accuracy
  • I have reviewed the content for spelling, grammar, and style

Copilot AI review requested due to automatic review settings December 12, 2025 21:08
@vercel
Copy link

vercel bot commented Dec 12, 2025

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

Project Deployment Review Updated (UTC)
docs-website Ready Ready Preview, Comment Dec 12, 2025 9:08pm

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR updates the ToolHive operator installation documentation to reflect a significant architectural change: CRDs are now included directly in the operator Helm chart rather than being installed via a separate toolhive-operator-crds chart. The documentation has been restructured to present two installation methods (Helm and kubectl) using a tabbed interface, with Helm remaining the recommended approach.

Key Changes

  • Simplified Helm installation to a single command that installs both operator and CRDs
  • Added kubectl installation method as an alternative using static manifests
  • Updated upgrade and uninstall procedures to reflect the new single-chart architecture
  • Added documentation for CRD configuration options (install.server, install.registry, install.virtualMcp, keep)

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 1 comment.

File Description
docs/toolhive/tutorials/quickstart-k8s.mdx Removed separate CRD installation step, simplified to single operator install command
docs/toolhive/tutorials/opentelemetry.mdx Removed separate CRD installation command from kind cluster setup
docs/toolhive/guides-k8s/intro.mdx Updated installation overview to mention both Helm and kubectl methods
docs/toolhive/guides-k8s/deploy-operator-helm.mdx Major restructure: added tabs for Helm vs kubectl methods, documented CRD configuration options, updated upgrade/uninstall procedures, removed references to separate CRD chart

Comment on lines +393 to +394
kubectl apply -f https://github.com/stacklok/toolhive/releases/download/VERSION/toolhive-operator-crds.yaml
kubectl apply -f https://github.com/stacklok/toolhive/releases/download/VERSION/toolhive-operator.yaml
Copy link

Copilot AI Dec 12, 2025

Choose a reason for hiding this comment

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

Inconsistent placeholder formatting. The placeholder VERSION should use angle brackets to match the convention used elsewhere in the document (line 53 uses <VERSION>). Change VERSION to <VERSION> in both URLs for consistency.

Suggested change
kubectl apply -f https://github.com/stacklok/toolhive/releases/download/VERSION/toolhive-operator-crds.yaml
kubectl apply -f https://github.com/stacklok/toolhive/releases/download/VERSION/toolhive-operator.yaml
kubectl apply -f https://github.com/stacklok/toolhive/releases/download/<VERSION>/toolhive-operator-crds.yaml
kubectl apply -f https://github.com/stacklok/toolhive/releases/download/<VERSION>/toolhive-operator.yaml

Copilot uses AI. Check for mistakes.
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