New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
ceph: add networking.k8s.io/v1 Ingress chart compatability #8666
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There is a typo in your commit message.
s/Kuberenetes/Kubernetes/
Kubernetes cluster versions 1.22+ must use the v1 Ingress version. Signed-off-by: Bryton Hall <email@bryton.io>
@leseb, good catch! Fixed. |
{{- if .Capabilities.APIVersions.Has "networking.k8s.io/v1beta1" }} | ||
{{- if .Capabilities.APIVersions.Has "networking.k8s.io/v1" }} | ||
apiVersion: networking.k8s.io/v1 | ||
{{ else if .Capabilities.APIVersions.Has "networking.k8s.io/v1beta1" }} | ||
apiVersion: networking.k8s.io/v1beta1 | ||
{{ else }} | ||
apiVersion: extensions/v1beta1 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is this condition still needed?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Depends on which k8s versions rook supports. networking.k8s.io/v1
was introduced in 1.19 and networking.k8s.io/v1beta1
was introduced in 1.14. Do we want to keep the latter to support <1.19 or remove it and force those users install an older version of the chart?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I meant this one apiVersion: extensions/v1beta1
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ah, yeah. That one's pretty old but the docs say 1.11 is the minimum supported version so it should probably stay.
ceph: add networking.k8s.io/v1 Ingress chart compatability (backport #8666)
Kubernetes cluster versions 1.22+ must use the v1 Ingress version.
Signed-off-by: Bryton Hall email@bryton.io
Description of your changes:
Which issue is resolved by this Pull Request:
Resolves #
Checklist:
make codegen
) has been run to update object specifications, if necessary.