rancher-sandbox / cluster-api-provider-harvester

A Cluster API Infrastructure Provider for Harvester
Apache License 2.0
21 stars 6 forks source link

kind/feature: Changing ClusterResourceSets Rolloutstrategy from ApplyOnce to Reconcile #41

Closed PatrickLaabs closed 2 months ago

PatrickLaabs commented 2 months ago

What this PR does / why we need it:

This PR changes the way we configure the ClusterResourceSets Rolloutstrategy from 'ApplyOne' to 'Reconcile'

This ensures, that the desired CRS are always synced back to the attached Clusters (mgmt/workload), so that a User does not need to manually re-create the CRS on the attached clusters if those are being deleted.

Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged): Fixes #39

Special notes for your reviewer:

Checklist:

belgaied2 commented 2 months ago

LGTM