Skip to content

Fix provider reconciler update loop#89

Merged
ruibaby merged 1 commit into
mainfrom
fix/provider-reconciler-update-loop
Jul 20, 2026
Merged

Fix provider reconciler update loop#89
ruibaby merged 1 commit into
mainfrom
fix/provider-reconciler-update-loop

Conversation

@ruibaby

@ruibaby ruibaby commented Jul 20, 2026

Copy link
Copy Markdown
Member

What

  • Avoid updating an AiProvider when its finalizer is already present.
  • Add regression coverage for idempotent provider reconciliation.

Why

AiProviderReconciler previously updated every non-deleted provider during each reconciliation, even when addFinalizers made no change. Since provider updates enqueue another reconciliation, this could create a continuous update loop and repeatedly invalidate cached provider clients.

How to test

./gradlew build
git diff --check

@ruibaby
ruibaby merged commit b4bd9a4 into main Jul 20, 2026
1 check passed
@ruibaby
ruibaby deleted the fix/provider-reconciler-update-loop branch July 20, 2026 07:17
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