stacklok / toolhive / 17854985096

73%

Build Type

push

github

Committed by web-flow

Commit Message

Thv registry deploy from controller (#1931)

* Add a service account and RBAC permissions for thv-registry-api

* Explicitly annotate mcpregistry controller to create deployments and services

* Optionally push the registry-api server to the kind cluster

* Create the deployment

* Create the service

* Actually deploy the service and the deployment

* fix lint

* actually deploy fix

* Replace DEPLOY_REGISTRY_API with ENABLE_EXPERIMENTAL_FEATURES

- Update Taskfile to use ENABLE_EXPERIMENTAL_FEATURES instead of DEPLOY_REGISTRY_API
- Replace registryAPI.serviceAccount.create condition with operator.features.experimental
- Registry API RBAC resources (ServiceAccount, ClusterRole, ClusterRoleBinding) now
  only created when experimental features are enabled
- Simplify deployment logic by consolidating feature flags

* Refactor registry-api deployment into a separate package

* Regenerate helm docs

* Reduce duplication with helper methods to get storage name and resource name for MCPRegistry

* Consolidate constants

* Add unit tests

* Consolidate logic in updating deployment to be the same as in updating service

* Review feedback: Only reconcile API service if sync err is nil

* Remove uneeded ConfigMap RBAC role

* Bump charts

Coverage Stats

286 of 770 new or added lines in 11 files covered. (37.14%)

3 existing lines in 2 files now uncovered.

15407 of 34852 relevant lines covered (44.21%)

15.72 hits per line