Skip to content

Make ingress compatible with format prior to 1.19-0 k8s version #1798

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

Merged
merged 4 commits into from
Feb 22, 2023
Merged

Make ingress compatible with format prior to 1.19-0 k8s version #1798

merged 4 commits into from
Feb 22, 2023

Conversation

sylbn
Copy link
Contributor

@sylbn sylbn commented Feb 22, 2023

The PR objective is to support old ingress API version.
In our case we use a 3.11 OpenShift (kubernetes 1.11) and we had this error message:

Error: INSTALLATION FAILED: unable to build kubernetes objects from release manifest: error validating "": error validating data: [ValidationError(Ingress.spec.rules[0].http.paths[0].backend): unknown field "service" in io.k8s.api.extensions.v1beta1.IngressBackend, ValidationError(Ingress.spec.rules[0].http.paths[0].backend): missing required field "serviceName" in io.k8s.api.extensions.v1beta1.IngressBackend, ValidationError(Ingress.spec.rules[0].http.paths[0].backend): missing required field "servicePort" in io.k8s.api.extensions.v1beta1.IngressBackend, ValidationError(Ingress.spec.rules[0].http.paths[0]): unknown field "pathType" in io.k8s.api.extensions.v1beta1.HTTPIngressPath]

So I added a if to check if the kubernetes is in version prior to 1.19-0 to use the old ingress API

@CLAassistant
Copy link

CLAassistant commented Feb 22, 2023

CLA assistant check
All committers have signed the CLA.

@diemol
Copy link
Member

diemol commented Feb 22, 2023

Can you please sign the CLA and also bump de version of the Charts + Changelog so it can be released right away?

@sylbn
Copy link
Contributor Author

sylbn commented Feb 22, 2023

I signed the SLA and modified the Changelog

@diemol
Copy link
Member

diemol commented Feb 22, 2023

You also would need to update the Charts.yml file to trigger the release.

Copy link
Member

@diemol diemol left a comment

Choose a reason for hiding this comment

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

Thank you, @sylvain-brun!

@diemol diemol merged commit 595a983 into SeleniumHQ:trunk Feb 22, 2023
@sylbn
Copy link
Contributor Author

sylbn commented Feb 22, 2023

Thank you for the quick answers

prashanth-volvocars pushed a commit to prashanth-volvocars/docker-selenium that referenced this pull request Jun 29, 2023
…niumHQ#1798)

* Make ingress compatible with format prior to 1.19-0 k8s version

* Modify CHANGELOG.md

* Modify Chart version
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.

3 participants