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

Watch status subresource updates to trigger reconcile #364

Merged
merged 1 commit into from
Nov 27, 2023

Conversation

scothis
Copy link
Contributor

@scothis scothis commented Nov 22, 2023

ServiceBindings setup a validating webhook to watch for changes on the service resource. This was only watching for requests on the main resource and did not observe the status subresource. This meant that if a provisioned service rotated the secret name and made no other changes, the new secret would not be observed until the regular reconcile of the ServiceBinding (up to 10 hours later).

This change watches the status subresource for tracked resources.

ServiceBindings setup a validating webhook to watch for changes on the
service resource. This was only watching for requests on the main
resource and did not observe the status subresource. This meant that if
a provisioned service rotated the secret name and made no other changes,
the new secret would not be observed until the regular reconcile of the
ServiceBinding (up to 10 hours later).

This change watches the status subresource for tracked resources.

Signed-off-by: Scott Andrews <[email protected]>
@scothis scothis requested review from sadlerap and a team November 22, 2023 17:37
@sadlerap sadlerap merged commit 0973d3d into servicebinding:main Nov 27, 2023
12 checks passed
@scothis scothis deleted the trigger-on-status branch November 27, 2023 21:21
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