Skip to content
This repository was archived by the owner on Oct 28, 2024. It is now read-only.

✨ Update CAPI VC Controller to Use a Cluster watch. #137

Closed
christopherhein opened this issue Jun 16, 2021 · 0 comments · Fixed by #200
Closed

✨ Update CAPI VC Controller to Use a Cluster watch. #137

christopherhein opened this issue Jun 16, 2021 · 0 comments · Fixed by #200
Labels
kind/cleanup Categorizes issue or PR as related to cleaning up code, process, or technical debt.
Milestone

Comments

@christopherhein
Copy link
Contributor

christopherhein commented Jun 16, 2021

User Story

As an operator I would like to the CAPI Controller for VC to watch the Cluster resource for updates so I don't need to poll the Cluster resource for changes.

Detailed Description

Currently in #136 this used ctrl.Result{RequeueAfter} which is polling the cache for updates to the Cluster object, this should be done by triggering a change when the Cluster changes instead making the system more efficient.

Anything else you would like to add:

Xref: #136

/kind cleanup
/milestone v0.1.x

@k8s-ci-robot k8s-ci-robot added this to the v0.1.x milestone Jun 16, 2021
@k8s-ci-robot k8s-ci-robot added the kind/cleanup Categorizes issue or PR as related to cleaning up code, process, or technical debt. label Jun 16, 2021
@christopherhein christopherhein changed the title ✨ Update CAPI VC Controller to Use a Cluster watch. Jun 16, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
kind/cleanup Categorizes issue or PR as related to cleaning up code, process, or technical debt.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants