-
Notifications
You must be signed in to change notification settings - Fork 2.5k
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 #1540 source: explicitly handle default annotations #1560
Conversation
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: squat The full list of commands accepted by this bot can be found here.
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
Welcome @squat! |
11ab08b
to
3448610
Compare
This commit fixes a bug that causes the Cloudflare provider to delete and recreate all records every minute. The controller is running into trouble when reconciling endpoints with values with implicit defaults. This is caused because the providers produce records with all values explicitly set, however the sources generate endpoints with implicit provider-specific values, i.e. when an annotation is not set. This causes the planner to always detect a diff. xref: kubernetes-sigs#1540 Signed-off-by: Lucas Servén Marín <lserven@gmail.com>
3448610
to
68ab84a
Compare
sounds good, I didn't want to go that route to avoid adding extra complexity to the system but I am happy with simply having a fix. Let's close 1540 as a dupe of 1463 for better visibility and you can close this PR if you like (: |
@squat: PR needs rebase. Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
Should we close this one @sheerun ? |
Yes I think so, #1536 will be merged very soon /close |
@sheerun: Closed this PR. In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
This commit fixes a bug that causes the Cloudflare provider to delete
and recreate all records every minute.
The controller is running into trouble when reconciling endpoints with
values with implicit defaults. This is caused because the providers
produce records with all values explicitly set, however the sources
generate endpoints with implicit provider-specific values, i.e. when an
annotation is not set. This causes the planner to always detect a diff.
Fixes: #1540
Signed-off-by: Lucas Servén Marín lserven@gmail.com