Skip to content

fix: scope ServiceSync service watches#22

Open
PKizzle wants to merge 1 commit into
jr42:mainfrom
PKizzle:fix/servicesync-watch-scope
Open

fix: scope ServiceSync service watches#22
PKizzle wants to merge 1 commit into
jr42:mainfrom
PKizzle:fix/servicesync-watch-scope

Conversation

@PKizzle

@PKizzle PKizzle commented May 22, 2026

Copy link
Copy Markdown
Contributor

Summary

  • index LoadBalancer Services by dynamic-prefix.io/name for DynamicPrefix fan-out
  • use the index in ServiceSync instead of listing every Service on DynamicPrefix changes
  • add optional --service-cache-label-selector and Helm value to narrow the Service informer cache for opt-in deployments
  • document labeling HA-managed Services when enabling the cache selector

Testing

  • go test ./internal/controller -count=1
  • make lint
  • make helm-lint

Fixes #20

@PKizzle PKizzle changed the title Scope ServiceSync service watches fix: scope ServiceSync service watches May 22, 2026
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.

servicesync watches all Services cluster-wide; add opt-in predicate

1 participant