Skip to content

feat: fork-native storage identity propagation#14

Merged
zachsmith1 merged 4 commits intomainfrom
feat/storage-identity-integration
Mar 3, 2026
Merged

feat: fork-native storage identity propagation#14
zachsmith1 merged 4 commits intomainfrom
feat/storage-identity-integration

Conversation

@zachsmith1
Copy link
Contributor

@zachsmith1 zachsmith1 commented Mar 3, 2026

Summary

  • Replace etcd lookup fallbacks, resource allowlists, revision-cluster memory maps, and CRD-specific resolver retry loops with durable identity from the kplane-dev/kubernetes fork's cacher hooks (IdentityFromKey, WrapWatchObject, UnwrapObject)
  • Introduce InformerRegistry and typedinformer package replacing InformerPool and scopedinformer

Test plan

  • 26/26 apiserver smoke tests pass (CRUD, watch, CRD lifecycle, RBAC, admission, webhooks, 200 VCP memory)
  • kplanetest conformance suite passes (2/2)
  • cluster-api-exp ./internal/... passes (36 packages, 0 failures)
  • Upstream kubernetes cacher + etcd3 test suites pass
  • 1.79 MiB/VCP heap overhead, 6.19/VCP goroutines

zachsmith1 and others added 4 commits February 19, 2026 21:17
Replace etcd lookup fallbacks, resource allowlists, revision-cluster
memory maps, and CRD-specific resolver retry loops with durable identity
from the kplane-dev/kubernetes fork's cacher hooks. All dependencies now
resolve from remote kplane-dev repos (kubernetes, storage, informer).

26 smoke tests pass. 1.79 MiB/VCP heap overhead. -3,699 lines net.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
…ne-dev/storage

Replace PlacementResolver with DefaultKeyLayout().ClusterFromKey() and
delete dead code (EntryCache, ClusterScopedObjectKey, keyutil helpers)
with zero production callers.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@zachsmith1 zachsmith1 merged commit c56d517 into main Mar 3, 2026
1 check passed
@zachsmith1 zachsmith1 deleted the feat/storage-identity-integration branch March 3, 2026 06:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant