search

openshift / openshift-sdn

Apache License 2.0
69 stars 63 forks source link

[DO NOT MERGE] Add flannelmt network plugin #217

Closed dcbw closed 8 years ago

dcbw commented 8 years ago

Pending flannel upstream acceptance of the flannel side, see https://github.com/coreos/flannel/pull/384

Comments/review gratefully accepted though.

danwinship commented 8 years ago

I had thought the plan was that the flannel plugin would go directly into origin, with openshift-sdn only being for the (eventually "legacy") openshift-ovs plugins?

dcbw commented 8 years ago

I had thought the plan was that the flannel plugin would go directly into origin, with openshift-sdn only being for the (eventually "legacy") openshift-ovs plugins?

We can do that too, but the current code uses quite a bit of the existing stuff on the openshift-sdn side. We could copy that to origin and sync periodically, and maybe that makes more sense since Ravi is putting stuff into origin now anyway. In the end, we still have to have somewhere for these pieces to go even when they are pure Kube and CNI plugins, and origin probably makes sense for that right now.

pravisankar commented 8 years ago

Needs rebase

dcbw commented 8 years ago

dropping this one for now...