Closed hardys closed 2 months ago
Setting this back to draft as @Danil-Grigorev pointed out that the repository override we used via clusterctl isn't working via the CAPI operator:
> kubectl get deployment -n capi-system -o yaml | grep image
image: registry.k8s.io/cluster-api/cluster-api-controller:v1.7.5
imagePullPolicy: IfNotPresent
> kubectl get deployment -n rke2-bootstrap-system -o yaml | grep image
image: ghcr.io/rancher/cluster-api-provider-rke2-bootstrap:v0.6.1
imagePullPolicy: IfNotPresent
I will try to find a different approach via the providers section
The scenario with repository images
overrides via clusterctl.yaml
will be working once https://github.com/kubernetes-sigs/cluster-api-operator/pull/598 is merged and released. cc @alexander-demicev
Ok this is now working with all images overridden to the OBS ones:
> for ns in capi-system capm3-system rke2-bootstrap-system rke2-control-plane-system; do kubectl get deployment -n $ns -o yaml | grep "image:"; done
image: registry.opensuse.org/isv/suse/edge/clusterapi/containers/images/cluster-api-controller:1.7.5
image: registry.opensuse.org/isv/suse/edge/clusterapi/containers/images/cluster-api-provider-metal3:1.7.1
image: registry.opensuse.org/isv/suse/edge/clusterapi/containers/images/ip-address-manager:1.7.1
image: registry.opensuse.org/isv/suse/edge/clusterapi/containers/images/cluster-api-provider-rke2-bootstrap:0.6.1
image: registry.opensuse.org/isv/suse/edge/clusterapi/containers/images/cluster-api-provider-rke2-controlplane:0.6.1
@alknopfler and @Danil-Grigorev please could you take another look and check if you're happy with this revision? Thanks!
This rebases to the latest 0.11.0 turtles release, and also rebases the versions of Core CAPI, CAPM3 and the RKE2 provider.
Note that to enable this to install reliably via EIB we currently need a workaround for https://github.com/rancher/turtles/issues/627 until a version with https://github.com/rancher/turtles/pull/705 is released, passing a values file like this should work:
It's then also necessary to pass a manifest as follows since the chart pre-install hook is no longer taking care of this.