stacklok / toolhive / 24343499161

push

github

Add TelemetryConfigRef support to MCPRemoteProxy (#4719)

* Add TelemetryConfigRef field and condition constants to MCPRemoteProxy

Brings MCPRemoteProxy to parity with MCPServer's telemetry API by adding
a TelemetryConfigRef field for referencing shared MCPTelemetryConfig
resources. Includes CEL mutual exclusivity validation with the deprecated
inline Telemetry field, TelemetryConfigHash in status for change
detection, and condition type/reason constants.

Part of #4620

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

* Add GetTelemetryConfigForMCPRemoteProxy helper

Namespace-scoped fetch for MCPTelemetryConfig referenced by an
MCPRemoteProxy. Returns (nil, nil) when the ref is nil or the resource
is not found, matching the MCPServer getTelemetryConfigForMCPServer
contract.

Part of #4620

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

* Add handleTelemetryConfig and MCPTelemetryConfig watch for MCPRemoteProxy

Wire up the reconciler to validate referenced MCPTelemetryConfig
resources, track config hashes in status, and reconcile when the
underlying MCPTelemetryConfig changes. Follows the same handler pattern
as handleToolConfig and the MCPServer telemetry handler.

Part of #4620

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

* Prefer TelemetryConfigRef over inline Telemetry in MCPRemoteProxy RunConfig

When building the RunConfig, resolve telemetry from the referenced
MCPTelemetryConfig first and fall back to the deprecated inline
Telemetry field. Adds ctx parameter to createRunConfigFromMCPRemoteProxy
to support the API fetch, matching the MCPServer runconfig pattern.

Part of #4620

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

* Regenerate deepcopy, CRD manifests, and API docs

Auto-generated from MCPRemoteProxy type changes: adds
telemetryConfigRef to CRD schema with CEL mutual exclusivity validation,
telemetryConfigHash to status, and updated API refer... (continued)

141 of 243 new or added lines in 6 files covered. (58.02%)

56970 of 86851 relevant lines covered (65.6%)

62.77 hits per line