charts icon indicating copy to clipboard operation
charts copied to clipboard

MinIO bundled chart does not support Ingress networking.k8s.io/v1

Open boomaker opened this issue 2 years ago • 2 comments

Summary

Sorry cypress Helm chart bundles MinIO v8.0.9 which does not support Ingresses version networking.k8s.io/v1 for Kubernetes 1.24+

This is caused by the following block in the MinIO _helpers.tpl:

{{/*
Return the appropriate apiVersion for ingress.
*/}}
{{- define "minio.ingress.apiVersion" -}}
{{- if semverCompare "<1.14-0" .Capabilities.KubeVersion.GitVersion -}}
{{- print "extensions/v1beta1" -}}
{{- else -}}
{{- print "networking.k8s.io/v1beta1" -}}
{{- end -}}
{{- end -}}

Environment

  • sorry-cypress version: 2.4.2
  • cypress version: various
  • helm chart version: 1.7.10

boomaker avatar Dec 05 '22 15:12 boomaker

I can patch the MinIO _helpers.tpl and create a PR with the new .tgz but I'm not convinced it's a good practice. WDYT?

boomaker avatar Dec 08 '22 12:12 boomaker

I think the chart needs to update the dependencies and use the new minio chart, that should solve the issues "permanently" and in a more "best-practice" oriented manner...

phoenix-henri avatar Mar 03 '23 16:03 phoenix-henri