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

ingress: Add support for tcp and udp services #2142

Merged
merged 1 commit into from
Nov 1, 2017

Conversation

fabiand
Copy link
Contributor

@fabiand fabiand commented Nov 1, 2017

This patch adds awareness for TCP and UDP services.
We need this as a preparation to support KubeVirt on minikube (i.e. we need to export a SPICE proxy).

@k8s-ci-robot k8s-ci-robot added cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. labels Nov 1, 2017
@minikube-bot
Copy link
Collaborator

Can one of the admins verify this patch?

@r2d4
Copy link
Contributor

r2d4 commented Nov 1, 2017

@minikube-bot ok to test

Copy link
Contributor

@r2d4 r2d4 left a comment

Choose a reason for hiding this comment

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

LGTM

@codecov-io
Copy link

Codecov Report

Merging #2142 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #2142   +/-   ##
=======================================
  Coverage   28.59%   28.59%           
=======================================
  Files          82       82           
  Lines        5372     5372           
=======================================
  Hits         1536     1536           
  Misses       3640     3640           
  Partials      196      196

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 9d9a581...fb4c312. Read the comment docs.

@r2d4 r2d4 merged commit 38e719a into kubernetes:master Nov 1, 2017
@fabiand fabiand deleted the ingressWithUdpAndTcp branch February 16, 2018 06:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. size/XS Denotes a PR that changes 0-9 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants