rancher-turtles: Remove rancher-turtles chart #351
Reference in New Issue
Block a user
Delete Branch "steven.hardy/Factory:rm_turtles"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
The rancher-turtles and rancher-turtles-airgap-resources charts
have been replaced by the rancher-integrated turtles in 2.13 and
the edge rancher-turtles-providers chart
LGTM, do we want to wait on an upstream release number to continue ?
@@ -179,0 +171,4 @@- prettyName: RancherTurtlesProvidersreleaseName: rancher-turtles-providerschart: '%%CHART_REPO%%/%%CHART_PREFIX%%rancher-turtles-providers'version: '%%CHART_MAJOR%%.0.4+up0.0.0'Still no upstream release number ?
Not yet AFAICS - this was derived from https://github.com/rancher/turtles/tree/main/charts/rancher-turtles-providers but it's not yet been released
I did remove a lot of the content from that chart, so another option is we don't consider this derived from the upstream chart and just copy the parts which are relevant to us (the RKE2 controlplane/bootstrap provider and the Fleet addon provider, everything else in https://github.com/rancher/turtles/tree/main/charts/rancher-turtles-providers/templates is currently not relevant/supported for the edge use-cases)
That said in future it's possible we'll support additional providers which are enabled by the upstream chart, so I wanted to try to align as much as possible with the same approach.
We can keep it for now, but felt strange to me they still didn't get a release for it
I have queried the timeline for an upstream release, but I'll merge this pending a response, and if necessary we can make a follow-up change to adjust the version and remove the +up suffix - I see there are a couple of recent fixes in the upstream chart we should add so I'll merge this to enable a PR containing those changes.