Skip to content
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

Updated ingress resources to adapt to K8S 1.17 #103

Merged
merged 2 commits into from
Sep 29, 2024
Merged

Conversation

ycyin
Copy link
Contributor

@ycyin ycyin commented Sep 29, 2024

Compatible with K8S 1.17.

The pathType field in the ingress resource is a feature introduced in K8s 1.18
Reference Links: https://opensource.googleblog.com/2020/09/kubernetes-ingress-goes-ga.html

Compatible with K8S 1.17
@LeoQuote
Copy link
Contributor

LeoQuote commented Sep 29, 2024

Will merge your pull request as long as you fixes all errors, but you really should consider upgrading your cluster, k8s 1.17 was release at 2019 and was marked eol at 2020.

Upgraded version: Compatible with K8S 1.17
@ycyin
Copy link
Contributor Author

ycyin commented Sep 29, 2024

Thank you for the suggestion, but we can't upgrade the k8s version until the transformation of our services that depend on k8s is complete, please help approve the workflow again.

@ycyin ycyin changed the title Update ingress.yaml Updated ingress resources to adapt to K8S 1.17 Sep 29, 2024
@LeoQuote LeoQuote merged commit 0087a96 into douban:master Sep 29, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants