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

fix: Remove dnspolicy lb immutable validation #968

Merged

Conversation

mikenairn
Copy link
Member

Removes the immutable validation on the loadBalancing field of the dnspolicy spec.

Updates made as part of section name support allow the controller to handle changes to endpoint record types, so it's now possible to switch from what we used to refer to as simple policy types (loadBalancing field not set) to loadbalanced (loadBalancing field set) and vice versa without deleting and re-creating the policy.

Removes the validation on the loadBalancing field of the dnspolicy spec.
Updates made as part of section name support allow the controller to
handle changes to endpoint record types. It is now possible to switch
from what we used to refer to as simple policy types (loadBalancing
field not set) to loadbalanced (loadBalancing field set) and vice versa
without deleting and re-creating the policy.

Signed-off-by: Michael Nairn <mnairn@redhat.com>
Copy link
Collaborator

@maleck13 maleck13 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/lgtm

@mikenairn mikenairn merged commit 27a29d6 into Kuadrant:main Oct 31, 2024
25 checks passed
@mikenairn mikenairn deleted the dnspolicy_remove_immutable_field_validation branch October 31, 2024 15:04
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