stacklok / toolhive / 24343499161
66%

DEFAULT BRANCH: main
Ran
Jobs 1
Files 618
Run time 1min
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst
coverage: 65.595% (-0.06%) from 65.657%
24343499161

push

github

web-flow 
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%)

48 existing lines in 9 files now uncovered.

56970 of 86851 relevant lines covered (65.6%)

62.77 hits per line

Uncovered Changes

Lines Coverage ∆ File
44
50.0
 -10.87% cmd/thv-operator/controllers/mcptelemetryconfig_controller.go
29
60.84
 1.21% cmd/thv-operator/controllers/mcpremoteproxy_controller.go
17
69.92
 -4.62% cmd/thv-operator/controllers/mcpremoteproxy_runconfig.go
10
87.46
 -2.67% cmd/thv-operator/controllers/mcpremoteproxy_deployment.go
2
91.3
 -0.61% cmd/thv-operator/pkg/controllerutil/config.go

Coverage Regressions

Lines Coverage ∆ File
14
74.44
 -5.19% pkg/client/config.go
11
68.83
 -14.29% pkg/client/discovery.go
8
23.56
 0.57% pkg/client/manager.go
6
76.15
 -5.5% pkg/secrets/keyring/keyctl_linux.go
3
71.85
 -1.11% pkg/ignore/processor.go
3
79.38
 -0.77% pkg/transport/proxy/httpsse/http_proxy.go
1
60.84
 1.21% cmd/thv-operator/controllers/mcpremoteproxy_controller.go
1
69.92
 -4.62% cmd/thv-operator/controllers/mcpremoteproxy_runconfig.go
1
50.0
 -10.87% cmd/thv-operator/controllers/mcptelemetryconfig_controller.go
Jobs
ID Job ID Ran Files Coverage
1 24343499161.1 618
65.6
 GitHub Action Run
Source Files on build 24343499161