Skip to content

Feature/target loading architecture#45

Merged
karimra merged 34 commits into
mainfrom
feature/target-loading-architecture
Apr 26, 2026
Merged

Feature/target loading architecture#45
karimra merged 34 commits into
mainfrom
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
@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: 2e863cf
Status: ✅  Deploy successful!
Preview URL: https://96a81c91.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
@Janooski
Copy link
Copy Markdown
Collaborator

Janooski commented Apr 20, 2026

Hey @karimra, can we merge this request and #43 as discussed in the meeting last Thursday?

Comment thread api/v1alpha1/targetsource_types.go Outdated
Comment thread api/v1alpha1/targetsource_types.go Outdated
Comment thread internal/controller/discovery/loaders/http_pull/loader.go Outdated
Comment thread internal/controller/discovery/loaders/http_push/loader.go Outdated
@mcdillson
Copy link
Copy Markdown
Collaborator

@karimra I merged the current state of feature/targetsource-crd-fields (#51) into this branch. in the corresponding PR I added some explanations about the kubebuilder annotations.

@denyost denyost requested a review from karimra April 22, 2026 11:58
@denyost denyost added the enhancement New feature or request label Apr 24, 2026
@karimra karimra merged commit 4d1f8c3 into main Apr 26, 2026
11 checks passed
@denyost denyost deleted the feature/target-loading-architecture branch April 30, 2026 07:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants