You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
This repository has been archived by the owner on Sep 24, 2021. It is now read-only.
Suggestions for an improvement
I suggest the issue should be fixed by checking the apiVersion with .Capabilities.APIVersions.Has
{{- if .Capabilities.APIVersions.Has "apiextensions.k8s.io/v1" }}
apiVersion: apiextensions.k8s.io/v1
{{- else }}
apiVersion: apiextensions.k8s.io/v1beta1
{{- end }}
The text was updated successfully, but these errors were encountered:
lspecian-olx
changed the title
apiextensions.k8s.io/v1beta1 apiVersion removed on k8s v1.19.0
apiextensions.k8s.io/v1beta1 apiVersion removed on k8s v1.22.0
May 25, 2021
Sign up for freeto subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Description
The apiextensions.k8s.io/v1beta1 apiVersion used in the CustomResourceDefinition was removed on k8s v1.22.0
Importance
The issue is a blocker to install the chart properly on k8s clusters with version v1.22.0
Location
https://github.com/awslabs/k8s-cloudwatch-adapter/blob/master/charts/k8s-cloudwatch-adapter-crd/templates/crd.yaml#L1
Suggestions for an improvement
I suggest the issue should be fixed by checking the apiVersion with .Capabilities.APIVersions.Has
{{- if .Capabilities.APIVersions.Has "apiextensions.k8s.io/v1" }}
apiVersion: apiextensions.k8s.io/v1
{{- else }}
apiVersion: apiextensions.k8s.io/v1beta1
{{- end }}
The text was updated successfully, but these errors were encountered: