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

fix missing pathType in ingress.yaml for kube cluster version 1.22+ #1450

Merged
merged 3 commits into from
Jan 23, 2022

Conversation

5hin0bi
Copy link
Contributor

@5hin0bi 5hin0bi commented Jan 21, 2022

fixes: #1448

pathType for preceeding and succeeding paths was also changed from ImplementationSpecific to Prefix according to the kube docs: https://kubernetes.io/docs/concepts/services-networking/ingress/

@5hin0bi 5hin0bi self-assigned this Jan 21, 2022
@5hin0bi 5hin0bi added type/bug Something isn't working scope/k8s K8s or helm stuff (really annoying) labels Jan 21, 2022
Copy link
Contributor

@azatsafin azatsafin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGMT

@5hin0bi 5hin0bi merged commit 86dad04 into master Jan 23, 2022
@5hin0bi 5hin0bi deleted the issues/1448 branch January 23, 2022 13:56
javalover123 pushed a commit to javalover123/kafka-ui that referenced this pull request Dec 7, 2022
…rovectus#1450)

* fix missing pathType in ingress.yaml kube cluster version 1.22+

* correct placing of pathType in ingress.yaml
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
scope/k8s K8s or helm stuff (really annoying) type/bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Missing Ingress pathType with k8s >=1.22
2 participants