Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Existing Ownership of sub resoucres [Edge Case] #397

Closed
Boomatang opened this issue Jan 19, 2024 · 1 comment
Closed

Existing Ownership of sub resoucres [Edge Case] #397

Boomatang opened this issue Jan 19, 2024 · 1 comment
Labels
kind/bug Something isn't working

Comments

@Boomatang
Copy link
Contributor

Outline

With the merging of #350 and #393 the kuadrant operator will respect existing ownership values on the limitador and authorino CRs. Once an existing ownership is detected the kuadrant operator reports the error in the kuadrant operator but also stops reconciling the rest of the resources. The example here is if the limitador CR is owned by an existing controller none of the resources for authorino will be reconciled. And an infinite loop of trying to reconcile is created.

Cause

How I believe a user may get into this state is as follows. The user was using one of the sub components (authorino or limitador) as a standalone product before adopting kuadrant as a whole. For the sub component the user would make use of sub component CR. They would have needed to created their owner custom controller to manage that resource and added the owerReference to the CR metadata. Or the user uses a GitOps system to manage the resource and that GitOps system adds the owerReference to the resources.

My personal belief would be a GitOps system adding an owerReference being the most likely cause of this edge case.

Expected Behaviour

  • Error report added to the Kuadrant CR status block - Currently happens
  • Stopping reconcile of resource which has an existing owerReference set - Currently happens
  • Continue to reconcile resources not related to the affected resource. - Does not happen.
  • Exposing metric, alerts and events around the issue. - Unknown current state
@Boomatang Boomatang added the kind/bug Something isn't working label Jan 19, 2024
@Boomatang
Copy link
Contributor Author

Closing: The state of the world refactor fixed this issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/bug Something isn't working
Projects
Status: Done
Development

No branches or pull requests

1 participant