Skip to content

Feature/target loading architecture#45

Open
denyost wants to merge 37 commits intomainfrom
feature/target-loading-architecture
Open

Feature/target loading architecture#45
denyost wants to merge 37 commits intomainfrom
feature/target-loading-architecture

Conversation

@denyost
Copy link
Copy Markdown
Collaborator

@denyost denyost commented Apr 10, 2026

Implemented the TargetSource loading pipeline within the package discovery:

  • discovery/core defines the loader interface and other necessary types
  • discovery/loaders hold the implementations for the different loader types, which observe the changes from the target sources
  • discovery/loader.go serves as the entrypoint, from which the corresponding loader gets called
  • target_manager.go consumes discovered target snapshots and reconciles them into Kubernetes Target CRs.
  • targetsource_controller.go orchestrates lifecycle management, wiring loaders and target managers together while keeping the reconcile loop focused on start/stop logic only.

@denyost denyost self-assigned this Apr 10, 2026
@denyost
Copy link
Copy Markdown
Collaborator Author

denyost commented Apr 10, 2026

Implemented the TargetSource loading pipeline:

  • loaders.go defines a pluggable loader interface and registry; loaders observe external sources of truth and emit target snapshots via channels.
  • target_manager.go consumes discovered target snapshots and reconciles them into Kubernetes Target CRs.
  • targetsource_controller.go orchestrates lifecycle management, wiring loaders and target managers together while keeping the reconcile loop focused on start/stop logic only.

@cloudflare-workers-and-pages
Copy link
Copy Markdown

cloudflare-workers-and-pages bot commented Apr 10, 2026

Deploying gnmic-operator with  Cloudflare Pages  Cloudflare Pages

Latest commit: 7a94e3f
Status: ✅  Deploy successful!
Preview URL: https://a2c17dad.gnmic-operator2.pages.dev
Branch Preview URL: https://feature-target-loading-archi.gnmic-operator2.pages.dev

View logs

@denyost denyost marked this pull request as ready for review April 16, 2026 17:44
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.

2 participants