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

Support 'networking.k8s.io/v1beta1' ingress apiVersion #6171

Merged
merged 4 commits into from Jan 16, 2020

Conversation

@ldez
Copy link
Member

ldez commented Jan 14, 2020

What does this PR do?

Adds support of 'networking.k8s.io/v1beta1' as ingress apiVersion.

The following apiVersion are supported:

  • networking.k8s.io/v1beta1
  • extensions/v1beta1

Motivation

https://kubernetes.io/blog/2019/07/18/api-deprecations-in-1-16/

More

  • Added/updated tests
  • Added/updated documentation

Additional Notes

@ldez ldez added this to the next milestone Jan 14, 2020
@ldez ldez added this to To review in v2 via automation Jan 14, 2020
@traefiker traefiker added the size/M label Jan 14, 2020
@ldez ldez force-pushed the ldez:fix/networking-ingress branch from 87283db to 0a6734e Jan 15, 2020
@ldez ldez force-pushed the ldez:fix/networking-ingress branch from f69d734 to cf28e85 Jan 15, 2020
@mpl
mpl approved these changes Jan 15, 2020
Copy link
Contributor

jbdoumenjou left a comment

LGTM

Copy link
Member

juliens left a comment

LGTM

@traefiker traefiker merged commit 72e702a into containous:master Jan 16, 2020
3 checks passed
3 checks passed
continuous-integration/travis-ci/pr The Travis CI build passed
Details
deploy/netlify Deploy preview ready!
Details
semaphoreci The build passed on Semaphore.
Details
v2 automation moved this from To review to Done Jan 16, 2020
@ldez ldez deleted the ldez:fix/networking-ingress branch Jan 16, 2020
@ravilr

This comment has been minimized.

Copy link
Contributor

ravilr commented Jan 22, 2020

@ldez Any plans to support and backport this in traefik v1.7 release branch also ?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
v2
Done
Linked issues

Successfully merging this pull request may close these issues.

None yet

6 participants
You can’t perform that action at this time.