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

Add UDP support in kubernetesCRD provider #6348

Merged
merged 2 commits into from Feb 26, 2020

Conversation

@jbdoumenjou
Copy link
Contributor

jbdoumenjou commented Feb 21, 2020

What does this PR do?

This PR handles UDP in the kubernetesCRD provider thanks to the IngressRouteUDP resource.

Motivation

Being able to define UDP load balancing using a kubernetes custom resource.

More

  • Added/updated tests
  • Added/updated documentation

Additional Notes

Co-authored-by: Mathieu Lonjaret mathieu.lonjaret@gmail.com

@ldez ldez added this to To review in v2 via automation Feb 21, 2020
@ldez ldez added this to the next milestone Feb 21, 2020
@jbdoumenjou jbdoumenjou force-pushed the jbdoumenjou:feature/k8s-udp branch 2 times, most recently from 20e04ef to 5af8ceb Feb 25, 2020
Copy link
Member

juliens left a comment

LGTM

@mpl
mpl approved these changes Feb 25, 2020
@ldez
ldez approved these changes Feb 26, 2020
Copy link
Member

ldez left a comment

LGTM

@traefiker traefiker force-pushed the jbdoumenjou:feature/k8s-udp branch from 5af8ceb to 45fff8a Feb 26, 2020
@traefiker traefiker merged commit 665aeb3 into containous:master Feb 26, 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 Feb 26, 2020
@ldez ldez mentioned this pull request Feb 27, 2020
@jbdoumenjou jbdoumenjou deleted the jbdoumenjou:feature/k8s-udp branch Mar 2, 2020
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

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