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

Update networking.md to list ACI CNI plugin #6367

Merged
merged 2 commits into from
Jan 11, 2018
Merged

Update networking.md to list ACI CNI plugin #6367

merged 2 commits into from
Jan 11, 2018

Conversation

anvega
Copy link

@anvega anvega commented Nov 17, 2017

Included ACI CNI plugin in "How to achieve this" networking options.


This change is Reviewable

@k8s-ci-robot k8s-ci-robot added the size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. label Nov 17, 2017
@k8s-ci-robot
Copy link
Contributor

Thanks for your pull request. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

📝 Please follow instructions at https://github.com/kubernetes/kubernetes/wiki/CLA-FAQ to sign the CLA.

It may take a couple minutes for the CLA signature to be fully registered; after that, please reply here with a new comment and we'll verify. Thanks.


Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@k8s-ci-robot k8s-ci-robot added the cncf-cla: no Indicates the PR's author has not signed the CNCF CLA. label Nov 17, 2017
@anvega
Copy link
Author

anvega commented Nov 17, 2017

CLA signed.

@k8s-ci-robot k8s-ci-robot added cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. and removed cncf-cla: no Indicates the PR's author has not signed the CNCF CLA. labels Nov 17, 2017
@k8sio-netlify-preview-bot
Copy link
Collaborator

k8sio-netlify-preview-bot commented Nov 17, 2017

Deploy preview ready!

Built with commit 004b677

https://deploy-preview-6367--kubernetes-io-master-staging.netlify.com

@xiangpengzhao
Copy link
Contributor

### ACI

[Cisco Application Centric Infrastructure](https://www.cisco.com/c/en/us/solutions/data-center-virtualization/application-centric-infrastructure/index.html) offers an integrated overlay and underlay SDN solution that supports containers, virtual machines, and bare metal servers. [ACI](github.com/noironetworks/aci-containers) provides container networking integration for ACI. An overview of the integration is provided [here](https://www.cisco.com/c/dam/en/us/solutions/collateral/data-center-virtualization/application-centric-infrastructure/solution-overview-c22-739493.pdf)

Copy link
Contributor

@steveperry-53 steveperry-53 Nov 28, 2017

Choose a reason for hiding this comment

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

The ACI link is broken in the preview. It needs https://.

Copy link
Author

@anvega anvega left a comment

Choose a reason for hiding this comment

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

Modified url to include https://

@tengqm tengqm closed this Jan 9, 2018
@tengqm tengqm reopened this Jan 9, 2018
@steveperry-53 steveperry-53 merged commit 6d01b3a into kubernetes:master Jan 11, 2018
chenopis added a commit that referenced this pull request Jan 12, 2018
…henopis-user-journeys

* 'master' of https://github.com/kubernetes/website:
  fix admission plugins orders (#6900)
  link to other terms (#6625)
  Fix See Also links. (#6948)
  --experimental-nvidia-gpus error (#6527)
  Add HA guide for kubeadm (#6458)
  remove duplicated and conflicted TOC instrument
  fix deadlink
  Clarify the token webhook posts the TokenReview object
  Update federation.md
  Update create-cluster-kubeadm.md
  Add note about viewing changes locally. (#6936)
  Create page that lists supported doc versions. (#6882)
  Document how to generate reference pages. (#6366)
  Update networking.md to list ACI CNI plugin (#6367)
  Update kubeadm-init.md (#6932)
  fix typo in doc kubeadm-alpha
  dulplicated sentences
  Update certificates.md
  Replace all kube-ui with kubernetes dashboard
  Add openstack internal load balancer option
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.

7 participants