toolhive icon indicating copy to clipboard operation
toolhive copied to clipboard

Add optional installation flag for Virtual MCP CRDs in operator-crds Helm chart

Open JAORMX opened this issue 1 month ago • 1 comments

Summary

The toolhive-operator-crds Helm chart currently installs all CRDs including Virtual MCP CRDs by default. Users who only need core MCP server management (without Virtual MCP aggregation features) should be able to skip installing these CRDs.

Background

Virtual MCP CRDs (54KB total):

  • VirtualMCPServer - Aggregates multiple backend MCP servers into unified endpoint
  • VirtualMCPCompositeToolDefinition - Defines reusable composite workflows

Core CRDs (96KB total):

  • MCPServer, MCPRegistry, MCPGroup, MCPRemoteProxy, MCPExternalAuthConfig, MCPToolConfig

Currently all 8 CRDs are always installed from deploy/charts/operator-crds/crds/ with no configuration options.

Proposed Solution

Add a Helm value to make Virtual MCP CRDs optional:

# values.yaml
crds:
  install:
    virtualMCP: true  # default false

Related Files

  • deploy/charts/operator-crds/crds/toolhive.stacklok.dev_virtualmcpservers.yaml
  • deploy/charts/operator-crds/crds/toolhive.stacklok.dev_virtualmcpcompositetooldefinitions.yaml
  • deploy/charts/operator-crds/values.yaml (currently empty)
  • cmd/thv-operator/main.go (controller registration at lines 184-199)

JAORMX avatar Nov 24 '25 16:11 JAORMX

See also: https://github.com/stacklok/toolhive/issues/2564

jhrozek avatar Nov 24 '25 16:11 jhrozek