• Home
  • Features
  • Pricing
  • Docs
  • Announcements
  • Sign In

stacklok / toolhive / 22754465418
64%

Build:
DEFAULT BRANCH: main
Ran 06 Mar 2026 08:06AM UTC
Jobs 1
Files 534
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

06 Mar 2026 07:57AM UTC coverage: 63.792% (+0.02%) from 63.768%
22754465418

push

github

web-flow
MCPRemoteProxy: Surface validation errors via Events and Status Conditions (#4024)

* Improve MCPRemoteProxy error reporting with events and status conditions

Add pre-deployment validation to the MCPRemoteProxy controller that
emits Kubernetes Events and sets a ConfigurationValid status condition
when configuration errors are detected. This replaces the previous
behavior where users had to inspect pod logs to understand why their
proxy was in CrashLoopBackOff.

Validations added:
- OIDC issuer URL scheme (HTTP vs HTTPS) with insecureAllowHTTP support
- Cedar authorization policy syntax checking
- Referenced authz ConfigMap existence and key validation
- Referenced header Secret existence validation

Closes #2289

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

* Reduce scope to OIDC issuer URL validation only

Remove authz policy syntax validation, ConfigMap/Secret reference
checks, and header secret validation to keep the PR focused on
foundation code (EventRecorder, ConfigurationValid condition,
validation pattern) that can be extended later.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

* Add integration tests for Kubernetes Event emission

Test that validation failures emit Warning events (visible via
kubectl describe) and that successful validations do not emit
spurious warning events.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

---------

Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>

57 of 65 new or added lines in 3 files covered. (87.69%)

7 existing lines in 4 files now uncovered.

46860 of 73458 relevant lines covered (63.79%)

75.84 hits per line

New Missed Lines in Diff

Lines Coverage ∆ File
1
9.28
-0.04% cmd/thv-operator/main.go
2
94.44
-5.56% cmd/thv-operator/pkg/validation/oidc_validation.go
5
63.68
1.54% cmd/thv-operator/controllers/mcpremoteproxy_controller.go

Uncovered Existing Lines

Lines Coverage ∆ File
1
63.68
1.54% cmd/thv-operator/controllers/mcpremoteproxy_controller.go
2
80.58
0.26% pkg/transport/proxy/httpsse/http_proxy.go
2
57.89
-3.51% pkg/transport/session/sse_session.go
2
94.67
-1.33% pkg/vmcp/composer/dag_executor.go
Jobs
ID Job ID Ran Files Coverage
1 22754465418.1 06 Mar 2026 08:06AM UTC 534
63.79
GitHub Action Run
Source Files on build 22754465418
  • Tree
  • List 534
  • Changed 11
  • Source Changed 6
  • Coverage Changed 9
Coverage ∆ File Lines Relevant Covered Missed Hits/Line
  • Back to Repo
  • Github Actions Build #22754465418
  • 5bff34d4 on github
  • Prev Build on main (#22738807826)
  • Next Build on main (#22754487908)
STATUS · Troubleshooting · Open an Issue · Sales · Support · CAREERS · ENTERPRISE · START FREE · SCHEDULE DEMO
ANNOUNCEMENTS · TWITTER · TOS & SLA · Supported CI Services · What's a CI service? · Automated Testing

© 2026 Coveralls, Inc