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 limitation note of using host-port #1615

Merged
merged 3 commits into from
Nov 22, 2016
Merged

Conversation

linfan
Copy link
Contributor

@linfan linfan commented Nov 3, 2016

This change is Reviewable

@@ -14,6 +14,8 @@ This page lists some of the available add-ons and links to their respective inst
* [Canal](https://github.com/tigera/canal/tree/master/k8s-install/kubeadm) unites Flannel and Calico, providing networking and network policy.
* [Romana](http://romana.io) is a Layer 3 networking solution for pod networks that also supports the [NetworkPolicy API](/docs/user-guide/networkpolicies/). Kubeadm add-on installation details available [here](https://github.com/romana/romana/tree/master/containerize).

Limitation: Due to [#31307](https://github.com/kubernetes/kubernetes/issues/31307), `HostPort` won't work with CNI networking plugin at the moment. That means all `hostPort` attribute in pod would be simply ignored.
Copy link
Member

Choose a reason for hiding this comment

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

Not sure this is the right place for this note since this isn't really talking about CNI specifically and should just be a list of network plugins?

Maybe it belongs here instead http://kubernetes.io/docs/admin/network-plugins/#cni

WDYT?

@linfan
Copy link
Contributor Author

linfan commented Nov 3, 2016

@caseydavenport Sure, good keep the add-on list clean : )
Have move it to the network-plugin page.

@linfan
Copy link
Contributor Author

linfan commented Nov 4, 2016

Have re-pushed the code, there was one commit used incorrect email and failed the check.

@thockin
Copy link
Member

thockin commented Nov 15, 2016

cc @freehan

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.

None yet

6 participants