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

stacklok / toolhive / 24182551495
65%

Build:
DEFAULT BRANCH: main
Ran 09 Apr 2026 09:24AM UTC
Jobs 1
Files 615
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

09 Apr 2026 09:18AM UTC coverage: 64.96% (-0.07%) from 65.026%
24182551495

push

github

web-flow
Add MCPServerEntry validation controller and MCPGroup integration (#4664)

* Add MCPServerEntry validation controller and MCPGroup integration

Introduce a validation-only controller for MCPServerEntry that checks
referenced resources (MCPGroup, MCPExternalAuthConfig, CA bundle
ConfigMap) exist and sets status conditions accordingly. The controller
never creates infrastructure or probes remote URLs.

Integrate MCPServerEntry into the MCPGroup controller so groups track
entry membership in Entries/EntryCount status fields, handle entry
deletion notifications, and watch for entry changes.

Wire up field indexing for MCPServerEntry.Spec.GroupRef and register
the controller in the operator's server controller setup. Extract
field indexer setup into setupGroupRefFieldIndexes to keep
setupServerControllers under the cyclomatic complexity limit.

Refs: #4656

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

* Add MCPServerEntry to Chainsaw RBAC assertions and integration test suites

The Chainsaw E2E tests assert on the exact ClusterRole contents, and
the integration test suites need field indexers for all types that
reference MCPGroup via spec.groupRef.

Add mcpserverentries and mcpserverentries/status to the RBAC golden
files for both multi-tenancy and single-tenancy Chainsaw test setups.
Register MCPServerEntry field indexer in all five integration test
suite BeforeSuite blocks.

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

* Address review feedback for MCPServerEntry controller

- Fix RBAC: reduce mcpserverentries verbs to get;list;watch (validation-only)
- Fix field indexer nesting bug in 4 integration test suites where
  MCPServerEntry indexer was unreachable on the happy path
- Add MCPGroup Ready phase check in validateGroupRef, matching
  MCPServer/MCPRemoteProxy behavior
- Return transient errors for requeue instead of persisting misleading
  NotFound conditions; sanitize error messages in status conditi... (continued)

269 of 510 new or added lines in 5 files covered. (52.75%)

6 existing lines in 3 files now uncovered.

55556 of 85524 relevant lines covered (64.96%)

62.51 hits per line

Uncovered Changes

Lines Coverage ∆ File
142
52.67
cmd/thv-operator/controllers/mcpserverentry_controller.go
50
64.1
-5.31% cmd/thv-operator/controllers/mcpgroup_controller.go
25
8.06
-0.74% cmd/thv-operator/main.go
24
34.83
1.17% cmd/thv-operator/api/v1alpha1/zz_generated.deepcopy.go

Coverage Regressions

Lines Coverage ∆ File
3
83.02
0.21% pkg/vmcp/composer/workflow_engine.go
2
71.43
-1.68% pkg/vmcp/k8s/manager.go
1
8.06
-0.74% cmd/thv-operator/main.go
Jobs
ID Job ID Ran Files Coverage
1 24182551495.1 09 Apr 2026 09:24AM UTC 615
64.96
GitHub Action Run
Source Files on build 24182551495
  • Tree
  • List 615
  • Changed 9
  • Source Changed 5
  • Coverage Changed 9
Coverage ∆ File Lines Relevant Covered Missed Hits/Line
  • Back to Repo
  • Github Actions Build #24182551495
  • b5427274 on github
  • Prev Build on main (#24181572055)
  • Next Build on main (#24184176203)
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