chore(helm): AS-711 remove support for old kubernetes versions

[afoley587]

Rationale

As discussed, any Kubernetes version older than 1.30.0 is EOL and no longer supported by kubernetes. Cloud providers may support as far back as 1.27. For example, Google supports 1.27.

In my opinion, taking a stance that we support non-EOL kubernetes versions seems like a decent stance to take. I am open to discussions on the PR!

Changes

Moves our minimum version from 1.18 to 1.30 in accordance with the kubernetes release lifecycle.

Note

We will need to make a corresponding voxel51/fiftyone PR as well to update deprecation.voxel51.com.

Checklist

• This PR maintains parity between Docker Compose and Helm

Testing

Docs only PR

[findtopher]

seems like we could simplify our ingress template since we can now make the base assumption that we're > 1.19...

helm/fiftyone-teams-app/templates/ingress.yaml
3:{{- if and .Values.ingress.className (not (semverCompare ">=1.18-0" .Capabilities.KubeVersion.GitVersion)) }}
8:{{- if semverCompare ">=1.19-0" .Capabilities.KubeVersion.GitVersion -}}
24:  {{- if and .Values.ingress.className (semverCompare ">=1.18-0" .Capabilities.KubeVersion.GitVersion) }}
48:            {{- if semverCompare ">=1.18-0" $.Capabilities.KubeVersion.GitVersion }}
52:              {{- if semverCompare ">=1.19-0" $.Capabilities.KubeVersion.GitVersion }}
64:            {{- if semverCompare ">=1.18-0" $.Capabilities.KubeVersion.GitVersion }}
68:              {{- if semverCompare ">=1.19-0" $.Capabilities.KubeVersion.GitVersion }}
83:            {{- if semverCompare ">=1.18-0" $.Capabilities.KubeVersion.GitVersion }}
87:              {{- if semverCompare ">=1.19-0" $.Capabilities.KubeVersion.GitVersion }}

[findtopher]

:thank_you:

🙇‍♂️

[kevin-dimichel]

Might now be a good time to clean up other references to old k8s versions?

For example:

• https://github.com/voxel51/fiftyone-teams-app-deploy/blob/main/tests/unit/helm/ingress_test.go

• fiftyone-teams-app-deploy/helm/fiftyone-teams-app/values.yaml

  Line 652 in da863f9

  # When not empty and Kubernetes version is >1.18.0, this value will be the Ingress class name.

  • which will update the references in README.md and README.md.gotmlp too

[findtopher]