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

Remove the experimental support for multiple networks. #633

Merged
merged 1 commit into from
Apr 19, 2017

Conversation

tomdee
Copy link
Contributor

@tomdee tomdee commented Mar 10, 2017

It is not widely used and it makes the code complicated.

There are a few other fixes in this PR (which would be split out if needed)

  • Add minikube-* targets to the make file to make it easier to test the kube-subnet-manager
  • Change CMD->ENTRYPOINT in the Dockerfiles (which makes the container a little easier to use)

I added an automated test using multiple networks (with multiple daemons)
and I added documentation describing how to use multiple networks (with multiple daemons)

@tomdee tomdee force-pushed the remove-multiple-networks branch 2 times, most recently from 9a98b61 to 47bfe96 Compare March 29, 2017 18:45
@tomdee tomdee force-pushed the remove-multiple-networks branch 2 times, most recently from ed59d3e to e00c9cd Compare April 12, 2017 23:07
@tomdee tomdee force-pushed the remove-multiple-networks branch 2 times, most recently from 321a78f to 915f291 Compare April 19, 2017 21:28
It is not widely used and it makes the code complicated.
@tomdee tomdee force-pushed the remove-multiple-networks branch from 915f291 to 01afb49 Compare April 19, 2017 21:31
@tomdee tomdee merged commit 1ab041f into flannel-io:master Apr 19, 2017
@tomdee tomdee deleted the remove-multiple-networks branch April 19, 2017 23:07
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.

1 participant