-
-
Notifications
You must be signed in to change notification settings - Fork 4
feat(crd): Stabilise Entra backend #801
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
Conversation
dervoeti
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good!
Just one thing: In the docs I still see opa.stackable.tech/v1alpha1 in some places / examples. This should be opa.stackable.tech/v1alpha2 now?
|
Oh yeah, you are right. I will update them asap. |
Release NotePlatform improvementThe Entra User Info Fetcher (UIF) backend has been stabilized with the introduction of OpaCluster |
Implements #791.
This PR stabilises the Entra backend by introducing
v1alpha2and adds a conversion webhook to convert betweenv1alpha1andv1alpha2.There is no Entra specific integration test with which this change can be tested, but applying a
v1alpha1OpaCluster results in a successful conversion tov1alpha2. The downgrade also works without any issues.Note for reviewers: It might be easier to review this on a commit-by-commit basis.