stacklok / toolhive / 23590683587

65%

Build Type

push

github

Committed by web-flow

Commit Message

Wire spec-driven replicas and session storage validation for MCPServer (#4364)

* Add CRD type changes for horizontal scaling

Add SessionStorageConfig struct and scaling fields to MCPServer and
VirtualMCPServer CRDs as the foundation for horizontal scaling support.

- Add SessionStorageConfig struct with Provider, Address, DB, KeyPrefix,
  and PasswordRef fields; CEL rule enforces address is required for redis
- Add Replicas *int32, BackendReplicas *int32, SessionStorage to MCPServerSpec
- Add Replicas *int32, SessionStorage to VirtualMCPServerSpec
- Nil pointer fields allow HPA to manage replicas without operator interference
- Regenerate zz_generated.deepcopy.go and CRD manifests

Closes: #4206

* Wire spec-driven replicas and session storage validation for MCPServer

- resolveDeploymentReplicas: nil-passthrough preserves HPA compatibility,
  stdio cap enforced at 1 as defense-in-depth
- Add terminationGracePeriodSeconds (30s default) to proxy runner pod spec
- validateStdioReplicaCap: Warning condition when stdio + spec.replicas > 1
- validateSessionStorageForReplicas: Warning condition when replicas > 1
  without Redis session storage configured

Closes #4217

---------

Co-authored-by: taskbot <taskbot@users.noreply.github.com>

Run Details

71 of 82 new or added lines in 1 file covered. (86.59%)

34 existing lines in 4 files now uncovered.

52058 of 79743 relevant lines covered (65.28%)

65.69 hits per line