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

stacklok / toolhive / 28099346748
67%

Build:
DEFAULT BRANCH: main
Ran 24 Jun 2026 12:50PM UTC
Jobs 1
Files 763
Run time 2min
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

24 Jun 2026 12:43PM UTC coverage: 67.123% (-0.06%) from 67.184%
28099346748

push

github

web-flow
Index MCPOIDCConfig referencing-workload lookups (#5608)

* Index MCPOIDCConfig referencing-workload lookups

findReferencingWorkloads rebuilt the MCPOIDCConfig reference list — on every
reconcile and on every deletion check — by listing all MCPServers,
VirtualMCPServers, and MCPRemoteProxies in the namespace and filtering them in
memory. That is O(all workloads) and scales poorly as a namespace fills up.

Register field indexes on the three workloads' oidcConfigRef fields and query
them with client.MatchingFields, so each lookup returns only the workloads that
actually reference the config. This mirrors the existing spec.groupRef indexes
(setupGroupRefFieldIndexes, mcpserverentry_controller) and is registered in
SetupWithManager, so it applies in production and the integration suite alike.

Behavior is unchanged — the computed reference list is identical, just produced
by an indexed lookup instead of a full in-memory scan. This is the template for
the same change across the other config controllers (see #5607).

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

* Add reverse-reference controller conventions to operator rules

Codify the field-indexer + level-triggered reverse-reference pattern (and the
anti-patterns it replaces) so future controller code does not regress to
list-all-and-filter lookups, hand-rolled stale-ref scans, or unnecessary
denormalized reference lists in status. Auto-loads on cmd/thv-operator/**.

See #5607 / #5608 for the worked example.

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

---------

Co-authored-by: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

19 of 56 new or added lines in 1 file covered. (33.93%)

70 existing lines in 4 files now uncovered.

69619 of 103719 relevant lines covered (67.12%)

66.26 hits per line

Uncovered Changes

Lines Coverage ∆ File
37
46.46
-2.01% cmd/thv-operator/controllers/mcpoidcconfig_controller.go

Coverage Regressions

Lines Coverage ∆ File
59
62.3
-4.75% pkg/workloads/manager.go
5
0.0
-100.0% pkg/workloads/sysproc_unix.go
4
46.46
-2.01% cmd/thv-operator/controllers/mcpoidcconfig_controller.go
2
96.47
0.0% pkg/authserver/storage/memory.go
Jobs
ID Job ID Ran Files Coverage
1 28099346748.1 24 Jun 2026 12:50PM UTC 763
67.12
GitHub Action Run
Source Files on build 28099346748
  • Tree
  • List 763
  • Changed 7
  • Source Changed 1
  • Coverage Changed 7
Coverage ∆ File Lines Relevant Covered Missed Hits/Line
  • Back to Repo
  • Github Actions Build #28099346748
  • 174208f2 on github
  • Prev Build on main (#28087037781)
  • Next Build on main (#28101234055)
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